Bug 1375271 - Add an explicit --target to osx l10n mozconfigs. r?catlee draft
authorMike Hommey <mh+mozilla@glandium.org>
Thu, 22 Jun 2017 06:48:44 +0900
changeset 598610 e149ca67bbdbea7d6bedabb530a1480e88fa5751
parent 597455 464b2a3c25aa1065760d9ecbb0870bca4a66c62e
child 634517 af0cbfb8ebd5fe5cd46a681b115c59304d03f5b2
push id65252
push userbmo:mh+mozilla@glandium.org
push dateWed, 21 Jun 2017 22:00:49 +0000
reviewerscatlee
bugs1375271, 1373150
milestone56.0a1
Bug 1375271 - Add an explicit --target to osx l10n mozconfigs. r?catlee Before bug 1373150, the l10n mozconfigs would include compilation-related mozconfigs, which include setting compiler paths, and setting --target for cross builds. After bug 1373150, the compilation-related mozconfigs are not included anymore, which made --target unset for l10n builds, leaving it to configure to guess what it is based on the host. Which for cross builds is Linux, so configure would set things up for a linux build, which doesn't work to do osx l10n repacks. So we add back an explicit --target to those mozconfigs, without including all the compilation-related things. We set the target to x86_64-apple-darwin, which is the same as what was set through build/macosx/cross-mozconfig.common. This is different from what the native osx builds would get (x86_64-apple-darwin11.2.0), but the extra version in that target is actually not relevant and native builds shouldn't care that it's gone.
browser/config/mozconfigs/macosx64/l10n-mozconfig
browser/config/mozconfigs/macosx64/l10n-mozconfig-devedition
--- a/browser/config/mozconfigs/macosx64/l10n-mozconfig
+++ b/browser/config/mozconfigs/macosx64/l10n-mozconfig
@@ -1,11 +1,14 @@
 . "$topsrcdir/browser/config/mozconfigs/common"
 . "$topsrcdir/build/mozconfig.no-compile"
 
+# We need to indicate the target for cross builds
+ac_add_options --target=x86_64-apple-darwin
+
 ac_add_options --with-l10n-base=../../l10n
 ac_add_options --enable-update-channel=${MOZ_UPDATE_CHANNEL}
 ac_add_options --with-branding=browser/branding/nightly
 
 if test "${MOZ_UPDATE_CHANNEL}" = "nightly"; then
 ac_add_options --with-macbundlename-prefix=Firefox
 fi
 
--- a/browser/config/mozconfigs/macosx64/l10n-mozconfig-devedition
+++ b/browser/config/mozconfigs/macosx64/l10n-mozconfig-devedition
@@ -1,11 +1,14 @@
 . "$topsrcdir/browser/config/mozconfigs/common"
 . "$topsrcdir/build/mozconfig.no-compile"
 
+# We need to indicate the target for cross builds
+ac_add_options --target=x86_64-apple-darwin
+
 ac_add_options --with-l10n-base=../../l10n
 ac_add_options --enable-update-channel=${MOZ_UPDATE_CHANNEL}
 ac_add_options --with-branding=browser/branding/aurora
 
 if test "${MOZ_UPDATE_CHANNEL}" = "nightly"; then
 ac_add_options --with-macbundlename-prefix=Firefox
 fi