Closed
Bug 1053439
Opened 9 years ago
Closed 9 years ago
MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS are no longer required
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla34
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
Details
Attachments
(1 file)
|
1.72 KB,
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
Once the fix to bug 1050086 lands on comm-central there is no need for MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS in mozilla-central
Attachment #8472590 -
Flags: review?(mh+mozilla)
|
||
Comment 1•9 years ago
|
||
Comment on attachment 8472590 [details] [diff] [review] Remove MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS [Checked in m-i: Comment 2] Review of attachment 8472590 [details] [diff] [review]: ----------------------------------------------------------------- ::: configure.in @@ -8320,5 @@ > fi > > if test "$MOZ_APP_COMPONENT_INCLUDE"; then > AC_DEFINE_UNQUOTED(MOZ_APP_COMPONENT_INCLUDE, "$MOZ_APP_COMPONENT_INCLUDE") > fi Can you remove this too in a followup?
Attachment #8472590 -
Flags: review?(mh+mozilla) → review+
Comment on attachment 8472590 [details] [diff] [review] Remove MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS [Checked in m-i: Comment 2] https://hg.mozilla.org/integration/mozilla-inbound/rev/53e5b89085a4
Attachment #8472590 -
Attachment description: Remove MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS → Remove MOZ_APP_COMPONENT_MODULES and MOZ_APP_EXTRA_LIBS [Checked in m-i: Comment 2]
Attachment #8472590 -
Flags: checkin+
|
||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/53e5b89085a4
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
QA Whiteboard: [qa-]
|
||
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
•