Closed
Bug 750574
Opened 11 years ago
Closed 11 years ago
[10.8] Build fails with "arch: posix_spawnp: egrep: Bad CPU type in executable"
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla15
People
(Reporter: zpao, Assigned: zpao)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
1.04 KB,
patch
|
ted
:
review+
ted
:
feedback+
|
Details | Diff | Splinter Review |
At least as of 10.8 DP3, it would appear that maybe Apple is no longer shipping universal binaries for (at least) egrep, so the fix we made to build on Lion (bug 655339) is no longer a valid option. I just successfully built with FIXED_EGREP just pointing to egrep so perhaps we should go back to that.
Assignee | ||
Updated•11 years ago
|
Blocks: mountain-lion-compat
Why is this confidential?
Comment 2•11 years ago
|
||
I'm going to assume "accidentally". Steven: is that grep workaround still necessary on current Lion? Could we simply remove it entirely?
Group: mozilla-corporation-confidential
Comment 3•11 years ago
|
||
> Steven: is that grep workaround still necessary on current Lion? Yes :-( I just did a test build on OS X 10.7.3, with FIXED_EGREP pointing to egrep. I suspect Apple will never fix bug 655339 on Lion (that they somehow consider that too big a change for a minor update).
Assignee | ||
Comment 4•11 years ago
|
||
I marked it confidential because I wasn't really sure where the line is for non-public information about 10.8. I figured it was better to start overly cautious.
Comment 5•11 years ago
|
||
(In reply to comment #4) My understanding is that we're free to discuss Apple's pre-release builds to the extent that they directly effects us (e.g. bugs that we'd need to work around; new features that we'd need to support). I got this from Josh, and have followed it since I started working on Mozilla.
Assignee | ||
Comment 6•11 years ago
|
||
This is working on MoLo but not sure if it'll break anything else. I think this will only hit Lion and then for older versions we'll just use egrep as it was. That should be ok if only Lion AKA darwin 11 was broken (sounds like the case).
Attachment #623867 -
Flags: feedback?(ted.mielczarek)
Comment 7•11 years ago
|
||
Comment on attachment 623867 [details] [diff] [review] Patch v0.1 Review of attachment 623867 [details] [diff] [review]: ----------------------------------------------------------------- That should be fine.
Attachment #623867 -
Flags: feedback?(ted.mielczarek) → feedback+
Updated•11 years ago
|
Assignee: nobody → paul
Assignee | ||
Comment 8•11 years ago
|
||
Comment on attachment 623867 [details] [diff] [review] Patch v0.1 I pushed to try (https://tbpl.mozilla.org/?tree=Try&rev=2d3af9fda07e) and it looks fine, so let's make this official.
Attachment #623867 -
Flags: review?(ted.mielczarek)
Updated•11 years ago
|
Attachment #623867 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 9•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a0c0d4a72648
Target Milestone: --- → mozilla15
Comment 10•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a0c0d4a72648
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•