Skip to content

Print build on save loop stderr#2356

Closed
SuperAuguste wants to merge 1 commit into
masterfrom
auguste/build-on-save-loop-stderr
Closed

Print build on save loop stderr#2356
SuperAuguste wants to merge 1 commit into
masterfrom
auguste/build-on-save-loop-stderr

Conversation

@SuperAuguste

Copy link
Copy Markdown
Member

Helpful for diagnosing build on save issues.

"";
defer allocator.free(stderr);

log.debug("build on save stderr:\n{s}", .{stderr});

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
log.debug("build on save stderr:\n{s}", .{stderr});
if (stderr.len != 0) {
log.debug("build on save stderr:\n{s}", .{stderr});
}

@Techatrix

Copy link
Copy Markdown
Member

I coincidentally applied similar changes in #2400. The difference is that stderr is only printed if error.EndOfStream is returned which is probably for the better.

@Techatrix Techatrix closed this Jul 13, 2025
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