Closed
Bug 570440
Opened 14 years ago
Closed 14 years ago
disable-javaxpcom not recognized after configure.in change
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.3a5
People
(Reporter: mozilla-bugs, Assigned: mbrubeck)
References
Details
Attachments
(1 file, 4 obsolete files)
4.89 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
When this commit happened, the explicit empty string for MOZ_JAVAXPCOM was removed: http://hg.mozilla.org/mozilla-central/rev/21f0727c27a6 This seems to enable javaxpcom even though I have --disable-javaxpcom explicitly set. I flipped the logic to check for it disabled and that seems to allow me to build again. I don't know if that's the proper solution, but I am submitting the patch for review.
Attachment #449572 -
Flags: review?(ted.mielczarek)
Reporter | ||
Comment 1•14 years ago
|
||
Sorry, I left some debug output code in there.
Attachment #449572 -
Attachment is obsolete: true
Attachment #449573 -
Flags: review?(ted.mielczarek)
Attachment #449572 -
Flags: review?(ted.mielczarek)
Reporter | ||
Comment 2•14 years ago
|
||
Comment on attachment 449573 [details] [diff] [review] Flip enable javaxpcom check to disable javaxpcom check v2 I'm withdrawing the review request until I verify that it builds.
Attachment #449573 -
Flags: review?(ted.mielczarek)
Comment 4•14 years ago
|
||
From bug 570463: Since the patch from bug 557824 is committed, MOZ_ARG_ENABLE_BOOL(javaxpcom, ...) no longer overwrites MOZ_JAVAXPCOM variable when --disable-javaxpcom is passed. MOZ_AVAXPCOM is set in xulrunner/confvars.sh for xulrunner, so it's impossible to disable it (it's useful for crosscompiling). Due to the way AC_ARG_ENABLE works (not passing argument is equal to false condition), I don't see any clear way to fix it. In my proposed patch I use MOZ_ARG_ENABLE_BOOL or MOZ_ARG_DISABLE_BOOL depending on initial MOZ_JAVAXPCOM value (the second macro has no healp string to not duplicate the first).
Attachment #449594 -
Flags: review?(ted.mielczarek)
Comment 5•14 years ago
|
||
Comment on attachment 449594 [details] [diff] [review] fix v1.0 This shouldn't be necessary... ENABLE_ and DISABLE_ are mirrors of eachother, and can take three arguments, so it should be possible to do: MOZ_ARG_ENABLE_BOOL(javaxpcom, --enable-javaxpcom, MOZ_JAVAXPCOM=1, MOZ_JAVAXPCOM=)
Attachment #449594 -
Flags: review?(ted.mielczarek) → review-
Comment 6•14 years ago
|
||
You're right. I had an other impression after reading AC_ARG_ENABLE documentation (that the else condition is invoked when no explicit arg is passed, so it would overwrite the default value). I've checked that it's not true. A patch, that partially reverts the patch from bug 557824, will do the trick then. Now I don't see the reason for similar changes in http://hg.mozilla.org/mozilla-central/rev/21f0727c27a6 and I expect more similar problems with other configs.
Comment 7•14 years ago
|
||
Attachment #449594 -
Attachment is obsolete: true
Attachment #449625 -
Flags: review?(benjamin)
Updated•14 years ago
|
Attachment #449625 -
Flags: review?(benjamin) → review+
Reporter | ||
Comment 8•14 years ago
|
||
Reassigning to Jacek. I realized later that the simplest solution is usually best and was going that add that patch. Thanks for catching that.
Assignee: mozilla-bugs → jacek
Updated•14 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 11•14 years ago
|
||
There are several similar changes made by bug 557824 (e.g. NSS_DISABLE_DBM). Should those be reverted too?
Assignee | ||
Comment 12•14 years ago
|
||
Restore all of the MOZ_ARG_*_BOOL arguments removed by attachment 449184 [details] [diff] [review].
Assignee: jacek → mbrubeck
Attachment #449915 -
Flags: review?
Assignee | ||
Updated•14 years ago
|
Attachment #449915 -
Flags: review? → review?(benjamin)
Updated•14 years ago
|
Attachment #449915 -
Flags: review?(benjamin) → review+
Comment 13•14 years ago
|
||
(In reply to comment #12) > Created an attachment (id=449915) [details] Does this patch obsolete attachment 449573 [details] [diff] [review] and attachment 449625 [details] [diff] [review]?
Updated•14 years ago
|
Attachment #449573 -
Attachment is obsolete: true
Updated•14 years ago
|
Attachment #449625 -
Attachment is obsolete: true
Comment 14•14 years ago
|
||
Yes, it does.
Comment 16•14 years ago
|
||
Pushed with conflicts fixed: http://hg.mozilla.org/mozilla-central/rev/7043a72809a8
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
OS: Linux → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•