fix: move cargo-wdk output to stderr #527
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
cargo-wdkto write its output to stderr instead of stdoutRationale
As noted in #526, build tools like
cargo build,rustc, andgccemit their human-readable output to stderr, leaving stdout available for machine-readable output (e.g.,cargo build --message-format=json). This PR bringscargo-wdkin line with this convention.Changes
crates/cargo-wdk/src/trace.rsto add.with_writer(std::io::stderr)to thetracing_subscriberconfigurationinfo!,debug!,warn!, anderror!macro output to stderrBenefits
cargo wdk build > output.log), all build output now goes to the same stream ascargo buildoutputTesting
The change uses the standard
tracing_subscriber::fmt::SubscriberBuilder::with_writerAPI, which is well-documented and widely used. CI will verify the build passes on supported platforms.Fixes #526