[10.8] Build fails with "arch: posix_spawnp: egrep: Bad CPU type in executable"

RESOLVED FIXED in mozilla15

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: zpao, Assigned: zpao)

Tracking

(Blocks: 1 bug)

unspecified
mozilla15
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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.
Blocks: 728102
Why is this confidential?
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
> 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).
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.
(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.
Created attachment 623867 [details] [diff] [review]
Patch v0.1

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 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+
Assignee: nobody → paul
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)
Attachment #623867 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/a0c0d4a72648
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/a0c0d4a72648
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.