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

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

Issue 1685743004: Enable coredumps for dart2js unit tests on buildbot. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 10 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
« no previous file with comments | « tools/bots/bot_utils.py ('k') | 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 os 13 import os
14 import platform 14 import platform
15 import re 15 import re
16 import shutil 16 import shutil
17 import socket 17 import socket
18 import string 18 import string
19 import subprocess 19 import subprocess
20 import sys 20 import sys
21 21
22 import bot 22 import bot
23 import bot_utils
23 24
24 DARTIUM_BUILDER = r'none-dartium-(linux|mac|windows)' 25 DARTIUM_BUILDER = r'none-dartium-(linux|mac|windows)'
25 DART2JS_BUILDER = ( 26 DART2JS_BUILDER = (
26 r'dart2js-(linux|mac|windows)(-(jsshell))?-(debug|release)(-(checked|host-ch ecked))?(-(host-checked))?(-(minified))?(-(x64))?(-(cps))?-?(\d*)-?(\d*)') 27 r'dart2js-(linux|mac|windows)(-(jsshell))?-(debug|release)(-(checked|host-ch ecked))?(-(host-checked))?(-(minified))?(-(x64))?(-(cps))?-?(\d*)-?(\d*)')
27 DART2JS_FULL_BUILDER = r'full-(linux|mac|win7|win8)(-(ie10|ie11))?(-checked)?(-m inified)?-(\d+)-(\d+)' 28 DART2JS_FULL_BUILDER = r'full-(linux|mac|win7|win8)(-(ie10|ie11))?(-checked)?(-m inified)?-(\d+)-(\d+)'
28 WEB_BUILDER = ( 29 WEB_BUILDER = (
29 r'dart2js-(ie9|ie10|ie11|ff|safari|chrome|chromeOnAndroid|safarimobilesim|op era|drt)-(win7|win8|mac10\.7|mac10\.8|mac10\.9|linux)(-(all|html))?(-(csp))?(-(\ d+)-(\d+))?') 30 r'dart2js-(ie9|ie10|ie11|ff|safari|chrome|chromeOnAndroid|safarimobilesim|op era|drt)-(win7|win8|mac10\.7|mac10\.8|mac10\.9|linux)(-(all|html))?(-(csp))?(-(\ d+)-(\d+))?')
30 31
31 IE_VERSIONS = ['ie10', 'ie11'] 32 IE_VERSIONS = ['ie10', 'ie11']
32 33
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
277 pass 278 pass
278 279
279 unit_test_flags = [flag for flag in flags if flag.startswith('--shard')] 280 unit_test_flags = [flag for flag in flags if flag.startswith('--shard')]
280 # Run the unit tests in checked mode (the VM's checked mode). 281 # Run the unit tests in checked mode (the VM's checked mode).
281 unit_test_flags.append('--checked') 282 unit_test_flags.append('--checked')
282 unit_test_flags.append('--vm-options=--abort-on-assertion-errors') 283 unit_test_flags.append('--vm-options=--abort-on-assertion-errors')
283 if runtime == 'd8': 284 if runtime == 'd8':
284 # The dart2js compiler isn't self-hosted (yet) so we run its 285 # The dart2js compiler isn't self-hosted (yet) so we run its
285 # unit tests on the VM. We avoid doing this on the builders 286 # unit tests on the VM. We avoid doing this on the builders
286 # that run the browser tests to cut down on the cycle time. 287 # that run the browser tests to cut down on the cycle time.
287 TestStep("dart2js_unit", mode, system, 'none', 'vm', ['dart2js', 'try'], 288 with bot_utils.CoredumpEnabler():
288 unit_test_flags, arch) 289 TestStep("dart2js_unit", mode, system, 'none', 'vm', ['dart2js', 'try'],
290 unit_test_flags, arch)
kustermann 2016/02/11 13:45:19 Is it intentionally only for 'd8' runtime?
289 291
290 if compiler == 'dart2js' and runtime == 'drt': 292 if compiler == 'dart2js' and runtime == 'drt':
291 # Ensure that we run the "try" tests on Content Shell. 293 # Ensure that we run the "try" tests on Content Shell.
292 TestStep("incremental_compilation", mode, system, 'none', runtime, 294 TestStep("incremental_compilation", mode, system, 'none', runtime,
293 ['try'], unit_test_flags, arch) 295 ['try'], unit_test_flags, arch)
294 296
295 if compiler == 'dart2js' and runtime in ['ie10', 'ie11']: 297 if compiler == 'dart2js' and runtime in ['ie10', 'ie11']:
296 TestStep(compiler, mode, system, compiler, runtime, 298 TestStep(compiler, mode, system, compiler, runtime,
297 ['html', 'pkg', 'samples', 'co19'], flags, arch) 299 ['html', 'pkg', 'samples', 'co19'], flags, arch)
298 else: 300 else:
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
413 if build_info.mode == 'debug': 415 if build_info.mode == 'debug':
414 target = 'dart2js_bot_debug' 416 target = 'dart2js_bot_debug'
415 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode, 417 args = [sys.executable, './tools/build.py', '--mode=' + build_info.mode,
416 '--arch=' + build_info.arch, target] 418 '--arch=' + build_info.arch, target]
417 print 'Build SDK and d8: %s' % (' '.join(args)) 419 print 'Build SDK and d8: %s' % (' '.join(args))
418 bot.RunProcess(args) 420 bot.RunProcess(args)
419 421
420 422
421 if __name__ == '__main__': 423 if __name__ == '__main__':
422 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler) 424 bot.RunBot(GetBuildInfo, RunCompilerTests, build_step=BuildCompiler)
OLDNEW
« no previous file with comments | « tools/bots/bot_utils.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698