-
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
Reenable System.Runtime.Serialization.Xml.Tests
#68921
Conversation
This was disabled due to dotnet#59926, which was then tracked by dotnet#61061 . The stack overflow is due to one specific test though, and the rest of the tests can be enabled if we skip this one - `DCS_DeeplyLinkedData`.
The condition on the library in tests.proj covers non-wasm cases too, but the issue that it references is closed. I will run non-wasm jobs also to confirm that this doesn't break others. |
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsThis was disabled due to #59926, The stack overflow is due to one specific test though, and the rest of
|
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 it is hitting the linker problem
SkipOnPlatform would be better imho, this is unlikely to be fixed. |
.. unlikely to be fixed (- vargaz).
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-extra-platforms |
Azure Pipelines successfully started running 1 pipeline(s). |
This was disabled due to #59926,
which was then tracked by #61061
.
The stack overflow is due to one specific test though, and the rest of
the tests can be enabled if we skip this one -
DCS_DeeplyLinkedData
.Fixes #61061 .