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

Side by Side Diff: chrome/chrome.gyp

Issue 1182663007: Fix some build symbol configuration. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Created 5 years, 6 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/BUILD.gn ('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) 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 331 matching lines...) Expand 10 before | Expand all | Expand 10 after
342 ], 342 ],
343 'sources': [ 343 'sources': [
344 'tools/mac_helpers/infoplist_strings_util.mm', 344 'tools/mac_helpers/infoplist_strings_util.mm',
345 ], 345 ],
346 }, 346 },
347 ], # targets 347 ], # targets
348 }], # OS=="mac" 348 }], # OS=="mac"
349 ['OS=="linux"', 349 ['OS=="linux"',
350 { 'targets': [ 350 { 'targets': [
351 { 351 {
352 # GN version: //chrome:linux_symbols
352 'target_name': 'linux_symbols', 353 'target_name': 'linux_symbols',
353 'type': 'none', 354 'type': 'none',
354 'conditions': [ 355 'conditions': [
355 ['linux_dump_symbols==1', { 356 ['linux_dump_symbols==1', {
356 'actions': [ 357 'actions': [
357 { 358 {
358 'action_name': 'dump_symbols', 359 'action_name': 'dump_symbols',
359 'inputs': [ 360 'inputs': [
360 '<(DEPTH)/build/linux/dump_app_syms', 361 '<(DEPTH)/build/linux/dump_app_syms.py',
361 '<(PRODUCT_DIR)/dump_syms', 362 '<(PRODUCT_DIR)/dump_syms',
362 '<(PRODUCT_DIR)/chrome', 363 '<(PRODUCT_DIR)/chrome',
363 ], 364 ],
364 'outputs': [ 365 'outputs': [
365 '<(PRODUCT_DIR)/chrome.breakpad.<(target_arch)', 366 '<(PRODUCT_DIR)/chrome.breakpad.<(target_arch)',
366 ], 367 ],
367 'action': ['<(DEPTH)/build/linux/dump_app_syms', 368 'action': ['python',
369 '<(DEPTH)/build/linux/dump_app_syms.py',
368 '<(PRODUCT_DIR)/dump_syms', 370 '<(PRODUCT_DIR)/dump_syms',
369 '<(linux_strip_binary)', 371 '<(linux_strip_binary)',
370 '<(PRODUCT_DIR)/chrome', 372 '<(PRODUCT_DIR)/chrome',
371 '<@(_outputs)'], 373 '<@(_outputs)'],
372 'message': 'Dumping breakpad symbols to <(_outputs)', 374 'message': 'Dumping breakpad symbols to <(_outputs)',
373 'process_outputs_as_sources': 1, 375 'process_outputs_as_sources': 1,
374 }, 376 },
375 ], 377 ],
376 'dependencies': [ 378 'dependencies': [
377 'chrome', 379 'chrome',
(...skipping 416 matching lines...) Expand 10 before | Expand all | Expand 10 after
794 '<(DEPTH)/third_party/kasko' 796 '<(DEPTH)/third_party/kasko'
795 ], 797 ],
796 }, 798 },
797 }, 799 },
798 }, 800 },
799 }, 801 },
800 ], 802 ],
801 }], 803 }],
802 ], # 'conditions' 804 ], # 'conditions'
803 } 805 }
OLDNEW
« no previous file with comments | « chrome/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698