Skip to content

Stop capturing platform tool error output after startup - #691

Merged
soujay merged 1 commit into
masterfrom
stop-capturing-stderr-after-startup
Oct 26, 2026
Merged

soujay merged 1 commit into
masterfrom
stop-capturing-stderr-after-startup

Conversation

@soujay

@soujay soujay commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor
  • Since PR Report platform tool startup failure with an error message  #684, the launcher keeps each platform tool's stderr so it can report why a tool failed to start. The ErrorDataReceived handler stays attached for the whole session, so the captured output keeps growing while the sample runs, even though it is only ever read during startup.

  • PR Bound captured ServiceControl stderr #689 tries to address this by keeping the last 8 KB. For a missing runtime, the useful lines (“You must install or update .NET”, Framework: … version) are at the top. They could be dropped on a machine with many runtimes or with a long stack trace.

  • This PR is an alternative to PR Bound captured ServiceControl stderr #689, where we have a flag to captureErrorOutput and it stops recording stderr once startup succeeds

@soujay
soujay marked this pull request as ready for review October 26, 2026 21:46
@soujay
soujay merged commit 46bab8c into master Oct 26, 2026
4 checks passed
@soujay
soujay deleted the stop-capturing-stderr-after-startup branch October 26, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants