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

Side by Side Diff: scripts/slave/compile.py

Issue 2856030: [Mac] remove -parallelizeTargets since causes xcode to sometimes build an inv... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/buildbot/
Patch Set: Created 10 years, 5 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 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """A tool to build chrome, executed by buildbot. 6 """A tool to build chrome, executed by buildbot.
7 7
8 When this is run, the current directory (cwd) should be the outer build 8 When this is run, the current directory (cwd) should be the outer build
9 directory (e.g., chrome-release/build/). 9 directory (e.g., chrome-release/build/).
10 10
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 # TODO(mmoss) Support the old 'args' usage until we're confident the master is 49 # TODO(mmoss) Support the old 'args' usage until we're confident the master is
50 # switched to passing '--solution' everywhere. 50 # switched to passing '--solution' everywhere.
51 if not '-project' in args: 51 if not '-project' in args:
52 # TODO(mmoss) Temporary hack to ignore the Windows --solution flag that is 52 # TODO(mmoss) Temporary hack to ignore the Windows --solution flag that is
53 # passed to all builders. This can be taken out once the master scripts are 53 # passed to all builders. This can be taken out once the master scripts are
54 # updated to only pass platform-appropriate --solution values. 54 # updated to only pass platform-appropriate --solution values.
55 if not options.solution or os.path.splitext(options.solution)[1] != '.xcodep roj': 55 if not options.solution or os.path.splitext(options.solution)[1] != '.xcodep roj':
56 options.solution = 'all.xcodeproj' 56 options.solution = 'all.xcodeproj'
57 command.extend(['-project', options.solution]) 57 command.extend(['-project', options.solution])
58 58
59 # With the 3.2.x toolchain, it seems xcodebuild doesn't honor the project's
60 # BuildIndependentTargetsInParallel setting (the IDE does). Work around this
61 # by directly adding the command line argument to force parallel target
62 # building. (radr://8150925)
63 command.extend(['-parallelizeTargets'])
64
65 # Most of the bot hostnames are in the pattern of "base####.subnet.domain". 59 # Most of the bot hostnames are in the pattern of "base####.subnet.domain".
66 # To use the "base" and "subnet" to pull a list of distccd servers to use the 60 # To use the "base" and "subnet" to pull a list of distccd servers to use the
67 # that bot. That way, the files can easily be tweaked to affect all of the 61 # that bot. That way, the files can easily be tweaked to affect all of the
68 # bots in a given location. 10.5 and 10.6 get different Xcode versions with 62 # bots in a given location. 10.5 and 10.6 get different Xcode versions with
69 # slightly different versions of gcc and the SDKs, so we need a different set 63 # slightly different versions of gcc and the SDKs, so we need a different set
70 # of hosts for each toolchain. 64 # of hosts for each toolchain.
71 uname_tuple = os.uname() 65 uname_tuple = os.uname()
72 full_hostname = uname_tuple[1] 66 full_hostname = uname_tuple[1]
73 os_revision = uname_tuple[2] 67 os_revision = uname_tuple[2]
74 split_full_hostname = full_hostname.split('.', 2) 68 split_full_hostname = full_hostname.split('.', 2)
(...skipping 374 matching lines...) Expand 10 before | Expand all | Expand 10 after
449 'scons' : main_scons, 443 'scons' : main_scons,
450 'xcode' : main_xcode, 444 'xcode' : main_xcode,
451 'scons_v8' : main_scons_v8, 445 'scons_v8' : main_scons_v8,
452 } 446 }
453 main = build_tool_map.get(options.build_tool) 447 main = build_tool_map.get(options.build_tool)
454 if not main: 448 if not main:
455 sys.stderr.write('Unknown build tool %s.\n' % repr(options.build_tool)) 449 sys.stderr.write('Unknown build tool %s.\n' % repr(options.build_tool))
456 sys.exit(2) 450 sys.exit(2)
457 451
458 sys.exit(main(options, args)) 452 sys.exit(main(options, args))
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