Page MenuHomePhabricator
Differential D21037

When "ArcanistRuntime" exits with a nonzero exit code, emit that exit code
ClosedPublic
Actions

Authored by epriestley on Feb 27 2020, 2:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 9:12 PM
Unknown Object (File)
Wed, Dec 18, 1:59 PM
Unknown Object (File)
Tue, Dec 17, 10:02 AM
Unknown Object (File)
Sun, Dec 8, 7:32 PM
Unknown Object (File)
Tue, Dec 3, 11:03 AM
Unknown Object (File)
Tue, Nov 26, 6:48 PM
Unknown Object (File)
Nov 21 2024, 10:55 PM
Unknown Object (File)
Nov 21 2024, 7:31 PM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rARC5451d2875221: When "ArcanistRuntime" exits with a nonzero exit code, emit that exit code
Summary

See https://discourse.phabricator-community.org/t/failed-arc-patch-does-not-return-non-zero-exit-code/3584.

Returning an integer from a top-level PHP file doesn't actually affect the process exit code, as much as I might wish it does.

Test Plan

Ran arc patch adflsnadfsln; echo $?, saw a nonzero exit code after this fix.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 27 2020, 2:16 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Feb 27 2020, 2:17 PM
Harbormaster failed remote builds in B23928: Diff 50107!
This revision was not accepted when it landed; it landed in state Changes Planned.Feb 27 2020, 2:17 PM
This revision was landed with ongoing or failed builds.
Closed by commit rARC5451d2875221: When "ArcanistRuntime" exits with a nonzero exit code, emit that exit code (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rARC5451d2875221: When "ArcanistRuntime" exits with a nonzero exit code, emit that exit code.

Revision Contents
Changeset List

PathSize
bin/
2 lines
2 lines
support/
init/
4 lines

Diff 50108

View Options

bin/arc

Loading...
View Options

bin/phage

Loading...
View Options

support/init/init-arcanist.php

Loading...
Phabricator · Built by Phacility