1207897 - Add a configure option to build multiple build backends

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Mike Hommey [:glandium]]

With bug 1207893, it will be possible to invoke mach build-backend such as to create multiple backends at once. We should allow people to opt-in to more backends in their mozconfig (think opting-in to the eclipse backend)

Comment 1

[Mike Hommey [:glandium]]

Attachment: Add a configure option to build multiple build backends

Also record what backends were requested at configure time so that running
`mach build-backend` or `config.status` uses the same set.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 8665700 [details] [diff] [review]
Add a configure option to build multiple build backends

Review of attachment 8665700 [details] [diff] [review]:
-----------------------------------------------------------------

I'll give r+ despite the comment because I trust you to test things before landing.

::: python/mozbuild/mozbuild/mach_commands.py
@@ +565,1 @@
>      def build_backend(self, backend='RecursiveMake', diff=False):

If `backend` defaults to "RecursiveMake", won't that lead to "--backend R e c u r s i v e M a k e" due to the .extend(backend) below?

Actually, I'm not sure how this worked before, since [] + str == list(iter(str))

Did I miss something in an earlier patch?

Comment 3

[Mike Hommey [:glandium]]

No, that's an oversight on my part, the default is meant to be removed (and in fact, considering how the command line handler works, that default in the function definition is actually ignored, aiui).

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/617e63c7c9ae

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/617e63c7c9ae