search for: have_python

...AC_MSG_RESULT([$PYTHON_INCLUDEDIR]) + # Debian: python-2.7.pc, python-3.2.pc + PKG_CHECK_MODULES([PYTHON], [python-"$PYTHON_VERSION"],[ + AC_SUBST([PYTHON_CFLAGS]) + AC_SUBST([PYTHON_LIBS]) + AC_SUBST([PYTHON_VERSION]) + AC_DEFINE([HAVE_PYTHON],[1],[Python library found at compile time]) + ],[ + PKG_CHECK_MODULES([PYTHON], [python],[ + AC_SUBST([PYTHON_CFLAGS]) + AC_SUBST([PYTHON_LIBS]) + AC_SUBST([PYTHON_VERSION]) + AC_DEFINE([HAVE_PYTHON],[1],[Python library...

...DIR]) > + # Debian: python-2.7.pc, python-3.2.pc > + PKG_CHECK_MODULES([PYTHON], [python-"$PYTHON_VERSION"],[ > + AC_SUBST([PYTHON_CFLAGS]) > + AC_SUBST([PYTHON_LIBS]) > + AC_SUBST([PYTHON_VERSION]) > + AC_DEFINE([HAVE_PYTHON],[1],[Python library found at compile time]) > + ],[ > + PKG_CHECK_MODULES([PYTHON], [python],[ > + AC_SUBST([PYTHON_CFLAGS]) > + AC_SUBST([PYTHON_LIBS]) > + AC_SUBST([PYTHON_VERSION]) > + AC_DEFINE([HAV...

