[PATCH] configure: use cffi.FFI().verify() to test buildability of CFFI bindings
Tomi Ollila
tomi.ollila at iki.fi
Mon Jun 8 22:34:46 PDT 2020
Checking existence of pyconfig.h to determine whether CFFI-based
notmuch bindings are buildable is not enough; for example Fedora 32
ships pyconfig.h in python3-libs package, but python3-devel is required
to be installed for the bindings to build.
Executing cffi.FFI().verify() is pretty close to what is done in
bindings/python-cffi/notmuch2/_build.py to get the c code part of the
bindings built.
---
I just could not get this right last time. Probably I concentrated on
testing in debian 10 container so much, that the small differences in
other systems went unnoticed (had that fedora 32 host polluted by
installed python3-devel when doing final tests on that system)...
configure | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/configure b/configure
index f4b3c61a..d85fe5d2 100755
--- a/configure
+++ b/configure
@@ -711,14 +711,12 @@ if [ $have_python -eq 0 ]; then
errors=$((errors + 1))
fi
-have_python3_dev=0
+have_python3=0
if [ $have_python -eq 1 ]; then
- printf "Checking for python3 dev (>= 3.5)..."
- if "$python" -c 'import os, sys, sysconfig;
-assert sys.version_info >= (3,5)
-assert os.path.isfile(sysconfig.get_config_h_filename())' >/dev/null 2>&1; then
+ printf "Checking for python3 (>= 3.5)..."
+ if "$python" -c 'import sys, sysconfig; assert sys.version_info >= (3,5)'; >/dev/null 2>&1; then
printf "Yes.\n"
- have_python3_dev=1
+ have_python3=1
else
printf "No (will not install CFFI-based python bindings).\n"
fi
@@ -726,9 +724,9 @@ fi
have_python3_cffi=0
have_python3_pytest=0
-if [ $have_python3_dev -eq 1 ]; then
+if [ $have_python3 -eq 1 ]; then
printf "Checking for python3 cffi and setuptools... "
- if "$python" -c 'import cffi; import setuptools' >/dev/null 2>&1; then
+ if "$python" -c 'import cffi,setuptools; cffi.FFI().verify()' >/dev/null 2>&1; then
printf "Yes.\n"
have_python3_cffi=1
else
--
2.26.2
More information about the notmuch
mailing list