Skip to content

Commit 3919edb

Browse files
bnoordhuisMyles Borins
authored and
Myles Borins
committed
build: don't build icu with -fno-rtti
ICU should be compiled with -frtti (and it sets that flag in its gyp file) but it was also inheriting the -fno-rtti flag from common.gypi, breaking the build on some systems. Fixes: #8867 PR-URL: #8886 Reviewed-By: Johan Bergström <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]>
1 parent a3a184d commit 3919edb

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tools/icu/icu-generic.gyp

+1
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@
6969
[ 'os_posix == 1 and OS != "mac" and OS != "ios"', {
7070
'cflags': [ '-Wno-deprecated-declarations' ],
7171
'cflags_cc': [ '-frtti' ],
72+
'cflags_cc!': [ '-fno-rtti' ],
7273
}],
7374
[ 'OS == "mac" or OS == "ios"', {
7475
'xcode_settings': {'GCC_ENABLE_CPP_RTTI': 'YES' },

0 commit comments

Comments
 (0)