Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 'target_defaults': { | 5 'target_defaults': { |
| 6 'variables': { | 6 'variables': { |
| 7 'chrome_dll_target': 0, | 7 'chrome_dll_target': 0, |
| 8 }, | 8 }, |
| 9 'target_conditions': [ | 9 'target_conditions': [ |
| 10 ['chrome_dll_target==1', { | 10 ['chrome_dll_target==1', { |
| (...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 58 ], # conditions | 58 ], # conditions |
| 59 }], | 59 }], |
| 60 ], | 60 ], |
| 61 }, | 61 }, |
| 62 'conditions': [ | 62 'conditions': [ |
| 63 ['OS=="mac" or OS=="win"', { | 63 ['OS=="mac" or OS=="win"', { |
| 64 'targets': [ | 64 'targets': [ |
| 65 { | 65 { |
| 66 'variables': { | 66 'variables': { |
| 67 'chrome_dll_target': 1, | 67 'chrome_dll_target': 1, |
| 68 'default_apps_files': [ | |
| 69 'browser/resources/default_apps/gmail.crx', | |
| 70 'browser/resources/default_apps/youtube.crx', | |
| 71 ], | |
| 68 'conditions' : [ | 72 'conditions' : [ |
| 69 ['OS=="win" and optimize_with_syzygy==1', { | 73 ['OS=="win" and optimize_with_syzygy==1', { |
| 70 # On Windows we use build chrome_dll as an intermediate target | 74 # On Windows we use build chrome_dll as an intermediate target |
| 71 # then have a subsequent step which either optimizes it to its | 75 # then have a subsequent step which either optimizes it to its |
| 72 # final location, or copies it to its final location, depending | 76 # final location, or copies it to its final location, depending |
| 73 # on whether or not optimize_with_syzygy==1. Please, refer to | 77 # on whether or not optimize_with_syzygy==1. Please, refer to |
| 74 # chrome_dll_syzygy.gypi for the subsequent defintion of the | 78 # chrome_dll_syzygy.gypi for the subsequent defintion of the |
| 75 # Windows chrome_dll target. | 79 # Windows chrome_dll target. |
| 76 'dll_target_name': 'chrome_dll_initial', | 80 'dll_target_name': 'chrome_dll_initial', |
| 77 }, { | 81 }, { |
| (...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 156 'ProgramDatabaseFile': '$(OutDir)\\initial\\chrome_dll.pdb ', | 160 'ProgramDatabaseFile': '$(OutDir)\\initial\\chrome_dll.pdb ', |
| 157 'OutputFile': '$(OutDir)\\initial\\chrome.dll', | 161 'OutputFile': '$(OutDir)\\initial\\chrome.dll', |
| 158 }], ['incremental_chrome_dll==1', { | 162 }], ['incremental_chrome_dll==1', { |
| 159 'OutputFile': '$(OutDir)\\initial\\chrome.dll', | 163 'OutputFile': '$(OutDir)\\initial\\chrome.dll', |
| 160 'UseLibraryDependencyInputs': "true", | 164 'UseLibraryDependencyInputs': "true", |
| 161 }], | 165 }], |
| 162 ], | 166 ], |
| 163 }, | 167 }, |
| 164 }, | 168 }, |
| 165 'conditions': [ | 169 'conditions': [ |
| 170 ['buildtype=="Official" and branding=="Chrome"', { | |
|
Mark Mentovai
2011/09/01 21:55:34
I am positive that this condition is correct for t
SteveT
2011/09/01 22:39:55
Yup - it's correct for both. I've actually already
| |
| 171 'copies': [ | |
| 172 { | |
| 173 # This location is for the win build. The chrome.release | |
|
Mark Mentovai
2011/09/01 21:55:34
win = Windows.
SteveT
2011/09/01 22:39:55
Done.
| |
| 174 # file ensures that these files are copied into the | |
| 175 # installer. | |
| 176 'destination': '<(PRODUCT_DIR)/default_apps', | |
| 177 'files': [ | |
| 178 '<@(default_apps_files)', | |
| 179 ] | |
| 180 }, | |
| 181 ], | |
| 182 }], | |
| 166 ['incremental_chrome_dll==1 and optimize_with_syzygy==0', { | 183 ['incremental_chrome_dll==1 and optimize_with_syzygy==0', { |
| 167 # Linking to a different directory and then hardlinking back | 184 # Linking to a different directory and then hardlinking back |
| 168 # to OutDir is a workaround to avoid having the .ilk for | 185 # to OutDir is a workaround to avoid having the .ilk for |
| 169 # chrome.exe and chrome.dll conflicting. See crbug.com/92528 | 186 # chrome.exe and chrome.dll conflicting. See crbug.com/92528 |
| 170 # for more information. Done on the dll instead of the exe so | 187 # for more information. Done on the dll instead of the exe so |
| 171 # that people launching from VS don't need to modify | 188 # that people launching from VS don't need to modify |
| 172 # $(TargetPath) for the exe. | 189 # $(TargetPath) for the exe. |
| 173 'msvs_postbuild': 'tools\\build\\win\\hardlink_failsafe.bat $( OutDir)\\initial\\chrome.dll $(OutDir)\\chrome.dll' | 190 'msvs_postbuild': 'tools\\build\\win\\hardlink_failsafe.bat $( OutDir)\\initial\\chrome.dll $(OutDir)\\chrome.dll' |
| 174 }] | 191 }] |
| 175 ] | 192 ] |
| (...skipping 407 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 583 'conditions': [ | 600 'conditions': [ |
| 584 ['debug_devtools!=0', { | 601 ['debug_devtools!=0', { |
| 585 'files': [ | 602 'files': [ |
| 586 '<(PRODUCT_DIR)/resources/inspector', | 603 '<(PRODUCT_DIR)/resources/inspector', |
| 587 ], | 604 ], |
| 588 }], | 605 }], |
| 589 ], | 606 ], |
| 590 }, | 607 }, |
| 591 ], | 608 ], |
| 592 'conditions': [ | 609 'conditions': [ |
| 610 ['buildtype=="Official" and branding=="Chrome"', { | |
| 611 'copies': [ | |
| 612 { | |
| 613 # This location is for the mac build. | |
|
Mark Mentovai
2011/09/01 21:55:34
Mac has a capital M.
SteveT
2011/09/01 22:39:55
Done.
| |
| 614 'destination': | |
| 615 '<(PRODUCT_DIR)/$(CONTENTS_FOLDER_PATH)/Default Apps', | |
| 616 'files': [ | |
| 617 '<@(default_apps_files)', | |
| 618 ] | |
| 619 }, | |
| 620 ], | |
| 621 }], | |
| 593 ['mac_breakpad==1', { | 622 ['mac_breakpad==1', { |
| 594 'variables': { | 623 'variables': { |
| 595 # A real .dSYM is needed for dump_syms to operate on. | 624 # A real .dSYM is needed for dump_syms to operate on. |
| 596 'mac_real_dsym': 1, | 625 'mac_real_dsym': 1, |
| 597 }, | 626 }, |
| 598 'sources': [ | 627 'sources': [ |
| 599 'app/breakpad_mac.mm', | 628 'app/breakpad_mac.mm', |
| 600 'app/breakpad_mac.h', | 629 'app/breakpad_mac.h', |
| 601 ], | 630 ], |
| 602 'dependencies': [ | 631 'dependencies': [ |
| (...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 750 'LinkIncremental': '<(msvs_debug_link_nonincremental)', | 779 'LinkIncremental': '<(msvs_debug_link_nonincremental)', |
| 751 }, | 780 }, |
| 752 }, | 781 }, |
| 753 }, | 782 }, |
| 754 }, | 783 }, |
| 755 }, # target chrome_dll | 784 }, # target chrome_dll |
| 756 ], | 785 ], |
| 757 }], | 786 }], |
| 758 ], | 787 ], |
| 759 } | 788 } |
| OLD | NEW |