All things using FINAL_LIBRARY are expected to be static libraries. So we can just imply FORCE_STATIC_LIB when FINAL_LIBRARY is set.
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.
I had forgotten about tests.
Attachment #832803 - Flags: review?(gps)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
You need to log in before you can comment on or make changes to this bug.