Fix rapidjson build for gcc8 another way 07/25707/2
authorMats Wichmann <mats@linux.com>
Thu, 7 Jun 2018 15:00:15 +0000 (09:00 -0600)
committerMats Wichmann <mats@linux.com>
Sat, 9 Jun 2018 16:11:13 +0000 (16:11 +0000)
Uplifiting rapidjson version apparently not sufficient,
so do a temporary fix to disable Werror.

Also changed link to point to official github location.

Change-Id: I762636f3185c3d75b28aad237da69ba9cf0c6884
Signed-off-by: Mats Wichmann <mats@linux.com>
extlibs/rapidjson/SConscript

index bcad3db..446815c 100644 (file)
@@ -15,6 +15,13 @@ target_arch = env.get('TARGET_ARCH')
 
 host_os = sys.platform
 
+# Previous fix to uplift rapidjson insufficient for gcc8
+# So remove -Werror build flag when building rapidjson
+# TODO: remove this when there is a release containing a fix to
+# https://github.com/Tencent/rapidjson/issues/1246 and
+# https://github.com/Tencent/rapidjson/issues/1205
+while '-Werror' in rapidjson_env['CCFLAGS']: rapidjson_env['CCFLAGS'].remove('-Werror')
+
 ######################################################################
 # Build flags
 ######################################################################
@@ -26,6 +33,6 @@ if not os.path.exists(path):
     rapidjson_env = Environment(ENV=os.environ)
     rapidjson_zip = env.Download(
         'v1.1.0.zip',
-        'https://github.com/miloyip/rapidjson/archive/v1.1.0.zip')
+        'https://github.com/Tencent/rapidjson/archive/v1.1.0.zip')
     rapidjson_dir = env.UnpackAll('rapidjson', rapidjson_zip)
     os.rename("rapidjson-1.1.0", "rapidjson")