Closed Bug 599737 Opened 9 years ago Closed 9 years ago

Fully end-of-life Python 2.4 for build support

Categories

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

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
mozilla2.0b8

People

(Reporter: sgautherie, Assigned: sgautherie)

References

Details

Attachments

(1 file)

No description provided.
Flags: in-testsuite-
While there, just tell me whether you want to add support for 'python2.7' (directory) or not.
Attachment #478642 - Flags: review?(ted.mielczarek)
Blocks: 599743
Comment on attachment 478642 [details] [diff] [review]
(Av1) Remove remnants in configure.in
[Checked in: See comment 10]


>-dnl We require version 2.4 or newer of Python to build,
>-dnl and 2.5 or newer on Windows.

Can you just fix this comment to be correct with respect to what we currently require?

Feel free to add python2.7 to the list of binary names we look for.

r=me with that comment change.
Attachment #478642 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 478642 [details] [diff] [review]
(Av1) Remove remnants in configure.in
[Checked in: See comment 10]

"approval2.0=?":
Trivial configure cleanup, zero risk.
Attachment #478642 - Flags: approval2.0?
Blocks: 601676
(In reply to comment #4)

I never implied that I would be targeting m-1.9.2.
m-2.0 isn't up-to-date: "Thu Jun 03 11:26:32 2010 -0700".
Comment on attachment 478642 [details] [diff] [review]
(Av1) Remove remnants in configure.in
[Checked in: See comment 10]

http://hg.mozilla.org/mozilla-central/rev/6946b5d25ac3
Av1, with comment 2 suggestion(s).
Attachment #478642 - Attachment description: (Av1) Remove remnants in configure.in → (Av1) Remove remnants in configure.in [Checked in: See comment 7]
Attachment #478642 - Flags: approval2.0?
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
I backed out all the changesets in your push because they weren't approved.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment on attachment 478642 [details] [diff] [review]
(Av1) Remove remnants in configure.in
[Checked in: See comment 10]

(In reply to comment #8)
> I backed out all the changesets in your push because they weren't approved.

See discussion in bug 595756 :-<
Attachment #478642 - Flags: approval2.0?
Attachment #478642 - Attachment description: (Av1) Remove remnants in configure.in [Checked in: See comment 7] → (Av1) Remove remnants in configure.in [Checked in: See comment 7]
Attachment #478642 - Flags: approval2.0? → approval2.0+
Comment on attachment 478642 [details] [diff] [review]
(Av1) Remove remnants in configure.in
[Checked in: See comment 10]

http://hg.mozilla.org/mozilla-central/rev/08f79fca7386
Attachment #478642 - Attachment description: (Av1) Remove remnants in configure.in [Checked in: See comment 7] → (Av1) Remove remnants in configure.in [Checked in: See comment 10]
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
(In reply to comment #8)
> I backed out all the changesets in your push because they weren't approved.

http://hg.mozilla.org/mozilla-central/rev/6946b5d25ac3
(In reply to comment #11)
> (In reply to comment #8)
> > I backed out all the changesets in your push because they weren't approved.
> 
> http://hg.mozilla.org/mozilla-central/rev/6946b5d25ac3

http://hg.mozilla.org/mozilla-central/rev/ea371c9d775b !
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.