...ian: python-2.7.pc, python-3.2.pc > > + PKG_CHECK_MODULES([PYTHON], [python-"$PYTHON_VERSION"],[ > > + AC_SUBST([PYTHON_CFLAGS]) > > + AC_SUBST([PYTHON_LIBS]) > > + AC_SUBST([PYTHON_VERSION]) > > + AC_DEFINE([HAVE_PYTHON],[1],[Python library found at compile time]) > > + ],[ > > + PKG_CHECK_MODULES([PYTHON], [python],[ > > + AC_SUBST([PYTHON_CFLAGS]) > > + AC_SUBST([PYTHON_LIBS]) > > + AC_SUBST([PYTHON_VERSION]) > > +...

...([PACKAGE_VERSION_FULL]) + dnl Early gnulib initialization. HEADING([Configuring Gnulib]) gl_EARLY @@ -79,8 +105,6 @@ dnl These are required to get common/*/Makefile.am files dnl to work. We should further decouple these in future XXX AM_CONDITIONAL([ENABLE_APPLIANCE],[true]) AM_CONDITIONAL([HAVE_PYTHON],[false]) dnl Disables a test -AC_DEFINE_UNQUOTED([PACKAGE_VERSION_FULL],["$PACKAGE_VERSION"],[Version string.]) -AC_SUBST([PACKAGE_VERSION_FULL], [$PACKAGE_VERSION]) dnl Produce output files. HEADING([Generating output files]) -- 2.23.0

...rint (sys.version_info@<:@1@:>@)"` PYTHON_VERSION="$PYTHON_VERSION_MAJOR.$PYTHON_VERSION_MINOR" AC_MSG_RESULT([$PYTHON_VERSION]) + # Debian: python-2.7.pc, python-3.2.pc PKG_CHECK_MODULES([PYTHON], [python-"$PYTHON_VERSION"],[ + have_python_module=1 AC_SUBST([PYTHON_CFLAGS]) AC_SUBST([PYTHON_LIBS]) AC_SUBST([PYTHON_VERSION]) AC_DEFINE([HAVE_PYTHON],[1],[Python library found at compile time]) ],[ PKG_CHECK_MODULES([PYTHON], [python],[ + have_python...

...]) AC_CHECK_PROGS([PYCODESTYLE],[pycodestyle],[no]) - AM_CONDITIONAL([HAVE_PYCODESTYLE], [test "x$PYCODESTYLE" != "xno"]) fi AC_SUBST(PYTHON_PREFIX) @@ -98,3 +97,5 @@ AS_IF([test "x$enable_python" != "xno"],[ ]) AM_CONDITIONAL([HAVE_PYTHON], [test "x$PYTHON" != "xno" && test "x$have_python_module" = "x1" ]) +AM_CONDITIONAL([HAVE_PYCODESTYLE], + [test -n "$PYCODESTYLE" && test "x$PYCODESTYLE" != "xno"]) -- 2.24.1

This is the same fix I posted a few days ago for hivex (<1369351909-23821-3-git-send-email-bengen@hilluzination.de>). Cheers, -Hilko

Building against multiple versions of Python is a lot of fun as it is. Add autoconf/automake/libtool (and possible distribution-specific stuff) to the mix and things get really interesting: On Debian and Ubuntu, building against python 3.3 requires an extra include path as documented in <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=692387>. As discussed in

On Thu, Jan 09, 2020 at 06:37:49PM +0100, Pino Toscano wrote: > There is nothing in the Python bindings that require results from > configure, and gnulib is not used already. > --- > generator/python.ml | 6 ------ > python/handle.c | 2 -- > 2 files changed, 8 deletions(-) I guess the motivation is because we cannot create this header correctly from PyPi so it'll be

This commit starts to split our massive, monolithic configure.ac file into smaller files, using the m4_include mechanism to combine them. I don't know if we should really do this, so I'm open to comments about it. However: - Our configure.ac script is 1800+ lines long, and that's pretty long. - configure.ac lacks structure; splitting it up might improve that. - From what I read,

...roups.ml \ diff --git a/python/Makefile.am b/python/Makefile.am index 673de3157..9aca202b5 100644 --- a/python/Makefile.am +++ b/python/Makefile.am @@ -38,8 +38,9 @@ EXTRA_DIST = \ setup.py.in \ run-bindtests \ run-python-tests \ + t/__init__.py \ t/README \ - t/*.py + t/test[0-9]*.py if HAVE_PYTHON diff --git a/rust/Makefile.am b/rust/Makefile.am index c96607621..979e98bd4 100644 --- a/rust/Makefile.am +++ b/rust/Makefile.am @@ -26,7 +26,6 @@ EXTRA_DIST = \ tests/*.rs \ src/*.rs \ src/bin/*.rs \ - Cargo.toml \ README.md \ run-bindtests \ run-tests diff --git a/tests/daemon/Makef...

...l Early gnulib initialization. > HEADING([Configuring Gnulib]) > gl_EARLY > @@ -79,8 +105,6 @@ dnl These are required to get common/*/Makefile.am files > dnl to work. We should further decouple these in future XXX > AM_CONDITIONAL([ENABLE_APPLIANCE],[true]) > AM_CONDITIONAL([HAVE_PYTHON],[false]) dnl Disables a test > -AC_DEFINE_UNQUOTED([PACKAGE_VERSION_FULL],["$PACKAGE_VERSION"],[Version string.]) > -AC_SUBST([PACKAGE_VERSION_FULL], [$PACKAGE_VERSION]) > Sure ACK. I guess ‘virt-v2v --version’ now prints the extra field because of print_version_and_exit in...

...). I'll reverse the question: if building the standalone tarball works fine with no config.h, why shouldn't the build-in-libguestfs do the same? The bindings do not use gnulib and use only few system headers and APIs, and nothing of the config.h content really matters (the closest thing is HAVE_PYTHON, which is obviously true...). In the end, it is just a binding that wraps the C API for a different language (Python in this case), so it definitely _ought to_ not depend on the way libguestfs itself was configured/built. Moreover: the same situation applies to the other bindings too (those with C...

...optional symbols in libpython. old_LIBS="$LIBS" @@ -1183,6 +1191,7 @@ AS_IF([test "x$enable_python" != "xno"],[ AC_SUBST(PYTHON_PREFIX) AC_SUBST(PYTHON_VERSION) AC_SUBST(PYTHON_INSTALLDIR) + AC_SUBST(PYTHON_EXT_SUFFIX) ]) AM_CONDITIONAL([HAVE_PYTHON], [test "x$PYTHON" != "xno" && test "x$PYTHON_LIBS" != "x" ]) diff --git a/python/Makefile.am b/python/Makefile.am index ecc25d5..d513695 100644 --- a/python/Makefile.am +++ b/python/Makefile.am @@ -56,7 +56,7 @@ libguestfsmod_la_LIBADD = \ $(L...

...A_DIST = \ setup.py.in \ run-bindtests \ run-python-tests \ + test-pycodestyle.sh \ t/__init__.py \ t/README \ t/test[0-9]*.py @@ -117,6 +118,10 @@ if ENABLE_APPLIANCE TESTS += run-python-tests endif ENABLE_APPLIANCE +if HAVE_PYCODESTYLE +TESTS += test-pycodestyle.sh +endif + endif HAVE_PYTHON # Extra clean. diff --git a/python/test-pycodestyle.sh b/python/test-pycodestyle.sh new file mode 100755 index 000000000..63b453170 --- /dev/null +++ b/python/test-pycodestyle.sh @@ -0,0 +1,28 @@ +#!/bin/bash - +# libguestfs +# Copyright (C) 2020 Red Hat Inc. +# +# This program is free software;...

...A_DIST = \ setup.py.in \ run-bindtests \ run-python-tests \ + test-pycodestyle.sh \ t/__init__.py \ t/README \ t/test[0-9]*.py @@ -117,6 +118,10 @@ if ENABLE_APPLIANCE TESTS += run-python-tests endif ENABLE_APPLIANCE +if HAVE_PYCODESTYLE +TESTS += test-pycodestyle.sh +endif + endif HAVE_PYTHON # Extra clean. diff --git a/python/test-pycodestyle.sh b/python/test-pycodestyle.sh new file mode 100755 index 000000000..2a5bf3fac --- /dev/null +++ b/python/test-pycodestyle.sh @@ -0,0 +1,30 @@ +#!/bin/bash - +# libguestfs +# Copyright (C) 2020 Red Hat Inc. +# +# This program is free software;...

...kefile.am +++ b/tests/Makefile.am @@ -115,7 +115,8 @@ EXTRA_DIST = \ test-pattern-largest-for-qemu.sh \ test-python-exception.sh \ test.pl \ - test.py \ + test-python.py \ + test-python-plugin.py \ test-rate.sh \ test-rate-dynamic.sh \ test.rb \ @@ -787,18 +788,10 @@ endif HAVE_PERL if HAVE_PYTHON TESTS += \ + test-python.py \ test-python-exception.sh \ test-shebang-python.sh \ $(NULL) -LIBGUESTFS_TESTS += test-python - -test_python_SOURCES = test-lang-plugins.c test.h -test_python_CFLAGS = \ - -DLANG='"python"' -DSCRIPT='"$(srcdir)/test.py"' \ -...

...mu.sh \ > test-python-exception.sh \ > test.pl \ > - test.py \ > + test-python.py \ > + test-python-plugin.py \ > test-rate.sh \ > test-rate-dynamic.sh \ > test.rb \ > @@ -787,18 +788,10 @@ endif HAVE_PERL > if HAVE_PYTHON > > TESTS += \ > + test-python.py \ > test-python-exception.sh \ > test-shebang-python.sh \ > $(NULL) > -LIBGUESTFS_TESTS += test-python > - > -test_python_SOURCES = test-lang-plugins.c test.h > -test_python_CFLAGS = \ > - -DLANG...

These three patches ought to fix configure script detection of Ruby, especially on Debian where the Ruby C extensions library can be something like '-lruby1.8'. Rich.

*** BLURB HERE? *** Pino Toscano (4): build: stop shipping files generated by configure docs: fix out-of-source documentation build generator: do not generate mlv2v files when not needed perl: fix path to Build.PL docs/Makefile.am | 4 ++-- generator/main.ml | 11 +++++++---- perl/Makefile.am | 2 +- python/Makefile.am | 3 ++- tests/daemon/Makefile.am |