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] Reduce the output from tests. #61593

Merged
merged 1 commit into from
Nov 18, 2021
Merged

Conversation

maraf
Copy link
Member

@maraf maraf commented Nov 15, 2021

  • Do not printout exceptions from failing task tests.
  • Remove default MONO_LOG_MASK=gc from debug configuration.

- Do not printout exceptions from failing task tests.
- Remove default MONO_LOG_MASK=gc from debug configuration.
@maraf maraf added the arch-wasm WebAssembly architecture label Nov 15, 2021
@maraf maraf added this to the 7.0.0 milestone Nov 15, 2021
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@ghost
Copy link

ghost commented Nov 15, 2021

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details
  • Do not printout exceptions from failing task tests.
  • Remove default MONO_LOG_MASK=gc from debug configuration.
Author: maraf
Assignees: -
Labels:

arch-wasm

Milestone: 7.0.0

// Setting this env var allows Diagnostic.Debug to write to stderr. In a browser environment this
// output will be sent to the console. Right now this is the only way to emit debug logging from
// corlib assemblies.
monoeg_g_setenv ("COMPlus_DebugWriteToStdErr", "1", 0);
// monoeg_g_setenv ("COMPlus_DebugWriteToStdErr", "1", 0);
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if we should revisit the way this works and send Debug.* output to the proper js console

Copy link
Member

Choose a reason for hiding this comment

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

Just for reference this is what it does

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants