-
-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Test w/ WASI SDK >=22 #120371
Comments
With WASI SDK 22:
I'm going to quickly check that it is WASI SDK 22 causing the failures and not |
The failures don't happen under WASI SDK 21, so it's a new issue. I'm going to try the WASI SDK 23 pre-release to see if it's a bug in wasi-libc. |
Under WASI SDK 23 pre-release, 17 tests failed: |
test_dbm FAILED (errors=13, skipped=3)
|
test_dbm_dumb looks similar to test_dbm |
@brettcannon do you have a branch you're running these tests on, and/or instructions for reproducing the failures? I'll take a look when I get a chance (but note that I'll be out on vacation starting Thursday and all of next week, so I might not get to it until after that). |
Ah, I think I can explain the In this commit I added stub implementations of As a temporary measure, perhaps we could force autotools to not use |
@dicej this is all off What I don't get is why does WASI SDK 23 have so many more failures when there was just the test_tarfile failure under 22 (and I'm running the tests again to make sure dad brain didn't cause me to get the different failures wrong)? |
Like I said above, my theory is that autoconf couldn't find a |
Looks like my hunch was right and the amount of failures holds true for WASI SDK 22 as well. |
Oops, I misread your question, sorry. I thought you were asking about 21 and earlier vs. 22 and later. |
I should be able to edit https://github.com/python/cpython/blob/main/Tools/wasm/config.site-wasm32-wasi to explicitly opt out of |
Looks like I have to turn off all the stubs covered by WebAssembly/wasi-libc@cc62fa8 . |
@dicej is there a list of stubs vs. implemented symbols in wasi-libc? That way I can just process some file to update the denylist of functions which don't work? |
Sorry, no; not that I know of. |
I opened WebAssembly/wasi-libc#515 to request such a list. |
Required disabling stub functions now provided by wasi-libc.
Required disabling stub functions now provided by wasi-libc. (cherry picked from commit f589f26) Co-authored-by: Brett Cannon <brett@python.org>
Everything is now passing under WASI SDK 22! |
https://github.com/WebAssembly/wasi-sdk/releases/tag/wasi-sdk-22
https://github.com/WebAssembly/wasi-sdk/releases/tag/wasi-sdk-23 (currently pre-release)
Linked PRs
The text was updated successfully, but these errors were encountered: