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

Side by Side Diff: chrome/chrome.gyp

Issue 1187233002: Revert of Fix some build symbol configuration. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
353 'target_name': 'linux_symbols', 352 'target_name': 'linux_symbols',
354 'type': 'none', 353 'type': 'none',
355 'conditions': [ 354 'conditions': [
356 ['linux_dump_symbols==1', { 355 ['linux_dump_symbols==1', {
357 'actions': [ 356 'actions': [
358 { 357 {
359 'action_name': 'dump_symbols', 358 'action_name': 'dump_symbols',
360 'inputs': [ 359 'inputs': [
361 '<(DEPTH)/build/linux/dump_app_syms.py', 360 '<(DEPTH)/build/linux/dump_app_syms',
362 '<(PRODUCT_DIR)/dump_syms', 361 '<(PRODUCT_DIR)/dump_syms',
363 '<(PRODUCT_DIR)/chrome', 362 '<(PRODUCT_DIR)/chrome',
364 ], 363 ],
365 'outputs': [ 364 'outputs': [
366 '<(PRODUCT_DIR)/chrome.breakpad.<(target_arch)', 365 '<(PRODUCT_DIR)/chrome.breakpad.<(target_arch)',
367 ], 366 ],
368 'action': ['python', 367 'action': ['<(DEPTH)/build/linux/dump_app_syms',
369 '<(DEPTH)/build/linux/dump_app_syms.py',
370 '<(PRODUCT_DIR)/dump_syms', 368 '<(PRODUCT_DIR)/dump_syms',
371 '<(linux_strip_binary)', 369 '<(linux_strip_binary)',
372 '<(PRODUCT_DIR)/chrome', 370 '<(PRODUCT_DIR)/chrome',
373 '<@(_outputs)'], 371 '<@(_outputs)'],
374 'message': 'Dumping breakpad symbols to <(_outputs)', 372 'message': 'Dumping breakpad symbols to <(_outputs)',
375 'process_outputs_as_sources': 1, 373 'process_outputs_as_sources': 1,
376 }, 374 },
377 ], 375 ],
378 'dependencies': [ 376 'dependencies': [
379 'chrome', 377 'chrome',
(...skipping 416 matching lines...) Expand 10 before | Expand all | Expand 10 after
796 '<(DEPTH)/third_party/kasko' 794 '<(DEPTH)/third_party/kasko'
797 ], 795 ],
798 }, 796 },
799 }, 797 },
800 }, 798 },
801 }, 799 },
802 ], 800 ],
803 }], 801 }],
804 ], # 'conditions' 802 ], # 'conditions'
805 } 803 }
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