Closed Bug 536440 Opened 14 years ago Closed 14 years ago

HOST_AR should not be overridden in configure

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.9.3a1

People

(Reporter: mwu, Assigned: mwu)

References

Details

Attachments

(1 file)

Remove unnecessary HOST_AR setting
592 bytes, patch
ted
: review+
Details | Diff | Splinter Review 
HOST_AR is set to '$(AR)' as a default in configure. However, there are already checks before that properly set HOST_AR, so this breaks cross compiling.
Attachment #418885 - Attachment is patch: true
Attachment #418885 - Attachment mime type: application/octet-stream → text/plain
Attachment #418885 - Attachment description: Remove incorrect HOST_AR setting → Remove unnecessary HOST_AR setting
Attachment #418885 - Flags: review?(ted.mielczarek)
Assignee: nobody → mwu
Attachment #418885 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 418885 [details] [diff] [review]
Remove unnecessary HOST_AR setting

Do you want to do something with the HOST_AR_FLAGS setting there as well?
Blocks: 538269
(In reply to comment #1)
> (From update of attachment 418885 [details] [diff] [review])
> Do you want to do something with the HOST_AR_FLAGS setting there as well?

Might as well. Opened bug 538269.
Keywords: checkin-needed
You could have just dealt with it here, since it's tiny, but no big deal.
http://hg.mozilla.org/mozilla-central/rev/a2945f365208
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.3a1
Blocks: 542187
Depends on: 543372
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.