Closed Bug 680795 Opened 13 years ago Closed 13 years ago

Remove support for PPC Darwin from configure

Categories

(Firefox Build System :: General, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: matjk7, Assigned: matjk7)

References

Details

Attachments

(1 file, 3 obsolete files)

      No description provided.
Flags: in-testsuite-
Attached patch patch (obsolete) — Splinter Review
Attachment #554752 - Flags: review?(ted.mielczarek)
Attached patch patch (obsolete) — Splinter Review
Forgot js/src/configure in previous patch.
Attachment #554752 - Attachment is obsolete: true
Attachment #554752 - Flags: review?(ted.mielczarek)
Attachment #554753 - Flags: review?(ted.mielczarek)
Attached patch patch (obsolete) — Splinter Review
Attachment #554753 - Attachment is obsolete: true
Attachment #554753 - Flags: review?(ted.mielczarek)
Attachment #554754 - Flags: review?(ted.mielczarek)
Attached patch patchSplinter Review
Forgot hg qref again. Sorry for bugspam.
Attachment #554754 - Attachment is obsolete: true
Attachment #554754 - Flags: review?(ted.mielczarek)
Attachment #554755 - Flags: review?(ted.mielczarek)
Matheus, I believe that TenFourFox (http://code.google.com/p/tenfourfox/) uses this.  Have you talked with TenFourFox's men about removing Darwin PPC config?  We should respect community.

Of course, when our target becomes 10.6 or later only, we can remove this.
Cameron Kaiser (maintainer of Tenfourfox) is CC'ed on this bug. If he's not ok with manually reverting this patch in the 10.4Fx repo we can wontfix this bug since it's just a small cleanup.
I firmly believe we should keep this support as long as possible. Removing it isn't worth it.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Attachment #554755 - Flags: review?(ted.mielczarek)
We do use this code in TenFourFox-land. It's not as big a deal to us personally since we dump our own changesets on top, so I can easily back it out, but there are vanilla PPC Firefox builders on 10.5 and this would definitely put them out of business. I agree with Makoto that when 10.5 is dropped, this code then becomes superfluous, but 10.5 isn't dead yet.
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: