Skip to content

Commit f5ad580

Browse files
committed
gh-151422: Don't link libffi into _ctypes_test.so
Minor build fix
1 parent f4f1020 commit f5ad580

2 files changed

Lines changed: 6 additions & 3 deletions

File tree

configure

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

configure.ac

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8447,9 +8447,12 @@ PY_STDLIB_MOD([_testmultiphase], [test "$TEST_MODULES" = yes], [test "$ac_cv_fun
84478447
PY_STDLIB_MOD([_testsinglephase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes])
84488448
PY_STDLIB_MOD([xxsubtype], [test "$TEST_MODULES" = yes])
84498449
PY_STDLIB_MOD([_xxtestfuzz], [test "$TEST_MODULES" = yes])
8450+
dnl _ctypes_test contains only plain C test helpers and does not use libffi
8451+
dnl itself, so it does not need $LIBFFI_CFLAGS/$LIBFFI_LIBS. We use the
8452+
dnl have_libffi check so the module is only built alongside a working _ctypes.
84508453
PY_STDLIB_MOD([_ctypes_test],
84518454
[test "$TEST_MODULES" = yes], [test "$have_libffi" = yes -a "$ac_cv_func_dlopen" = yes],
8452-
[$LIBFFI_CFLAGS], [$LIBFFI_LIBS $LIBM])
8455+
[], [$LIBM])
84538456

84548457
dnl Limited API template modules.
84558458
dnl Emscripten does not support shared libraries yet.

0 commit comments

Comments
 (0)