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

Side by Side Diff: chrome/BUILD.gn

Issue 1657143002: [GN] Guard the "pangocairo" config with (use_pango || use_cairo). (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Guard another pango target Created 4 years, 10 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 | « build/config/linux/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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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("//build/config/chrome_build.gni") 5 import("//build/config/chrome_build.gni")
6 import("//build/config/compiler/compiler.gni") 6 import("//build/config/compiler/compiler.gni")
7 import("//build/config/features.gni") 7 import("//build/config/features.gni")
8 import("//build/config/locales.gni") 8 import("//build/config/locales.gni")
9 import("//build/config/sanitizers/sanitizers.gni") 9 import("//build/config/sanitizers/sanitizers.gni")
10 import("//build/config/ui.gni") 10 import("//build/config/ui.gni")
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
184 "//content/public/app:both", 184 "//content/public/app:both",
185 ] 185 ]
186 if (enable_plugins && enable_pdf) { 186 if (enable_plugins && enable_pdf) {
187 deps += [ "//pdf" ] 187 deps += [ "//pdf" ]
188 } 188 }
189 189
190 public_deps = [ 190 public_deps = [
191 ":xdg_mime", # Needs to be public for installer to consume files. 191 ":xdg_mime", # Needs to be public for installer to consume files.
192 ] 192 ]
193 193
194 # Needed for chrome_main.cc initialization of libraries.
195 configs += [ "//build/config/linux:pangocairo" ]
196
197 # GYP has this in a 'profiling==0 and linux_disable_pie==0' condition. 194 # GYP has this in a 'profiling==0 and linux_disable_pie==0' condition.
198 # but GN doesn't have either of these flags. 195 # but GN doesn't have either of these flags.
199 ldflags = [ "-pie" ] 196 ldflags = [ "-pie" ]
200 197
198 if (use_pango || use_cairo) {
199 # Needed for chrome_main.cc initialization of libraries.
200 configs += [ "//build/config/linux:pangocairo" ]
201 }
202
201 if (use_x11) { 203 if (use_x11) {
202 configs += [ 204 configs += [
203 "//build/config/linux:x11", 205 "//build/config/linux:x11",
204 "//build/config/linux:xext", 206 "//build/config/linux:xext",
205 ] 207 ]
206 } 208 }
207 } 209 }
208 210
209 if (is_mac) { 211 if (is_mac) {
210 sources += [ "app/chrome_exe_main_mac.c" ] 212 sources += [ "app/chrome_exe_main_mac.c" ]
(...skipping 850 matching lines...) Expand 10 before | Expand all | Expand 10 after
1061 "//chrome/app/theme/$branding_path_component/product_logo_48.png", 1063 "//chrome/app/theme/$branding_path_component/product_logo_48.png",
1062 "//chrome/tools/build/linux/chrome-wrapper", 1064 "//chrome/tools/build/linux/chrome-wrapper",
1063 "//third_party/xdg-utils/scripts/xdg-mime", 1065 "//third_party/xdg-utils/scripts/xdg-mime",
1064 "//third_party/xdg-utils/scripts/xdg-settings", 1066 "//third_party/xdg-utils/scripts/xdg-settings",
1065 ] 1067 ]
1066 outputs = [ 1068 outputs = [
1067 "$root_out_dir/{{source_file_part}}", 1069 "$root_out_dir/{{source_file_part}}",
1068 ] 1070 ]
1069 } 1071 }
1070 } 1072 }
OLDNEW
« no previous file with comments | « build/config/linux/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698