Bug 1348796 - Use buildbot --platform arg to determine --enable-debug vs --disable-debug. r=arai, a=test-only
authorSteve Fink <sfink@mozilla.com>
Mon, 20 Mar 2017 10:11:43 -0700
changeset 375531 421f65e89a85a685168f4af3c5ca7e832c0b8346
parent 375530 8dc2b4bb0c68d84e4429ce71ef0dbdcc35cd7a81
child 375532 50b97054d0ed0d9b434501f35955252950764a97
push id10967
push userryanvm@gmail.com
push dateTue, 28 Mar 2017 21:42:19 +0000
treeherdermozilla-aurora@3e5bbabd5279 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersarai, test-only
bugs1348796
milestone54.0a2
Bug 1348796 - Use buildbot --platform arg to determine --enable-debug vs --disable-debug. r=arai, a=test-only
js/src/devtools/automation/autospider.py
--- a/js/src/devtools/automation/autospider.py
+++ b/js/src/devtools/automation/autospider.py
@@ -34,17 +34,17 @@ DIR = directories(os.path, os.getcwd())
 PDIR = directories(posixpath, os.environ["PWD"],
                    fixup=lambda s: re.sub(r'^(\w):', r'/\1', s))
 
 parser = argparse.ArgumentParser(
     description='Run a spidermonkey shell build job')
 parser.add_argument('--dep', action='store_true',
                     help='do not clobber the objdir before building')
 parser.add_argument('--platform', '-p', type=str, metavar='PLATFORM',
-                    default='', help='build platform')
+                    default='', help='build platform, including a suffix ("-debug" or "") used by buildbot to override the variant\'s "debug" setting. The platform can be used to specify 32 vs 64 bits.')
 parser.add_argument('--timeout', '-t', type=int, metavar='TIMEOUT',
                     default=10800,
                     help='kill job after TIMEOUT seconds')
 parser.add_argument('--objdir', type=str, metavar='DIR',
                     default=env.get('OBJDIR', 'obj-spider'),
                     help='object directory')
 group = parser.add_mutually_exclusive_group()
 group.add_argument('--optimize', action='store_true',
@@ -155,16 +155,19 @@ CONFIGURE_ARGS = variant['configure-args
 
 opt = args.optimize
 if opt is None:
     opt = variant.get('optimize')
 if opt is not None:
     CONFIGURE_ARGS += (" --enable-optimize" if opt else " --disable-optimize")
 
 opt = args.debug
+if opt is None and args.platform:
+    # Override variant['debug'].
+    opt = ('-debug' in args.platform)
 if opt is None:
     opt = variant.get('debug')
 if opt is not None:
     CONFIGURE_ARGS += (" --enable-debug" if opt else " --disable-debug")
 
 # Any jobs that wish to produce additional output can save them into the upload
 # directory if there is such a thing, falling back to OBJDIR.
 env.setdefault('MOZ_UPLOAD_DIR', OBJDIR)