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

Side by Side Diff: tools/gn/secondary/third_party/icu/BUILD.gn

Issue 239543013: Work on Mac GN build (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix line endings Created 6 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 | Annotate | Revision Log
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 # Meta target that includes both icuuc and icui18n. Most targets want both. 5 # Meta target that includes both icuuc and icui18n. Most targets want both.
6 # You can depend on the individually if you need to. 6 # You can depend on the individually if you need to.
7 group("icu") { 7 group("icu") {
8 deps = [ 8 deps = [
9 ":icui18n", 9 ":icui18n",
10 ":icuuc", 10 ":icuuc",
(...skipping 205 matching lines...) Expand 10 before | Expand all | Expand 10 after
216 "//build/config/compiler:no_rtti", # ICU uses RTTI. 216 "//build/config/compiler:no_rtti", # ICU uses RTTI.
217 "//build/config/compiler:chromium_code", 217 "//build/config/compiler:chromium_code",
218 ] 218 ]
219 configs += [ 219 configs += [
220 "//build/config/compiler:rtti", 220 "//build/config/compiler:rtti",
221 "//build/config/compiler:no_chromium_code", 221 "//build/config/compiler:no_chromium_code",
222 ] 222 ]
223 223
224 configs += [ ":icu_code" ] 224 configs += [ ":icu_code" ]
225 direct_dependent_configs = [ ":icu_config" ] 225 direct_dependent_configs = [ ":icu_config" ]
226
227 if (is_clang) {
228 # uspoof.h has a U_NAMESPACE_USE macro. That's a bug,
229 # the header should use U_NAMESPACE_BEGIN instead.
230 # http://bugs.icu-project.org/trac/ticket/9054
231 configs -= [ "//build/config/clang:extra_warnings" ]
232
233 cflags = [
234 "-Wno-header-hygiene",
235 # Looks like a real issue, see http://crbug.com/114660
236 "-Wno-return-type-c-linkage",
237 ]
238 }
226 } 239 }
227 240
228 component("icuuc") { 241 component("icuuc") {
229 sources = [ 242 sources = [
230 "source/common/bmpset.cpp", 243 "source/common/bmpset.cpp",
231 "source/common/brkeng.cpp", 244 "source/common/brkeng.cpp",
232 "source/common/brkiter.cpp", 245 "source/common/brkiter.cpp",
233 "source/common/bytestream.cpp", 246 "source/common/bytestream.cpp",
234 "source/common/caniter.cpp", 247 "source/common/caniter.cpp",
235 "source/common/chariter.cpp", 248 "source/common/chariter.cpp",
(...skipping 185 matching lines...) Expand 10 before | Expand all | Expand 10 after
421 # the icudt46_dat symbol. 434 # the icudt46_dat symbol.
422 "android/icudt46l_dat.S", 435 "android/icudt46l_dat.S",
423 "linux/icudt46l_dat.S", 436 "linux/icudt46l_dat.S",
424 "mac/icudt46l_dat.S", 437 "mac/icudt46l_dat.S",
425 ] 438 ]
426 defines = [ "U_HIDE_DATA_SYMBOL" ] 439 defines = [ "U_HIDE_DATA_SYMBOL" ]
427 440
428 # TODO(brettw) support use_system_icu and icu_use_data_file_flag. 441 # TODO(brettw) support use_system_icu and icu_use_data_file_flag.
429 } 442 }
430 } 443 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698