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

Side by Side Diff: chrome/chrome.gyp

Issue 1881053002: Rework build/mac/tweak_info_plist.py to not rely on environment variables. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comment Created 4 years, 8 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
« no previous file with comments | « chrome/app_shim/app_shim.gypi ('k') | chrome/chrome_dll_bundle.gypi » ('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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 'variables': { 5 'variables': {
6 'chromium_code': 1, 6 'chromium_code': 1,
7 7
8 # Define the common dependencies that contain all the actual 8 # Define the common dependencies that contain all the actual
9 # Chromium functionality. This list gets pulled in below by 9 # Chromium functionality. This list gets pulled in below by
10 # the link of the actual chrome (or chromium) executable on 10 # the link of the actual chrome (or chromium) executable on
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 ], 221 ],
222 }, 222 },
223 { 223 {
224 # Modify the Info.plist as needed. The script explains why this 224 # Modify the Info.plist as needed. The script explains why this
225 # is needed. This is also done in the chrome and chrome_dll 225 # is needed. This is also done in the chrome and chrome_dll
226 # targets. In this case, --breakpad=0, --keystone=0, and --scm=0 226 # targets. In this case, --breakpad=0, --keystone=0, and --scm=0
227 # are used because Breakpad, Keystone, and SCM keys are 227 # are used because Breakpad, Keystone, and SCM keys are
228 # never placed into the helper. 228 # never placed into the helper.
229 'postbuild_name': 'Tweak Info.plist', 229 'postbuild_name': 'Tweak Info.plist',
230 'action': ['<(tweak_info_plist_path)', 230 'action': ['<(tweak_info_plist_path)',
231 '--plist=${TARGET_BUILD_DIR}/${INFOPLIST_PATH}',
231 '--breakpad=0', 232 '--breakpad=0',
232 '--keystone=0', 233 '--keystone=0',
233 '--scm=0'], 234 '--scm=0'],
234 }, 235 },
235 ], 236 ],
236 'conditions': [ 237 'conditions': [
237 ['mac_breakpad==1', { 238 ['mac_breakpad==1', {
238 'variables': { 239 'variables': {
239 # A real .dSYM is needed for dump_syms to operate on. 240 # A real .dSYM is needed for dump_syms to operate on.
240 'mac_real_dsym': 1, 241 'mac_real_dsym': 1,
(...skipping 570 matching lines...) Expand 10 before | Expand all | Expand 10 after
811 # expired. crbug.com/577698 812 # expired. crbug.com/577698
812 '../components/components.gyp:startup_metric_utils_common', 813 '../components/components.gyp:startup_metric_utils_common',
813 ], 814 ],
814 }], 815 }],
815 ], 816 ],
816 }, 817 },
817 ], 818 ],
818 }], 819 }],
819 ], # 'conditions' 820 ], # 'conditions'
820 } 821 }
OLDNEW
« no previous file with comments | « chrome/app_shim/app_shim.gypi ('k') | chrome/chrome_dll_bundle.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698