FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.15] gh-151422: Don't link libffi into _ctypes_test.so (GH-151423) by miss-islington · Pull Request #151516 · python/cpython · GitHub

/ cpython Public
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension .ac  (1) .py  (2) No extension  (1) All 3 file types selected
Viewed files
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Unified
Split
Hide whitespace
Diff view
Unified
Split
Hide whitespace
3 changes: 2 additions & 1 deletion Lib/test/test_ctypes/test_as_parameter.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
c_short, c_int, c_long, c_longlong,
c_byte, c_wchar, c_float, c_double,
ArgumentError)
from test.support import import_helper, skip_if_sanitizer
from test.support import import_helper, skip_if_sanitizer, skip_emscripten_stack_overflow
_ctypes_test = import_helper.import_module("_ctypes_test")


Expand Down Expand Up @@ -193,6 +193,7 @@ class S8I(Structure):
(9*2, 8*3, 7*4, 6*5, 5*6, 4*7, 3*8, 2*9))

@skip_if_sanitizer('requires deep stack', thread=True)
@skip_emscripten_stack_overflow()
def test_recursive_as_param(self):
class A:
pass
Expand Down
7 changes: 7 additions & 0 deletions Lib/test/test_ctypes/test_structures.py
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Original file line number Diff line number Diff line change
Expand Up @@ -299,9 +299,16 @@ class X(Structure):
self.assertEqual(s.first, got.first)
self.assertEqual(s.second, got.second)

@unittest.skipIf(support.is_wasm32, "wasm ABI is incompatible with test expectations")
def _test_issue18060(self, Vector):
# Regression tests for gh-62260

# This test passes a struct of two doubles by value to atan2(), whose C
# signature is atan2(double, double), so it only works on platforms
# where the abi of a function that takes a struct with two doubles
# matches the abi of a function that takes two doubles. The wasm32 ABI
# does not satisfy this condition and the test breaks.

# The call to atan2() should succeed if the
# class fields were correctly cloned in the
# subclasses. Otherwise, it will segfault.
Expand Down
4 changes: 2 additions & 2 deletions configure

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 3 additions & 1 deletion configure.ac
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Original file line number Diff line number Diff line change
Expand Up @@ -8479,9 +8479,11 @@ PY_STDLIB_MOD([_testmultiphase], [test "$TEST_MODULES" = yes], [test "$ac_cv_fun
PY_STDLIB_MOD([_testsinglephase], [test "$TEST_MODULES" = yes], [test "$ac_cv_func_dlopen" = yes])
PY_STDLIB_MOD([xxsubtype], [test "$TEST_MODULES" = yes])
PY_STDLIB_MOD([_xxtestfuzz], [test "$TEST_MODULES" = yes])
dnl Check have_libffi so _ctypes_test is only built if _ctypes is built.
dnl _ctypes_test doesn't use libffi directly.
PY_STDLIB_MOD([_ctypes_test],
[test "$TEST_MODULES" = yes], [test "$have_libffi" = yes -a "$ac_cv_func_dlopen" = yes],
[$LIBFFI_CFLAGS], [$LIBFFI_LIBS $LIBM])
[], [$LIBM])

dnl Limited API template modules.
dnl Emscripten does not support shared libraries yet.
Expand Down
Loading

Back | FazBrowse Home | New Git URL