Closed
Bug 434055
Opened 16 years ago
Closed 16 years ago
python version check should happen in client.py
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: dougt, Assigned: dougt)
Details
Attachments
(1 file)
498 bytes,
patch
|
ted
:
review-
|
Details | Diff | Splinter Review |
some systems install python 2.3 and do not have 2.5 installed. Since 2.5 is required, we should do a simple test in client.py and notify the developer that a newer version is required.
Assignee | ||
Updated•16 years ago
|
Assignee: nobody → doug.turner
Assignee | ||
Comment 1•16 years ago
|
||
note that we had to put the test for the version in between the imports because I am not sure that OptionParser is available in older versions of python.
Attachment #321274 -
Flags: review?
Assignee | ||
Updated•16 years ago
|
Attachment #321274 -
Flags: review? → review?(ted.mielczarek)
Comment 2•16 years ago
|
||
FWIW: 14.3 optparse -- More powerful command line option parser New in version 2.3.
Comment 3•16 years ago
|
||
Do we really require Python 2.5? I was pretty sure we'd been avoiding that dependency. What exactly requires it?
Assignee | ||
Comment 4•16 years ago
|
||
@ben - cool; thanks. We probably should continue to keep the test very high for the poor developer that somehow screwed up and has a 2.2 install.
In response to comment # 3, see bug 427750, "Require python >= 2.5 to build Mozilla".
Comment 6•16 years ago
|
||
Comment on attachment 321274 [details] [diff] [review] adds a check for python 2.5 AFAIK we don't require Python 2.5 yet, so if there are instances where we do, those are bugs.
Attachment #321274 -
Flags: review?(ted.mielczarek) → review-
Assignee | ||
Comment 7•16 years ago
|
||
.py is not needed any longer.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
Comment 8•16 years ago
|
||
We still require python 2.3 for the build system (preprocessor.py and others).
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
•