Problem: configure can error when outputting the mozilla-config.h file. This can happen when `sort` or `egrep` fail. The error is not caught and the configure process continues. Problem exists on tip. Expected solution: command error result code is caught and configure aborts.
Per our discussion on IRC, the result is that the reporter can't build after upgrading to MacOS 10.7.
Created attachment 547481 [details] [diff] [review] configure.in patch to catch command execution errors Trivial patch to configure.in to catch execution errors. Not sure if I caught all errors of this type in the file. Haven't tested that configure still works when the commands do work (as egrep is failing on my machine, but that is a separate problem), so would like verification before review.
I can successfully configure with this patch on my Linux box. Do you have any idea why egrep is failing you?
This patch will cause us to error out if we ever try to write an empty file. Is that ever valid? Is your egrep returning 1 (no lines found) or 2 (error)?
> Do you have any idea why egrep is failing you? egrep is failing with "Regular expression too big." GNU grep 2.5.1. Stock from OS X 10.7. Strangely, when I run commands in my existing bash shell it works, so I suspect my build env is busted. I'll keep investigating, but I think we should treat as a separate issue. > This patch will cause us to error out if we ever try to write an empty file. > Is that ever valid? I'm pretty sure there will always be something to write, as I highly doubt you can get through configure without any #define's written to confdefs.h. And, the second usage of egrep should never be over an empty file, as there are static templates being added to the #define's.
Bug 655339 covers the underlying egrep issue.
Comment on attachment 547481 [details] [diff] [review] configure.in patch to catch command execution errors This looks reasonable, but the same change should be made to js/src's configure too.
Created attachment 548248 [details] [diff] [review] configure.in catch execution errors v2 Replaces existing patch. Adds same code to js/src/configure.in
Created attachment 548251 [details] [diff] [review] configure.in catch execution errors v2 (with proper commit message) Same as previous but with proper commit message (first patch for me!).
> Bug 673209 - configure continues after failure when generating mozilla-config.h. r=khuey Not to be ridiculously picky, but could you please change this to indicate what you did to fix the problem, rather than what the problem was?
Created attachment 548256 [details] [diff] [review] configure.in catch execution errors v2 (with more proper commit message) Clear up commit message, per jlebar's request. Diff content identical.
Pushed to inbound: http://hg.mozilla.org/integration/mozilla-inbound/rev/9cdf76433248