Page MenuHomePhabricator

dispatch: don't change error status if flushing stdio fails
ClosedPublic

Authored by martinvonz on Oct 11 2021, 7:30 PM.

Details

Summary

If we already have a non-zero exit code, I don't think we should
change it to 255 because we fail to flush stdio. This may not matter
yet, but it will matter when I make a killed pager result in exit code
250 (it's currently 255).

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.Oct 11 2021, 7:30 PM
pulkit accepted this revision.Oct 12 2021, 9:20 AM
This revision is now accepted and ready to land.Oct 12 2021, 9:20 AM