Closed
Bug 939039
Opened 11 years ago
Closed 11 years ago
Make FINAL_LIBRARY imply FORCE_STATIC_LIB
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla28
People
(Reporter: glandium, Assigned: glandium)
References
Details
(Whiteboard: [qa-])
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
All things using FINAL_LIBRARY are expected to be static libraries. So we can just imply FORCE_STATIC_LIB when FINAL_LIBRARY is set.
Assignee | ||
Comment 1•11 years ago
|
||
This also moves the LIBXUL_LIBRARY implies FORCE_STATIC_LIB test in the emitter
instead of config.mk, since both variables are defined in moz.build exclusively.
backend.mk can end up with both LIBXUL_LIBRARY and FORCE_STATIC_LIB defined, but
it's just fine.
Note this patch alone makes the build fail. It requires the patch I'm going to attach to be folded into it, but i thought it was better, for review, to separate the logic change to the mostly automated FORCE_STATIC_LIB removals that go with it.
Attachment #832786 -
Flags: review?(gps)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #832787 -
Flags: review?(gps)
Assignee | ||
Comment 3•11 years ago
|
||
I had forgotten about tests.
Attachment #832803 -
Flags: review?(gps)
Assignee | ||
Updated•11 years ago
|
Attachment #832786 -
Attachment is obsolete: true
Attachment #832786 -
Flags: review?(gps)
Updated•11 years ago
|
Attachment #832803 -
Flags: review?(gps) → review+
Updated•11 years ago
|
Attachment #832787 -
Flags: review?(gps) → review+
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a02c4f6de455
https://hg.mozilla.org/mozilla-central/rev/7fac57f507f2
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Updated•11 years ago
|
Whiteboard: [qa-]
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•