Bug 1277553 - Specify adjust sandbox token if not beta or release build. r=sebastian draft
authorMichael Comella <michael.l.comella@gmail.com>
Fri, 03 Jun 2016 17:10:40 -0700
changeset 375343 96930d595ebc22c06dadea1d28782ec6a2c023c2
parent 375342 1a474e3506c180e6c7403a377594203a96b41675
child 522841 88cd62baebe61d8cc354666d0dd686ab6ee40261
push id20246
push usermichael.l.comella@gmail.com
push dateSat, 04 Jun 2016 00:15:38 +0000
reviewerssebastian
bugs1277553
milestone49.0a1
Bug 1277553 - Specify adjust sandbox token if not beta or release build. r=sebastian I didn't include nalexander's MOZ_INSTALL_TRACKING suggestion because my make skills are too shaky to make this worthwhile. Specifying a keyfile when MOZ_INSTALL_TRACKING is disabled isn't harmful afaik (though it's a little spammy). Also, added code comment duplicated for emphasis: # (bug 1277553) In Aurora -> Beta simulation builds, no update channel is # specified, causing an assertion to throw that MOZ_INSTALL_TRACKING is # specified but the keyfile is not. In this case, we add a default keyfile. # This has the disadvantage that if our beta/release checks above ever # fail, we'll come to this default case and the compile-time check to # specify a valid keyfile will be broken. I don't have any better # alternatives. MozReview-Commit-ID: 7tmemvpDaW8
mobile/android/config/mozconfigs/common
--- a/mobile/android/config/mozconfigs/common
+++ b/mobile/android/config/mozconfigs/common
@@ -51,16 +51,25 @@ ac_add_options --enable-update-channel=$
 # Treat warnings as errors (modulo ALLOW_COMPILER_WARNINGS).
 ac_add_options --enable-warnings-as-errors
 
 ac_add_options --with-mozilla-api-keyfile=/builds/mozilla-fennec-geoloc-api.key
 if test "$MOZ_UPDATE_CHANNEL" = "release" ; then
     ac_add_options --with-adjust-sdk-keyfile=/builds/adjust-sdk.token
 elif test "$MOZ_UPDATE_CHANNEL" = "beta" ; then
     ac_add_options --with-adjust-sdk-keyfile=/builds/adjust-sdk-beta.token
+else
+    # (bug 1277553) In Aurora -> Beta simulation builds, no update channel is
+    # specified, causing an assertion to throw that MOZ_INSTALL_TRACKING is
+    # specified but the keyfile is not. In this case, we add a default keyfile.
+    # This has the disadvantage that if our beta/release checks above ever
+    # fail, we'll come to this default case and the compile-time check to
+    # specify a valid keyfile will be broken. I don't have any better
+    # alternatives.
+    ac_add_options --with-adjust-sdk-keyfile="$topsrcdir/mobile/android/base/adjust-sdk-sandbox.token"
 fi
 export SOCORRO_SYMBOL_UPLOAD_TOKEN_FILE=/builds/crash-stats-api.token
 
 # Package js shell.
 export MOZ_PACKAGE_JSSHELL=1
 
 # Use ccache
 . "$topsrcdir/build/mozconfig.cache"