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

Side by Side Diff: chromecast/crash/BUILD.gn

Issue 2288953002: [Chromecast] Only include all libstdc++/libgcc symbols in executable as needed (Closed)
Patch Set: Created 4 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
« build/config/chromecast/BUILD.gn ('K') | « chromecast/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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 import("//chromecast/chromecast.gni") 5 import("//chromecast/chromecast.gni")
6 import("//testing/test.gni") 6 import("//testing/test.gni")
7 7
8 source_set("crash") { 8 source_set("crash") {
9 sources = [ 9 sources = [
10 "app_state_tracker.cc", 10 "app_state_tracker.cc",
(...skipping 25 matching lines...) Expand all
36 "//chromecast/base", 36 "//chromecast/base",
37 "//chromecast/base:cast_version", 37 "//chromecast/base:cast_version",
38 "//components/metrics", 38 "//components/metrics",
39 "//components/prefs", 39 "//components/prefs",
40 ] 40 ]
41 41
42 if (chromecast_branding == "public") { 42 if (chromecast_branding == "public") {
43 deps += [ "//chromecast/base:cast_sys_info" ] 43 deps += [ "//chromecast/base:cast_sys_info" ]
44 } else { 44 } else {
45 deps += [ "//chromecast/base:cast_sys_info_shlib" ] 45 deps += [ "//chromecast/base:cast_sys_info_shlib" ]
46
47 # whole archiving libgcc.a and libstdc++ is needed if dynamic share libs
48 # are used, see b/25566835
49 configs += [ "//build/config/chromecast:executable_whole_archive_config" ]
bcf 2016/08/29 21:22:51 How about put this under executable("crash_upload
antz1 2016/08/29 23:09:33 Done.
46 } 50 }
47 } 51 }
48 52
49 executable("crash_uploader") { 53 executable("crash_uploader") {
50 sources = [ 54 sources = [
51 "linux/crash_uploader.cc", 55 "linux/crash_uploader.cc",
52 ] 56 ]
53 57
54 deps = [ 58 deps = [
55 ":crash", 59 ":crash",
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 "//breakpad:client", 102 "//breakpad:client",
99 "//chromecast/base:cast_sys_info", 103 "//chromecast/base:cast_sys_info",
100 "//chromecast/base:test_support", 104 "//chromecast/base:test_support",
101 "//components/metrics", 105 "//components/metrics",
102 "//components/prefs:test_support", 106 "//components/prefs:test_support",
103 "//testing/gmock", 107 "//testing/gmock",
104 "//testing/gtest", 108 "//testing/gtest",
105 ] 109 ]
106 } 110 }
107 } 111 }
OLDNEW
« build/config/chromecast/BUILD.gn ('K') | « chromecast/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698