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"


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: zpao, Assigned: zpao)


(Blocks 1 open bug)



(1 file)

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.
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.
Attached patch Patch v0.1Splinter Review
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 ( and it looks fine, so let's make this official.
Attachment #623867 - Flags: review?(ted.mielczarek)
Attachment #623867 - Flags: review?(ted.mielczarek) → review+
Closed: 11 years ago
Resolution: --- → FIXED
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.