Skip to content

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
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
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
@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants