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

Side by Side Diff: chrome/chrome_dll.gypi

Issue 7827021: Changed mac and win Official Chrome build to drop default app CRXs in the build/installer output. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 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 | chrome/installer/mini_installer/chrome.release » ('j') | 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 '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
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
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
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
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 }
OLDNEW
« no previous file with comments | « no previous file | chrome/installer/mini_installer/chrome.release » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698