-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[wasm] Bump emscripten to 3.1.9 #68765
Conversation
@@ -144,6 +144,7 @@ if (CLR_CMAKE_TARGET_UNIX OR CLR_CMAKE_TARGET_BROWSER) | |||
add_compile_options(-Wno-typedef-redefinition) | |||
add_compile_options(-Wno-c11-extensions) | |||
add_compile_options(-Wno-thread-safety-analysis) | |||
add_compile_options(-Wno-strict-prototypes) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this about zlib?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's about zlib, I ran into something like this with trying to bump us to 1.2.12. Ended up adding a function set_zlib_source_files_properties
to src/native/external/zlib.cmake
to turn off strict prototypes just for those source files:
3c3d19b
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like mono_assembly_load_corlib is missing a forward declaration in assembly.c as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in general we need to resolve these rather than just turning off warnings, zlib is obviously slightly different
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's about zlib, I ran into something like this with trying to bump us to 1.2.12. Ended up adding a function
set_zlib_source_files_properties
tosrc/native/external/zlib.cmake
to turn off strict prototypes just for those source files: 3c3d19b
Part of the warnings come indeed from zlib IIRC, there are many more in other parts as well.
I have opened #69142 to not forget to fix them.
@radekdoulik the packages are in #69127 so please prioritize this |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
To make tests build
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
helix failures, array length:
network failure (maybe related to previous array length?):
timeout:
another timeout:
|
I think the issue with ftruncate64 comes from this change: emscripten-core/emscripten@401e698#diff-217fe089a83fef74f6896c463a9f75ace675a7ad90364f0dde18050892840108L727-R718 |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
OSX failure is #69190 |
/azp run runtime-wasm |
Azure Pipelines successfully started running 1 pipeline(s). |
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
No description provided.