Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switch libunwind detection to using weak symbols #9479

Merged

Conversation

alexcrichton
Copy link
Member

@alexcrichton alexcrichton commented Oct 16, 2024

This is an attempt to resolve #8897.

Closes #8897.

@alexcrichton
Copy link
Member Author

Posting this to see what CI says, I won't be able to test this for a week or so.

@github-actions github-actions bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Oct 17, 2024
@alexcrichton alexcrichton marked this pull request as ready for review October 22, 2024 15:10
@alexcrichton alexcrichton requested a review from a team as a code owner October 22, 2024 15:10
@alexcrichton alexcrichton requested review from pchickey and removed request for a team October 22, 2024 15:10
@alexcrichton
Copy link
Member Author

Ok I've done a bit of local testing and I think this should be good to land

@alexcrichton alexcrichton added this pull request to the merge queue Oct 22, 2024
Merged via the queue into bytecodealliance:main with commit 460beae Oct 22, 2024
39 checks passed
@alexcrichton alexcrichton deleted the libunwind-weak-symbols branch October 22, 2024 16:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
wasmtime:api Related to the API of the `wasmtime` crate itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

libunwind warning when compiling a wasmtime project against musl
2 participants