Page MenuHomePhabricator

Use python2 instead of python in `arc anoid` shebang`
ClosedPublic

Authored by eadler on Mar 5 2016, 10:28 PM.
Tags
None
Referenced Files
F15418570: D15408.id.diff
Fri, Mar 21, 12:10 AM
F15411437: D15408.id.diff
Wed, Mar 19, 9:17 AM
F15400068: D15408.id37147.diff
Mon, Mar 17, 9:29 AM
F15398898: D15408.diff
Mon, Mar 17, 1:52 AM
F15393960: D15408.id.diff
Sat, Mar 15, 11:18 PM
F15386164: D15408.id37148.diff
Sat, Mar 15, 12:07 AM
F15385715: D15408.id37148.diff
Fri, Mar 14, 11:13 PM
F15384217: D15408.id.diff
Fri, Mar 14, 7:10 PM
Subscribers

Details

Summary

With the old shebang of #!/usr/bin/env python on machines with python 3 as the default python it would fail.
Prefer an explicit python2 like PEP 394 suggests.

Test Plan

ran ./arc anoid and played a game

Diff Detail

Repository
rARC Arcanist
Branch
python2 (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 11009
Build 13606: Run Core Tests
Build 13605: arc lint + arc unit

Event Timeline

eadler retitled this revision from to Use python2 instead of python in `arc anoid` shebang`.
eadler updated this object.
eadler edited the test plan for this revision. (Show Details)
eadler added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Mar 5 2016, 10:28 PM
This revision was automatically updated to reflect the committed changes.