Add -frtti to HarfBuzz CXXFLAGS instead of paching configure.ac

Change-Id: Ibc16c4e9c4a305d6fff764fcf3964a63c5322e14
Reviewed-on: https://gerrit.libreoffice.org/68921
Tested-by: Jenkins
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>

external/harfbuzz/ExternalProject_harfbuzz.mk

@@ -46,6 +46,7 @@ $(call gb_ExternalProject_get_state_target,harfbuzz,build) :
 				$(if $(filter $(true),$(gb_SYMBOL)),$(gb_DEBUGINFO_FLAGS)) \
 				$(if $(ENABLE_OPTIMIZED), \
 					$(gb_COMPILEROPTFLAGS),$(gb_COMPILERNOOPTFLAGS)) \
+				$(if $(ENABLE_RUNTIME_OPTIMIZATIONS),,-frtti) \
 				$(CXXFLAGS) $(CXXFLAGS_CXX11) \
 				$(ICU_UCHAR_TYPE) \
 				$(if $(filter LINUX,$(OS)),-fvisibility=hidden)' \

external/harfbuzz/UnpackedTarball_harfbuzz.mk

@@ -21,10 +21,4 @@ $(eval $(call gb_UnpackedTarball_add_patches,harfbuzz, \
     external/harfbuzz/msvc.patch \
 ))
 
-ifneq ($(ENABLE_RUNTIME_OPTIMIZATIONS),TRUE)
-$(eval $(call gb_UnpackedTarball_add_patches,harfbuzz, \
-    external/harfbuzz/harfbuzz-rtti.patch \
-))
-endif
-
 # vim: set noet sw=4 ts=4:

external/harfbuzz/harfbuzz-rtti.patch

---- configure
-+++ configure
-@@ -18959,9 +18959,7 @@
- 	# No threadsafe statics in C++ as we do it ourselves.
- 	# We don't use these features, so it's safe to disable them
- 	# even in the cases where we DO link to libstdc++.
--	# Put -fno-rtti before $CXXFLAGS such that users can re-enable it
--	# by overriding CXXFLAGS.
--	CXXFLAGS="-fno-rtti $CXXFLAGS -fno-exceptions -fno-threadsafe-statics"
-+	CXXFLAGS="$CXXFLAGS -fno-exceptions -fno-threadsafe-statics"
- 
- 	case "$host" in
- 		*-*-mingw*)