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

Side by Side Diff: tools/auto_bisect/builder.py

Issue 593253002: Follow-up to crrev.com/583793002 -- check if goma_dir is None. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 | « 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 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Classes and functions for building Chrome. 5 """Classes and functions for building Chrome.
6 6
7 This includes functions for running commands to build, as well as 7 This includes functions for running commands to build, as well as
8 specific rules about which targets to build. 8 specific rules about which targets to build.
9 """ 9 """
10 10
(...skipping 21 matching lines...) Expand all
32 """ 32 """
33 if bisect_utils.IsWindowsHost(): 33 if bisect_utils.IsWindowsHost():
34 if not opts.build_preference: 34 if not opts.build_preference:
35 opts.build_preference = 'msvs' 35 opts.build_preference = 'msvs'
36 36
37 if opts.build_preference == 'msvs': 37 if opts.build_preference == 'msvs':
38 if not os.getenv('VS100COMNTOOLS'): 38 if not os.getenv('VS100COMNTOOLS'):
39 raise RuntimeError( 39 raise RuntimeError(
40 'Path to visual studio could not be determined.') 40 'Path to visual studio could not be determined.')
41 else: 41 else:
42 # Need to re-escape goma dir, see crbug.com/394990.
43 if opts.goma_dir:
44 opts.goma_dir = opts.goma_dir.encode('string_escape')
42 SetBuildSystemDefault(opts.build_preference, opts.use_goma, 45 SetBuildSystemDefault(opts.build_preference, opts.use_goma,
43 # Need to re-escape goma dir, see crbug.com/394990 46 opts.goma_dir)
44 opts.goma_dir.encode('string_escape'))
45 else: 47 else:
46 if not opts.build_preference: 48 if not opts.build_preference:
47 if 'ninja' in os.getenv('GYP_GENERATORS', default=''): 49 if 'ninja' in os.getenv('GYP_GENERATORS', default=''):
48 opts.build_preference = 'ninja' 50 opts.build_preference = 'ninja'
49 else: 51 else:
50 opts.build_preference = 'make' 52 opts.build_preference = 'make'
51 53
52 SetBuildSystemDefault(opts.build_preference, opts.use_goma, opts.goma_dir) 54 SetBuildSystemDefault(opts.build_preference, opts.use_goma, opts.goma_dir)
53 55
54 if not SetupPlatformBuildEnvironment(opts): 56 if not SetupPlatformBuildEnvironment(opts):
(...skipping 393 matching lines...) Expand 10 before | Expand all | Expand 10 after
448 # (See http://crrev.com/170273005). So, we set this variable explicitly here 450 # (See http://crrev.com/170273005). So, we set this variable explicitly here
449 # in order to build Chrome on Android. 451 # in order to build Chrome on Android.
450 if 'GYP_DEFINES' not in os.environ: 452 if 'GYP_DEFINES' not in os.environ:
451 os.environ['GYP_DEFINES'] = 'OS=android' 453 os.environ['GYP_DEFINES'] = 'OS=android'
452 else: 454 else:
453 os.environ['GYP_DEFINES'] += ' OS=android' 455 os.environ['GYP_DEFINES'] += ' OS=android'
454 456
455 if opts.use_goma: 457 if opts.use_goma:
456 os.environ['GYP_DEFINES'] += ' use_goma=1' 458 os.environ['GYP_DEFINES'] += ' use_goma=1'
457 return not proc.returncode 459 return not proc.returncode
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