Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(518)

Side by Side Diff: tools/bots/compiler.py

Issue 12035004: Add support for having both 10.8 and 10.7 mac bots side by side on the buildbot waterfall (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/python 1 #!/usr/bin/python
2 2
3 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 3 # Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
4 # for details. All rights reserved. Use of this source code is governed by a 4 # for details. All rights reserved. Use of this source code is governed by a
5 # BSD-style license that can be found in the LICENSE file. 5 # BSD-style license that can be found in the LICENSE file.
6 6
7 """ 7 """
8 Dart2js buildbot steps 8 Dart2js buildbot steps
9 9
10 Runs tests for the dart2js compiler. 10 Runs tests for the dart2js compiler.
11 """ 11 """
12 12
13 import platform 13 import platform
14 import os 14 import os
15 import re 15 import re
16 import shutil 16 import shutil
17 import subprocess 17 import subprocess
18 import sys 18 import sys
19 19
20 import bot 20 import bot
21 21
22 DART2JS_BUILDER = ( 22 DART2JS_BUILDER = (
23 r'dart2js-(linux|mac|windows)(-(jsshell))?-(debug|release)(-(checked|host-ch ecked))?(-(host-checked))?(-(minified))?-?(\d*)-?(\d*)') 23 r'dart2js-(linux|mac|windows)(-(jsshell))?-(debug|release)(-(checked|host-ch ecked))?(-(host-checked))?(-(minified))?-?(\d*)-?(\d*)')
24 WEB_BUILDER = ( 24 WEB_BUILDER = (
25 r'dart2js-(ie9|ie10|ff|safari|chrome|opera)-(win7|win8|mac|linux)(-(all|html ))?(-(\d+)-(\d+))?') 25 r'dart2js-(ie9|ie10|ff|safari|chrome|opera)-(win7|win8|mac10\.8|mac10\.7|lin ux)(-(all|html))?(-(\d+)-(\d+))?')
Lasse Reichstein Nielsen 2013/01/21 08:02:00 mac10\.[78]
26 26
27 27
28 def GetBuildInfo(builder_name, is_buildbot): 28 def GetBuildInfo(builder_name, is_buildbot):
29 """Returns a BuildInfo object for the current buildbot based on the 29 """Returns a BuildInfo object for the current buildbot based on the
30 name of the builder. 30 name of the builder.
31 """ 31 """
32 compiler = None 32 compiler = None
33 runtime = None 33 runtime = None
34 mode = None 34 mode = None
35 system = None 35 system = None
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 if dart2js_pattern.group(10) == 'minified': 72 if dart2js_pattern.group(10) == 'minified':
73 minified = True 73 minified = True
74 shard_index = dart2js_pattern.group(11) 74 shard_index = dart2js_pattern.group(11)
75 total_shards = dart2js_pattern.group(12) 75 total_shards = dart2js_pattern.group(12)
76 else : 76 else :
77 return None 77 return None
78 78
79 if system == 'windows': 79 if system == 'windows':
80 system = 'win7' 80 system = 'win7'
81 81
82 # We have both 10.8 and 10.7 bots, functionality is the same.
83 if system == 'mac10.8' or system == 'mac10.7':
Lasse Reichstein Nielsen 2013/01/21 08:02:00 Is there a "startsWith" so you can just check if i
84 system = 'mac'
85
82 if (system == 'win7' and platform.system() != 'Windows') or ( 86 if (system == 'win7' and platform.system() != 'Windows') or (
83 system == 'mac' and platform.system() != 'Darwin') or ( 87 system == 'mac' and platform.system() != 'Darwin') or (
84 system == 'linux' and platform.system() != 'Linux'): 88 system == 'linux' and platform.system() != 'Linux'):
85 print ('Error: You cannot emulate a buildbot with a platform different ' 89 print ('Error: You cannot emulate a buildbot with a platform different '
86 'from your own.') 90 'from your own.')
87 return None 91 return None
88
89 return bot.BuildInfo(compiler, runtime, mode, system, checked, host_checked, 92 return bot.BuildInfo(compiler, runtime, mode, system, checked, host_checked,
90 minified, shard_index, total_shards, is_buildbot, 93 minified, shard_index, total_shards, is_buildbot,
91 test_set) 94 test_set)
92 95
93 96
94 def NeedsXterm(compiler, runtime): 97 def NeedsXterm(compiler, runtime):
95 return runtime in ['ie9', 'ie10', 'chrome', 'safari', 'opera', 'ff', 'drt'] 98 return runtime in ['ie9', 'ie10', 'chrome', 'safari', 'opera', 'ff', 'drt']
96 99
97 100
98 def TestStepName(name, flags): 101 def TestStepName(name, flags):
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 """ 322 """
320 with bot.BuildStep('Build SDK and d8'): 323 with bot.BuildStep('Build SDK and d8'):
321 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode, 324 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode,
322 'dart2js_bot'] 325 'dart2js_bot']
323 print 'Build SDK and d8: %s' % (' '.join(args)) 326 print 'Build SDK and d8: %s' % (' '.join(args))
324 bot.RunProcess(args) 327 bot.RunProcess(args)
325 328
326 329
327 if __name__ == '__main__': 330 if __name__ == '__main__':
328 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler) 331 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698