Skip to content
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] build paltests #112404

Merged
merged 73 commits into from
Mar 4, 2025
Merged

[wasm] build paltests #112404

merged 73 commits into from
Mar 4, 2025

Conversation

radekdoulik
Copy link
Member

No description provided.

radekdoulik and others added 30 commits January 21, 2025 13:51
Disable few parts, some might need closer look later.
And also skip creating pipe to communicate with it
…t it was so you can pull the test statuses out of a log file
To avoid error messages at the end of each run
Also fix disabled tests count
Add few more notes to failing memory related tests
Emscripten's mmap doesn't support that and because we don't have virtual memory
in wasm, it doesn't make sense to call it there
Also re-enable debug test, which works and disable few MT tests
@@ -119,6 +119,10 @@ if (CLR_CMAKE_TARGET_UNIX OR CLR_CMAKE_TARGET_BROWSER OR CLR_CMAKE_TARGET_WASI)
add_subdirectory(System.IO.Ports.Native)
endif ()

if (MONO_WASM_MT)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the MONO_WASM_MT?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is a new flag to indicate mono wasm multithreaded build. one of the changes broke the mono build, so this way we can fix that

@radekdoulik radekdoulik requested a review from janvorli February 27, 2025 16:02
radekdoulik and others added 6 commits March 3, 2025 10:40
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@radekdoulik radekdoulik merged commit 11408ab into dotnet:main Mar 4, 2025
173 of 177 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants