Skip to content

Conversation

@github-actions
Copy link
Contributor

Automated changes by create-pull-request GitHub action

@messense messense merged commit 4a3cf15 into main Dec 28, 2025
6 checks passed
@messense messense deleted the create-pull-request/patch branch December 28, 2025 00:28
@konstin
Copy link
Member

konstin commented Dec 28, 2025

I think this change broke CI: __cxa_thread_atexit_impl is now failing in https://github.com/PyO3/maturin/actions/runs/20560352488/job/59050131607?pr=2902

@messense
Copy link
Member

I think we should change that test to higher manylinux version, the symbol is only backlisted on manylinux <= 2.17.

messense added a commit that referenced this pull request Dec 29, 2025
messense pushed a commit that referenced this pull request Jan 6, 2026
PR #2901 updated the manylinux policies, which now include
`__cxa_thread_atexit_impl`. This symbol is [linked
weakly](https://github.com/rust-lang/rust/blob/4fa80a5e733e2202d7ca4c203c2fdfda41cfe7dc/library/std/src/sys/thread_local/destructors/linux_like.rs#L4)
by the Rust stdlib.

Rather than requiring a bump of the minimum manylinux platform tag for
users, ignore weak symbols in the check. This is in line with changes to
PyPA's auditwheel (pypa/auditwheel#665).
joseph-isaacs added a commit to vortex-data/vortex that referenced this pull request Jan 6, 2026
the PR that was incorrect: PyO3/maturin#2901

However I think we should pin maturin going forward?

---------

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants