Page MenuHomePhabricator

[Wilds] Slightly simplify fatal handling during "arc" setup
ClosedPublic

Authored by epriestley on Sep 18 2018, 8:10 PM.
Tags
None
Referenced Files
F13051965: D19693.id.diff
Fri, Apr 19, 6:03 AM
F13048216: D19693.id47047.diff
Thu, Apr 18, 5:39 PM
Unknown Object (File)
Thu, Apr 18, 10:09 AM
Unknown Object (File)
Tue, Apr 16, 11:06 PM
Unknown Object (File)
Sat, Apr 13, 6:34 AM
Unknown Object (File)
Fri, Apr 12, 10:31 PM
Unknown Object (File)
Fri, Apr 5, 3:33 PM
Unknown Object (File)
Mon, Apr 1, 7:15 AM
Subscribers
None

Details

Summary

Ref T13098. We currently handle fatals by printing a message and returning an error code, since this was the most direct adaptation from the old arcanist.php script.

Instead, throw an exception and then handle it above, since we can reasonably do that in one place now.

Test Plan

Ran arc.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 18 2018, 8:10 PM
Harbormaster failed remote builds in B20859: Diff 47047!
This revision is now accepted and ready to land.Sep 20 2018, 12:01 AM
This revision was automatically updated to reflect the committed changes.