Closed Bug 1254410 Opened 5 years ago Closed 5 years ago

Include app-specific configure files according to --enable-application/--enable-project

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: glandium, Unassigned)

References

Details

Attachments

(1 file)

Because --enable-application is the current way to do things, transpose it to configure.py, but since --enable-application=js doesn't make sense, make it an alias of a new --enable-project option.
Because --enable-application is the current way to do things, transpose
it to configure.py, but since --enable-application=js doesn't make
sense, make it an alias of a new --enable-project option.

This only partially moves --enable-application out of old-configure.in
because there are a lot of other things intertwined with it.

Review commit: https://reviewboard.mozilla.org/r/38621/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38621/
Attachment #8727722 - Flags: review?(cmanchester)
Please note this *will* actively break comm-central, but that the include paths are left relative so that comm-central can get get out of this on its own, by (I think) copying configure.py/configure.in and adding a top-level moz.configure that includes mozilla/moz.configure.
Attachment #8727722 - Flags: review?(cmanchester) → review+
Comment on attachment 8727722 [details]
MozReview Request: Bug 1254410 - Include app-specific configure files according to --enable-application/--enable-project

https://reviewboard.mozilla.org/r/38621/#review35477
https://hg.mozilla.org/mozilla-central/rev/4d6b7026be7e
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Depends on: 1254987
Blocks: 1255197
Depends on: 1255312
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.