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

Side by Side Diff: chrome/chrome_exe.gypi

Issue 7634025: Build chrome.exe into intermediate dir then copy it to output dir. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: The /j param isn't supported on XP 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 | « 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 (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 27 matching lines...) Expand all
38 # listed here. 38 # listed here.
39 'mac_bundle_resources!': [ 39 'mac_bundle_resources!': [
40 'app/app-Info.plist', 40 'app/app-Info.plist',
41 ], 41 ],
42 'xcode_settings': { 42 'xcode_settings': {
43 'CHROMIUM_STRIP_SAVE_FILE': 'app/app.saves', 43 'CHROMIUM_STRIP_SAVE_FILE': 'app/app.saves',
44 'INFOPLIST_FILE': 'app/app-Info.plist', 44 'INFOPLIST_FILE': 'app/app-Info.plist',
45 }, 45 },
46 'conditions': [ 46 'conditions': [
47 ['OS=="win"', { 47 ['OS=="win"', {
48 # TODO(scottbyer): This is a temporary workaround. The right fix
49 # is to change the output file to be in $(IntDir) for this project
50 # and the .dll project and use the hardlink script to link it back
51 # to $(OutDir).
52 'configurations': {
53 'Debug_Base': {
54 'msvs_settings': {
55 'VCLinkerTool': {
56 'LinkIncremental': '1',
57 },
58 },
59 },
60 },
61 'msvs_settings': { 48 'msvs_settings': {
62 'VCLinkerTool': { 49 'VCLinkerTool': {
63 'DelayLoadDLLs': [ 50 'DelayLoadDLLs': [
64 'dbghelp.dll', 51 'dbghelp.dll',
65 'dwmapi.dll', 52 'dwmapi.dll',
66 'uxtheme.dll', 53 'uxtheme.dll',
67 'ole32.dll', 54 'ole32.dll',
68 'oleaut32.dll', 55 'oleaut32.dll',
69 ], 56 ],
70 # Set /SUBSYSTEM:WINDOWS for chrome.exe itself. 57 # Set /SUBSYSTEM:WINDOWS for chrome.exe itself.
(...skipping 383 matching lines...) Expand 10 before | Expand all | Expand 10 after
454 'app/policy/cloud_policy_codegen.gyp:policy', 441 'app/policy/cloud_policy_codegen.gyp:policy',
455 ], 442 ],
456 'sources': [ 443 'sources': [
457 'app/chrome_exe.rc', 444 'app/chrome_exe.rc',
458 '<(SHARED_INTERMEDIATE_DIR)/chrome_version/chrome_exe_version.rc', 445 '<(SHARED_INTERMEDIATE_DIR)/chrome_version/chrome_exe_version.rc',
459 ], 446 ],
460 'msvs_settings': { 447 'msvs_settings': {
461 'VCLinkerTool': { 448 'VCLinkerTool': {
462 'ImportLibrary': '$(OutDir)\\lib\\chrome_exe.lib', 449 'ImportLibrary': '$(OutDir)\\lib\\chrome_exe.lib',
463 'ProgramDatabaseFile': '$(OutDir)\\chrome_exe.pdb', 450 'ProgramDatabaseFile': '$(OutDir)\\chrome_exe.pdb',
451 # In order to support incremental linking for both chrome exe and
452 # dll we need to build them to different directories (otherwise
453 # both files are called chrome.ilk and collide in the output
454 # directory. We build chrome.exe to the intermediate directory
455 # and add a postbuild step (below) to copy the resulting
456 # executable to the output dir.
457 'OutputFile': '$(IntDir)\\chrome.exe',
464 }, 458 },
465 }, 459 },
460 'msvs_postbuild': [
461 'XCOPY /Y /F /R /K /I "$(IntDir)\\chrome.exe" "$(OutDir)"',
Roger McFarlane (Google) 2011/08/12 18:40:33 Linking isn't supported on Windows XP (which is th
Roger McFarlane (Google) 2011/08/12 20:35:03 Muahahaha ... tools\build\win\hardlink_failsafe.ba
462 ],
466 }], 463 }],
467 ], 464 ],
468 }, 465 },
469 ], 466 ],
470 'conditions': [ 467 'conditions': [
471 ['OS=="win"', { 468 ['OS=="win"', {
472 'targets': [ 469 'targets': [
473 { 470 {
474 'target_name': 'chrome_nacl_win64', 471 'target_name': 'chrome_nacl_win64',
475 'type': 'executable', 472 'type': 'executable',
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 'configurations': { 507 'configurations': {
511 'Common_Base': { 508 'Common_Base': {
512 'msvs_target_platform': 'x64', 509 'msvs_target_platform': 'x64',
513 }, 510 },
514 }, 511 },
515 }, 512 },
516 ], 513 ],
517 }], 514 }],
518 ], 515 ],
519 } 516 }
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