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

Side by Side Diff: chrome/chrome_exe.gypi

Issue 7598008: Revert overly-wide change in dependency chain. Now the new build step will only be enabled when ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: CL post merge resolution Created 9 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 | « chrome/chrome_dll_syzygy.gypi ('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 # Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2011 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 { 5 {
6 'target_defaults': { 6 'target_defaults': {
7 'variables': { 7 'variables': {
8 'chrome_exe_target': 0, 8 'chrome_exe_target': 0,
9 }, 9 },
10 'target_conditions': [ 10 'target_conditions': [
(...skipping 410 matching lines...) Expand 10 before | Expand all | Expand 10 after
421 ], 421 ],
422 }], 422 }],
423 ['OS=="mac"', { 423 ['OS=="mac"', {
424 'dependencies': [ 424 'dependencies': [
425 # On Mac, make sure we've built chrome_dll, which contains all of 425 # On Mac, make sure we've built chrome_dll, which contains all of
426 # the library code with Chromium functionality. 426 # the library code with Chromium functionality.
427 'chrome_dll', 427 'chrome_dll',
428 ], 428 ],
429 }], 429 }],
430 ['OS=="win"', { 430 ['OS=="win"', {
431 'conditions': [
432 ['optimize_with_syzygy==1', {
433 # With syzygy enabled there is an intermediate target which
434 # builds an initial version of chrome_dll, then optimizes it
435 # to its final location. The optimization step also
436 # depends on chrome_exe, so here we depend on the initial
437 # chrome_dll.
438 'dependencies': ['chrome_dll_initial',]
439 }, {
440 'dependencies': ['chrome_dll',]
441 }],
442 ],
443 }],
444 ['OS=="win"', {
431 'dependencies': [ 445 'dependencies': [
432 # On Windows there is an intermediate target which builds an
433 # initial version of chrome_dll, then either optimizes it or
434 # copies to its final location. The optimization step also
435 # depends on chrome_exe, so here we depend on the initial
436 # chrome_dll.
437 'chrome_dll_initial',
438 'chrome_version_resources', 446 'chrome_version_resources',
439 'installer_util', 447 'installer_util',
440 'installer_util_strings', 448 'installer_util_strings',
441 '../base/base.gyp:base', 449 '../base/base.gyp:base',
442 '../breakpad/breakpad.gyp:breakpad_handler', 450 '../breakpad/breakpad.gyp:breakpad_handler',
443 '../breakpad/breakpad.gyp:breakpad_sender', 451 '../breakpad/breakpad.gyp:breakpad_sender',
444 '../sandbox/sandbox.gyp:sandbox', 452 '../sandbox/sandbox.gyp:sandbox',
445 'app/locales/locales.gyp:*', 453 'app/locales/locales.gyp:*',
446 'app/policy/cloud_policy_codegen.gyp:policy', 454 'app/policy/cloud_policy_codegen.gyp:policy',
447 ], 455 ],
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
502 'configurations': { 510 'configurations': {
503 'Common_Base': { 511 'Common_Base': {
504 'msvs_target_platform': 'x64', 512 'msvs_target_platform': 'x64',
505 }, 513 },
506 }, 514 },
507 }, 515 },
508 ], 516 ],
509 }], 517 }],
510 ], 518 ],
511 } 519 }
OLDNEW
« no previous file with comments | « chrome/chrome_dll_syzygy.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698