Bug 1496523 - Remove nodejs requirement for building source package. r=tomprince DONTBUILD
authorRob Lemley <rob@thunderbird.net>
Thu, 04 Oct 2018 20:42:01 +0000
changeset 33585 0399d51b31fe19d71082b1c6b2a4bb79c7493848
parent 33584 80500cc1cdfbaec19b550ea1ef509f0b9f6b8530
child 33586 74e34747deec61e12bf843ad99a0c2efa830e292
push id388
push userclokep@gmail.com
push dateMon, 28 Jan 2019 20:54:56 +0000
reviewerstomprince
bugs1496523
Bug 1496523 - Remove nodejs requirement for building source package. r=tomprince DONTBUILD The generated .mozconfig file for building the source tarball will require nodejs to be present. Firefox has included in their mozconfig for source builds: ac_add_options --disable-nodejs Differential Revision: https://phabricator.services.mozilla.com/D7788
mail/config/mozconfigs/linux64/source
--- a/mail/config/mozconfigs/linux64/source
+++ b/mail/config/mozconfigs/linux64/source
@@ -1,6 +1,11 @@
 . "$topsrcdir/comm/mail/config/mozconfigs/common"
 
 # The source "build" only needs a mozconfig because we use the build system as
 # our script for generating it. This allows us to run configure without any
 # extra dependencies on specific toolchains, e.g. gtk3.
 . "$topsrcdir/build/mozconfig.no-compile"
+
+# mozconfig.no-compile does not remove nodejs requirement.
+# The source mozconfig for FF includes the following:
+ac_add_options --disable-nodejs
+unset NODEJS