Closed
Bug 899807
Opened 12 years ago
Closed 12 years ago
Update python files to execute via /usr/bin/env python
Categories
(Android Background Services Graveyard :: Build & Test, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcomella, Assigned: mcomella)
Details
Currently it uses /usr/bin/python, which causes the script to fail on systems using python 3 as the default. Using /usr/bin/env allows the path to python to be set by the environment, allowing various environment variable workarounds (see https://wiki.archlinux.org/index.php/Python#Python_2 for some examples).
| Assignee | ||
Comment 1•12 years ago
|
||
(There are other files besides preprocess.py).
Assignee: nobody → michael.l.comella
Summary: Update preprocess.py to execute via /usr/bin/env python → Update python files to execute via /usr/bin/env python
| Assignee | ||
Comment 2•12 years ago
|
||
Flags: needinfo?(nalexander)
Comment 3•12 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #2)
> r? - https://github.com/mozilla-services/android-sync/pull/337
r=nalexander
Flags: needinfo?(nalexander)
Comment 4•12 years ago
|
||
On tickets like this, which don't land on m-c, I like to drop the github commit link (to develop's HEAD) and say something to the effect of "landed on git, doesn't need to land on m-c".
| Assignee | ||
Comment 5•12 years ago
|
||
Landed on git, doesn't need to land on m-c:
https://github.com/mozilla-services/android-sync/commit/d7ae32dfc9ecf62339c4faf140be8c04290c659c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•