647927 - Add SpiderMonkey builds that fail on warnings

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Phil Ringnalda (:philor)]

Assuming bug 646702 lands as-is, we want to add both opt and debug SpiderMonkey builds that pass --enable-sm-fail-on-warnings to configure, on Linux and Mac.

Comment 1

[Phil Ringnalda (:philor)]

Attachment: buildbot-configs, v.1

If I understood the patch that created the other SM builds, and the one that switched them to debug, then this ought to give us builds I can spot in tbpl as "warnaserr" and as "debug" for the debug ones.

Comment 2

[Phil Ringnalda (:philor)]

Attachment: tools/scripts, v.1

Comment 3

[Chris AtLee [:catlee]]

Comment on attachment 524129 [details] [diff] [review]
buildbot-configs, v.1

http://hg.mozilla.org/build/buildbot-configs/rev/0e743bbb306c

Comment 4

[Chris AtLee [:catlee]]

Comment on attachment 524130 [details] [diff] [review]
tools/scripts, v.1

http://hg.mozilla.org/build/tools/rev/c31c8e0757ce

Comment 5

[Chris AtLee [:catlee]]

Comment on attachment 524129 [details] [diff] [review]
buildbot-configs, v.1

Something broke thusly:

Traceback (most recent call last):
  File "/builds/buildbot/preproduction/slave/test-masters/sandbox/lib/python2.6/site-packages/buildbot-0.8.2_hg_c3083a37cd1a_default-py2.6.egg/buildbot/scripts/runner.py", line 1039, in doCheckConfig
    ConfigLoader(configFileName=configFileName)
  File "/builds/buildbot/preproduction/slave/test-masters/sandbox/lib/python2.6/site-packages/buildbot-0.8.2_hg_c3083a37cd1a_default-py2.6.egg/buildbot/scripts/checkconfig.py", line 31, in __init__
    self.loadConfig(configFile, check_synchronously_only=True)
  File "/builds/buildbot/preproduction/slave/test-masters/sandbox/lib/python2.6/site-packages/buildbot-0.8.2_hg_c3083a37cd1a_default-py2.6.egg/buildbot/master.py", line 624, in loadConfig
    exec f in localDict
  File "/builds/buildbot/preproduction/slave/test-masters/buildbot-configs/master_dir/master.cfg", line 50, in <module>
    projectObjects = generateProjectObjects(project, PROJECTS[project], SLAVES)
  File "/builds/buildbot/preproduction/slave/test-masters/buildbotcustom/misc.py", line 3265, in generateProjectObjects
    spiderMonkeyObjects = generateSpiderMonkeyObjects(config, SLAVES)
  File "/builds/buildbot/preproduction/slave/test-masters/buildbotcustom/misc.py", line 3134, in generateSpiderMonkeyObjects
    'env': config['env'][platform],
KeyError: 'linux64'

Comment 6

[Chris AtLee [:catlee]]

Attachment: buildbot-configs, v.2

Add missing environment settings

Comment 7

[Chris AtLee [:catlee]]

Comment on attachment 525706 [details] [diff] [review]
buildbot-configs, v.2

http://hg.mozilla.org/build/buildbot-configs/rev/95cc081632fe

Comment 8

[Phil Ringnalda (:philor)]

Running, and even more remarkable, green!

Comment 9

[Paul Biggar]

(In reply to comment #8)
> Running, and even more remarkable, green!

This sounds like a bug, but I read through one log, and it does appear to be working right and warning free.

I don't see this on tbpl.mozilla.org? Do I just need to wait a bit?

Comment 10

[Phil Ringnalda (:philor)]

Yup, or use http://dev.philringnalda.com/tbpl/?tree=TraceMonkey in the meantime - I really like to wait until after they are running to push the tbpl change, since I usually screw up something about build names if I don't.

Comment 11

[Nicholas Nethercote [inactive]]

(In reply to comment #9)
> (In reply to comment #8)
> > Running, and even more remarkable, green!
> 
> This sounds like a bug

I'm not surprised, it hasn't been that long since bug 609532 was backed out.