build: re-apply: don't require linking to work for manylinux builds
What changed, and why it matters
This commit changes the build configuration so that when building Python packages for 'manylinux' (a standard for portable Linux Python wheels), a failed link check against libpython is ignored. This is a build-system workaround, not a code change, and it does not alter cryptographic behavior, memory handling, or runtime security.
No security action required. Reviewers may want to confirm that the manylinux build still produces correct Python extension modules despite the skipped link test, but this is a build portability concern rather than a vulnerability.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch modifies tools/build-aux/m4/ax_python_devel.m4. When pythonexists is ‘no’ after the Python link test, it now checks if python_manylinux is ‘yes’. If so, it overrides pythonexists to ‘yes’ and skips the warning/error path. This accommodates manylinux environments where libpython.so is intentionally absent to produce portable wheels. The change is purely in the Autoconf macro and does not affect libwally-core’s C library or Python bindings at runtime.
Changed components
tools/build-aux/m4/ax_python_devel.m4Inspect captured patch +6 / −0
diff --git a/tools/build-aux/m4/ax_python_devel.m4 b/tools/build-aux/m4/ax_python_devel.m4
index 935056c..cbed7f3 100644
--- a/tools/build-aux/m4/ax_python_devel.m4
+++ b/tools/build-aux/m4/ax_python_devel.m4
@@ -443,6 +443,11 @@ print(sitedir)"`
AC_MSG_RESULT([$pythonexists])
if test ! "x$pythonexists" = "xyes"; then
+ if test "x$python_manylinux" = "xyes"; then
+ # Ignore linking errors for manylinux builds (no libpython.so)
+ pythonexists="yes"
+ else
+
AC_MSG_WARN([
Could not link test program to Python. Maybe the main Python library has been
installed in some non-standard library path. If so, pass it to configure,
@@ -460,6 +465,7 @@ print(sitedir)"`
ax_python_devel_found=no
PYTHON_VERSION=""
fi
+ fi
fi
#
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.