Page MenuHomeDevCentral

Log correctly stdout and stderr
ClosedPublic

Authored by dereckson on Apr 14 2023, 20:56.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Oct 24, 11:12
Unknown Object (File)
Thu, Oct 24, 10:39
Unknown Object (File)
Wed, Oct 23, 05:12
Unknown Object (File)
Wed, Oct 16, 07:25
Unknown Object (File)
Wed, Oct 16, 06:59
Unknown Object (File)
Fri, Oct 11, 13:56
Unknown Object (File)
Sun, Oct 6, 18:24
Unknown Object (File)
Sun, Oct 6, 18:20
Subscribers
None

Details

Summary

Stdio::piped() isn't necessary to capture the output of a process.

Also, a typo in the implementation confused stdout and stderr.

Test Plan
  • alkane serve, /init and /update
  • alkane deploy

Diff Detail

Repository
rALK Alkane
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson created this revision.

Don't bump version I've seen another issue

This revision is now accepted and ready to land.Apr 14 2023, 21:09
This revision was automatically updated to reflect the committed changes.