Issue 4677 - Add support for x86 builds in adblockbrowser-build

build.py

 #!/usr/bin/env python
 
 import glob
 import json
 import os
 import re
 import shutil
 import string
 import subprocess
 import sys
@@ skipping 39 matching lines @@
         if distribution_mode == DIST_STORE:
             mozconfig.write(". \"%s\"\n" % MOZCONFIG_STORE_PATH)
         if build_mode == BUILD_RELEASE:
             mozconfig.write(". \"%s\"\n" % MOZCONFIG_RELEASE_PATH)
         mozconfig.write(". \"%s\"\n" % MOZCONFIG_CUSTOM_PATH)
         return mozconfig_path
 
 
 def build(architecture, distribution_mode, build_mode, sdk_path, ndk_path):
     build_environment = os.environ.copy()
     build_environment["MOZCONFIG"] = generate_mozconfig(

[anton, 2016/12/01 07:03:58]

it might be more reliable not just remove `find_mozconfig` and `check_mozconfig`
and assume it's valid, but genereate_mozconfig and use `check_mozconfig` for
just generated mozconfig. It would be easier to merge build.py from upstream and
be sure generated config is still valid. Just an idea though.

[diegocarloslima, 2016/12/01 10:35:59]

I understand your point of view, but my reasons to remove mozconfig validation
is to avoid the addition of a lot of boilerplate code as we increase our build
variants. Instead, I added comments to each mozconfig stating that they
shouldn't be changed. For adding custom configuration, I have provided the
mozconfig-custom file. Actually, even with the checks, the user can easily
bypass it just by commenting out the check function calls. So none of the
approaches can prevent the user from screwing up the build process if he wants
to do so.

         architecture, distribution_mode, build_mode)
     build_environment["ANDROID_SDK_PATH"] = sdk_path
     build_environment["ANDROID_NDK_PATH"] = ndk_path
     obj_dir = OBJDIR_X86 if architecture == ARCH_X86 else OBJDIR_ARM
     build_environment["OBJDIR"] = obj_dir
     subprocess.check_call([os.path.join(ABB_PATH, "mach"), "clobber"],
                           env=build_environment)
     subprocess.check_call([MULTI_L10N_PATH, "--cfg", "adblockbrowser-cfg.py"],
                           env=build_environment)
 
@@ skipping 46 matching lines @@
 
     if do_sign:
         if len(sys.argv) < 3:
             print_usage()
             sys.exit(3)
         apk_path = sys.argv[2]
 
     subprocess.check_call([ENSURE_DEPENDENCIES_PATH])
     import config
 
-    architecture = config.ARCHITECTURE
-    if architecture not in (ARCH_ARM, ARCH_X86):
-        print >>sys.stderr, "Invalid architecture, check config.py"
-        sys.exit(4)

[anton, 2016/12/01 07:03:58]

From integration point of view (f.e. CI server - let's say someone has
configured CI server and it starts working incorrect one day) it's better not to
change existing return codes but just add new ones instead. Not a significant
remark, but anyway. It means that we should use `6` here and still use `4` and
`5` below.

[diegocarloslima, 2016/12/01 10:35:59]

Acknowledged.

-
     distribution_mode = config.DISTRIBUTION_MODE
     if distribution_mode not in ("standalone", "store"):
         print >>sys.stderr, "Invalid distribution mode, check config.py"
-        sys.exit(5)
+        sys.exit(4)
 
     build_mode = config.BUILD_MODE
     if build_mode not in ("devbuild", "release"):
         print >>sys.stderr, "Invalid build mode, check config.py"
+        sys.exit(5)
+
+    architecture = config.ARCHITECTURE
+    if architecture not in (ARCH_ARM, ARCH_X86):
+        print >>sys.stderr, "Invalid architecture, check config.py"
         sys.exit(6)
 
     if do_build:
         apk_path = build(architecture, distribution_mode, build_mode,
                          config.ANDROID_SDK_PATH, config.ANDROID_NDK_PATH)
     if do_sign:
         sign(apk_path, config.ANDROID_KEYSTORE_PATH, config.ANDROID_KEY_NAME)
     else:
         print apk_path