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

Unified Diff: tools/build.py

Issue 13100003: Made the default build target "most" which currently contains everything except (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
« tests/html/html.status ('K') | « tests/html/html.status ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/build.py
diff --git a/tools/build.py b/tools/build.py
index d2bf1bc240330a21f5e0168983eda19f07c5f24a..e2839c4f6875919483331ba8f077e361bc5a2b8b 100755
--- a/tools/build.py
+++ b/tools/build.py
@@ -232,12 +232,9 @@ def Main():
if not ProcessOptions(options, args):
parser.print_help()
return 1
- # Determine which targets to build. By default we build the "all" target.
+ # Determine which targets to build. By default we build the "most" target.
if len(args) == 0:
- if HOST_OS == 'macos':
- target = 'All'
- else:
- target = 'all'
+ target = 'most'
kustermann 2013/03/27 10:21:21 If somebody types in './tools/build.py -mrelease'
Andrei Mouravski 2013/03/27 17:45:40 This was a personal request from Dan, which is why
dgrove 2013/03/28 02:30:11 Yes - this was primarily for the VM team, but also
Andrei Mouravski 2013/03/28 15:06:23 To which I ask, do most people need the editor? Or
dgrove 2013/03/28 15:17:43 probably not, but this step takes so long that it
else:
target = args[0]
@@ -265,13 +262,7 @@ def Main():
project_file = 'dart.sln'
if os.path.exists('dart-%s.gyp' % CurrentDirectoryBaseName()):
project_file = 'dart-%s.sln' % CurrentDirectoryBaseName()
- if target == 'all':
- args = [options.devenv + os.sep + options.executable,
- '/build',
- build_config,
- project_file
- ]
- else:
+ if target == 'most':
args = [options.devenv + os.sep + options.executable,
'/build',
build_config,
« tests/html/html.status ('K') | « tests/html/html.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698