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

Unified Diff: BUILD.gn

Issue 2247953005: Fix fuzzer build on Mac for ICU (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu.git@master
Patch Set: Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index 3032b5079a5c8a3421ef7151de08300e5303f7ea..dec85130bcec50299e47be498ff551e6144d3346 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -8,6 +8,10 @@ if (is_android) {
import("//build/config/android/rules.gni")
}
+if (is_mac) {
+ import("build/config/sanitizers/sanitizers.gni")
+}
+
# Meta target that includes both icuuc and icui18n. Most targets want both.
# You can depend on the individually if you need to.
group("icu") {
@@ -954,6 +958,10 @@ component("icuuc") {
":icudata",
]
+ if (is_mac && (is_asan || is_lsan || is_msan || is_tsan)) {
jungshik at Google 2016/08/16 20:24:43 I guess this is not a right condition to check?
+ deps += [ "//buildtools/third_party/libc++abi" ]
+ }
+
configs -= [
"//build/config/compiler:no_rtti", # ICU uses RTTI.
"//build/config/compiler:chromium_code",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698