Closed
Bug 617596
Opened 15 years ago
Closed 15 years ago
python3 compatibility for imacro_asm.py
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: philip, Unassigned)
Details
Attachments
(1 file)
|
450 bytes,
patch
|
Details | Diff | Splinter Review |
Python 3.0 wants brackets in print(). Arch Linux apparently points /usr/bin/python to python3, so the SpiderMonkey build breaks. This fix seems to be enough to make the whole build work.
Attachment #496149 -
Flags: review?(jorendorff)
|
||
Comment 1•15 years ago
|
||
Let's hold off on this and see if bug 601676 fixes things for your Arch Linux users.
|
|
||
Comment 2•15 years ago
|
||
Comment on attachment 496149 [details] [diff] [review]
patch
The fix for bug 601676 has landed. I think it solves this problem the right way.
There are two reasons not to make this change. We don't want to allow all our build scripts to run as Python 3 without at least running 2to3 and pondering the output. And, we would just break Python 3 users again later by accident.
Thanks for taking the time to patch this and file the bug.
Clearing the review flag and resolving WONTFIX.
Attachment #496149 -
Flags: review?(jorendorff)
|
|
||
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•