Closed
Bug 464379
Opened 16 years ago
Closed 16 years ago
configure script prints python path unnecessarily
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jimb, Assigned: jimb)
Details
Attachments
(1 file)
953 bytes,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
configure.in (and js/src/configure.in, which is derived from it) have a gratuitous echo PYTHON=$PYTHON line in them. This is unnecessary, as configure prints the path of the python it found as part of AC_PATH_PROGS's normal behavior: ... checking for python... /usr/bin/python PYTHON=/usr/bin/python ...
Assignee | ||
Comment 1•16 years ago
|
||
Here's the commit that added this: 1.1764 <benjamin@smedbergs.us> 2007-01-31 08:12 No bug: checking to see which tinderboxes don't have python available. It seems unlikely that this echo was meant to stay in the configure script.
Attachment #347664 -
Flags: review?(benjamin)
Updated•16 years ago
|
Attachment #347664 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 2•16 years ago
|
||
Landed as tm changeset 730cb94a5b32.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•