Closed
Bug 666140
Opened 12 years ago
Closed 12 years ago
client.mk should fail if MOZCONFIG is specified but points to a nonexistent file
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla7
People
(Reporter: int3, Assigned: int3)
Details
(Whiteboard: fixed-in-bs)
Attachments
(1 file)
886 bytes,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20100101 Firefox/5.0 Build Identifier: I spent a good deal of time trying to figure out why my builds weren't working as expected, when I realized that my MOZCONFIG environment variable was actually pointing to a nonexistent file, and the build was just using its default options. I feel that the build system should highlight this error instead of defaulting silently. Reproducible: Always
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #540980 -
Flags: review?(khuey)
Assignee | ||
Updated•12 years ago
|
Hardware: x86 → All
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 540980 [details] [diff] [review] Fix for the issue described. >+ if [ -n "$_config" ] && ! [ -f "$_config" ]; then This should be ||, no?
Assignee | ||
Comment 3•12 years ago
|
||
Nope. If _config is not defined, then we should look for the mozconfig file in the default places like ~/.mozconfig. But if it is defined _and_ points to a file that doesn't exist, then we should throw an error.
My bad, I read another ! in there because that's what I expected.
Attachment #540980 -
Flags: review?(khuey) → review+
http://hg.mozilla.org/projects/build-system/rev/8625f655aeec
Whiteboard: fixed-in-bs
Assignee: nobody → jezreel
Status: NEW → ASSIGNED
http://hg.mozilla.org/mozilla-central/rev/8625f655aeec
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Comment 7•12 years ago
|
||
this broke my pymake builds of Thunderbird. For those builds, I set MOZCONFIG to /c/builds/<tree>/mozconfig, which was working earlier.
Comment 8•12 years ago
|
||
Does it not work to set MOZCONFIG=c:/builds/<tree>/mozconfig?
Comment 9•12 years ago
|
||
(In reply to comment #8) > Does it not work to set MOZCONFIG=c:/builds/<tree>/mozconfig? No, it fails the same way: g --disable-optimize --enable-chrome-format=jar --enable-application=../mail --d isable-official-branding --with-branding=../mail/branding/nightly --cache-file=. ././config.cache --srcdir=../../mozilla ../../mozilla/./build/autoconf/mozconfig2configure: line 118: Specified MOZCONFI G "../.mozconfig" does not exist!: No such file or directory configure: error: ../../mozilla/configure failed for mozilla It's odd that it's looking for ../.mozconfig - maybe it's an issue that Thunderbird builds out of <tree>/mozilla
Comment 10•12 years ago
|
||
Also breaks for me using a relative configure i.e. invoking ../configure with my mozconfig living in my srcdir so not actually specified in $MOZCONFIG.
Comment 11•12 years ago
|
||
Strange, running MOZCONFIG=/c/Users/Sid/mozilla/comm-central/.mozconfig-opt ../configure works fine for me with an updated mozilla-central.
So ... do we need to back this out our not?
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
•