configure should recognize Intel-based Macs

RESOLVED FIXED

Status

Firefox Build System
General
RESOLVED FIXED
13 years ago
2 months ago

People

(Reporter: Josh Aas, Assigned: Josh Aas)

Tracking

Trunk
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

640 bytes, patch
Mike Pinkerton (not reading bugmail)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

13 years ago
This patch was provided by Apple. It make configure recognize Intel-based Macs.
(Assignee)

Comment 1

13 years ago
Created attachment 185753 [details] [diff] [review]
Apple fix v1.0
Attachment #185753 - Flags: superreview?(benjamin)
Attachment #185753 - Flags: review?(joshmoz)
(Assignee)

Updated

13 years ago
Attachment #185753 - Flags: review?(joshmoz) → review+
shouldn't this patch configure.in rather than configure?

Comment 3

13 years ago
Comment on attachment 185753 [details] [diff] [review]
Apple fix v1.0

Yes, this patch must be against configure.in
Attachment #185753 - Flags: superreview?(benjamin) → superreview-
(Assignee)

Comment 4

13 years ago
Created attachment 185799 [details] [diff] [review]
put into configure.in, v1.1

oops
Attachment #185753 - Attachment is obsolete: true
Attachment #185799 - Flags: review?(pinkerton)
Comment on attachment 185799 [details] [diff] [review]
put into configure.in, v1.1

r=pink
Attachment #185799 - Flags: review?(pinkerton) → review+
(Assignee)

Updated

13 years ago
Attachment #185799 - Flags: superreview?(benjamin)

Updated

13 years ago
Attachment #185799 - Flags: superreview?(benjamin) → superreview+
(Assignee)

Updated

13 years ago
Attachment #185799 - Flags: approval1.8b3?

Updated

13 years ago
Attachment #185799 - Flags: approval1.8b3? → approval1.8b3+
(Assignee)

Comment 6

13 years ago
landed on trunk
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED

Comment 7

13 years ago
Perhaps these should be filed under Hardware: PC ;-)

Updated

2 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.