Closed Bug 464379 Opened 11 years ago Closed 11 years ago

configure script prints python path unnecessarily

Categories

(Firefox Build System :: General, defect, trivial)

All
Linux
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jimb, Assigned: jimb)

Details

Attachments

(1 file)

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
...
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)
Attachment #347664 - Flags: review?(benjamin) → review+
Landed as tm changeset 730cb94a5b32.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.