Page MenuHomePhabricator

Reduce the strength of "arc executing on arc" from an error to a warning
ClosedPublic

Authored by epriestley on Jul 21 2017, 6:47 PM.
Tags
None
Referenced Files
F15492179: D18264.id.diff
Sat, Apr 12, 12:05 PM
F15489307: D18264.id43923.diff
Fri, Apr 11, 7:48 AM
F15484873: D18264.id.diff
Wed, Apr 9, 8:43 PM
F15484286: D18264.id43924.diff
Wed, Apr 9, 5:25 PM
F15479554: D18264.id.diff
Tue, Apr 8, 8:07 AM
F15476132: D18264.diff
Mon, Apr 7, 4:21 AM
F15472221: D18264.diff
Sat, Apr 5, 11:34 AM
F15471919: D18264.id43924.diff
Sat, Apr 5, 9:37 AM
Subscribers
None

Details

Summary

See PHI13. This was introduced a very, very long time ago in D311 and D312, and I think T168 was the original report.

It prevents arc from being used in some semi-reasonable (maybe?) automation workflows where you're hooking some version of "Land Revision" up to arc land. This isn't necessarily the right approach, but I think the concession here to make this work is small.

Running arc against another copy of arc makes arc unit not work, but we provide a good error message. Most other arc operations still work correctly.

All of these situations are bizarre edge cases but I think we can safely warn and continue here. Even if we revert this behavior later, almost no one should be affected, since this essentially only impacts users developing arc itself.

Test Plan

Ran one copy of arc against another, saw a warning instead of an error. arc unit failed, but with a good error.

Diff Detail

Repository
rARC Arcanist
Branch
self1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17774
Build 23867: Run Core Tests
Build 23866: arc lint + arc unit