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

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

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.

Comment 1

[Mike Hommey [:glandium]]

Attachment: MozReview Request: Bug 1254410 - Include app-specific configure files according to --enable-application/--enable-project

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/

Comment 2

[Mike Hommey [:glandium]]

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.

Comment 3

[Chris Manchester (limited bugmail, email directly)]

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

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/4d6b7026be7e

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/4d6b7026be7e