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

Side by Side Diff: build/gyp_chromium

Issue 118583003: gyps! and builds! Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 4 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 | « build/filename_rules.gypi ('k') | chrome/browser/chrome_browser_main.cc » ('j') | 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/env python 1 #!/usr/bin/env python
2 2
3 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 3 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 # This script is wrapper for Chromium that adds some support for how GYP 7 # This script is wrapper for Chromium that adds some support for how GYP
8 # is invoked by Chromium beyond what can be done in the gclient hooks. 8 # is invoked by Chromium beyond what can be done in the gclient hooks.
9 9
10 import glob 10 import glob
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
250 250
251 # There shouldn't be a circular dependency relationship between .gyp files, 251 # There shouldn't be a circular dependency relationship between .gyp files,
252 # but in Chromium's .gyp files, on non-Mac platforms, circular relationships 252 # but in Chromium's .gyp files, on non-Mac platforms, circular relationships
253 # currently exist. The check for circular dependencies is currently 253 # currently exist. The check for circular dependencies is currently
254 # bypassed on other platforms, but is left enabled on the Mac, where a 254 # bypassed on other platforms, but is left enabled on the Mac, where a
255 # violation of the rule causes Xcode to misbehave badly. 255 # violation of the rule causes Xcode to misbehave badly.
256 # TODO(mark): Find and kill remaining circular dependencies, and remove this 256 # TODO(mark): Find and kill remaining circular dependencies, and remove this
257 # option. http://crbug.com/35878. 257 # option. http://crbug.com/35878.
258 # TODO(tc): Fix circular dependencies in ChromiumOS then add linux2 to the 258 # TODO(tc): Fix circular dependencies in ChromiumOS then add linux2 to the
259 # list. 259 # list.
260 if sys.platform not in ('darwin',): 260 if sys.platform not in ('darwin',) or True:
261 args.append('--no-circular-check') 261 args.append('--no-circular-check')
262 262
263 # We explicitly don't support the make gyp generator (crbug.com/348686). Be 263 # We explicitly don't support the make gyp generator (crbug.com/348686). Be
264 # nice and fail here, rather than choking in gyp. 264 # nice and fail here, rather than choking in gyp.
265 if re.search(r'(^|,|\s)make($|,|\s)', os.environ.get('GYP_GENERATORS', '')): 265 if re.search(r'(^|,|\s)make($|,|\s)', os.environ.get('GYP_GENERATORS', '')):
266 print 'Error: make gyp generator not supported (check GYP_GENERATORS).' 266 print 'Error: make gyp generator not supported (check GYP_GENERATORS).'
267 sys.exit(1) 267 sys.exit(1)
268 268
269 # Default to ninja on linux and windows, but only if no generator has 269 # Default to ninja on linux and windows, but only if no generator has
270 # explicitly been set. 270 # explicitly been set.
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 subprocess.check_call( 331 subprocess.check_call(
332 [sys.executable, os.path.join(script_dir, 'landmines.py')]) 332 [sys.executable, os.path.join(script_dir, 'landmines.py')])
333 333
334 if vs2013_runtime_dll_dirs: 334 if vs2013_runtime_dll_dirs:
335 x64_runtime, x86_runtime = vs2013_runtime_dll_dirs 335 x64_runtime, x86_runtime = vs2013_runtime_dll_dirs
336 vs_toolchain.CopyVsRuntimeDlls( 336 vs_toolchain.CopyVsRuntimeDlls(
337 os.path.join(chrome_src, GetOutputDirectory()), 337 os.path.join(chrome_src, GetOutputDirectory()),
338 (x86_runtime, x64_runtime)) 338 (x86_runtime, x64_runtime))
339 339
340 sys.exit(gyp_rc) 340 sys.exit(gyp_rc)
OLDNEW
« no previous file with comments | « build/filename_rules.gypi ('k') | chrome/browser/chrome_browser_main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698