1378400 - Expose a build time flag to build firefox without DevTools

Status: RESOLVED WONTFIX

(DevTools :: General, enhancement, P3)

Description

[Alexandre Poirot [:ochameau]]

For now, you can set MOZ_DEVTOOLS=addon in browser/confvars.sh to prevent shipping DevTools.
But we would need a build time flag to easily test Firefox with/without DevTools.

Comment 1

[Patrick Brosset <:pbro>]

DevTools bug triage (filter on CLIMBING SHOES).

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1378400 - Expose mozconfig flag to control devtools packaging.

Review commit: https://reviewboard.mozilla.org/r/156144/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/156144/

Comment 3

[Alexandre Poirot [:ochameau]]

Bug 1363533 introduced MOZ_DEVTOOLS=addon, this new patch allows setting:
  ac_add_options --enable-devtools=addon
in order to enable DevTools shim and no longer ship devtools in Firefox.

You can also use:
* "server" to only ship the backend with actors and debugger server
* "all" to ship everything (what we currently do), but that may be useful for other application like thunderbird, qbrt, ...

Comment 4

[Alexandre Poirot [:ochameau]]

Comment on attachment 8885274 [details]
Bug 1378400 - Expose mozconfig flag to control devtools packaging.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/156144/diff/1-2/

Comment 5

[Gregory Szorc [:gps]]

Comment on attachment 8885274 [details]
Bug 1378400 - Expose mozconfig flag to control devtools packaging.

https://reviewboard.mozilla.org/r/156144/#review161810

Please don't put new things in old-configure.in. Instead, this should go in moz.configure, probably in toolkit/moz.configure. Also flag glandium for the next review, as he's more in tune with moz.configure work.

Otherwise, this approach seems reasonable.