-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable building test modules that require dlopen()
under WASI SDK 21
#115983
Comments
Might be easiest to set |
Related: |
Maybe add a test for WASI? diff --git a/configure.ac b/configure.ac
index e615259dcb..6a5b5839e8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -7626,9 +7626,9 @@ PY_STDLIB_MOD([_testclinic_limited], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([_testlimitedcapi], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([_testinternalcapi], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([_testbuffer], [test "$TEST_MODULES" = yes])
-PY_STDLIB_MOD([_testimportmultiple], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes])
-PY_STDLIB_MOD([_testmultiphase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes])
-PY_STDLIB_MOD([_testsinglephase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes])
+PY_STDLIB_MOD([_testimportmultiple], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes -a "$ac_sys_system" != "WASI"])
+PY_STDLIB_MOD([_testmultiphase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes -a "$ac_sys_system" != "WASI"])
+PY_STDLIB_MOD([_testsinglephase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes -a "$ac_sys_system" != "WASI"])
PY_STDLIB_MOD([_testexternalinspection], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([xxsubtype], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([_xxtestfuzz], [test "$TEST_MODULES" = yes])
@@ -7638,8 +7638,8 @@ PY_STDLIB_MOD([_ctypes_test],
dnl Limited API template modules.
dnl Emscripten does not support shared libraries yet.
-PY_STDLIB_MOD([xxlimited], [], [test "$ac_cv_func_dlopen" = yes])
-PY_STDLIB_MOD([xxlimited_35], [], [test "$ac_cv_func_dlopen" = yes])
+PY_STDLIB_MOD([xxlimited], [], [test "$ac_cv_func_dlopen" = yes -a "$ac_sys_system" != "WASI"])
+PY_STDLIB_MOD([xxlimited_35], [], [test "$ac_cv_func_dlopen" = yes -a "$ac_sys_system" != "WASI"])
# substitute multiline block, must come after last PY_STDLIB_MOD()
AC_SUBST([MODULE_BLOCK]) |
The modules we should probably skip building under WASI as they must be built as shared modules: cpython/Modules/Setup.stdlib.in Lines 170 to 180 in c951e25
|
…ythonGH-116528) (cherry picked from commit 8c094c3) Co-authored-by: Brett Cannon <brett@python.org>
…GH-116528) (#120316) GH-115983: skip building shared modules for testing under WASI (GH-116528) (cherry picked from commit 8c094c3) Co-authored-by: Brett Cannon <brett@python.org>
_testimportmultiple
,_testmultiphase
,_testsinglephase
,xxlimited
,xxlimited_35
all get built under WASI SDK 21 thanks todlopen()
, but since that function doesn't work normally under WASI we should skip building them.cpython/configure.ac
Lines 7582 to 7593 in 6087315
We could either update
Tools/wasm/wasi.py
to disable test modules, updateconfigure.ac
so they don't get compiled under WASI, or see if we can get them compiled in statically.Linked PRs
The text was updated successfully, but these errors were encountered: