From: Thomas Petazzoni Date: Tue, 2 Feb 2016 15:31:33 +0000 (+0100) Subject: glog: fix atomic built-in problem X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8df95843ed1090b7be41b541bbcf111b86cb4c4d;p=buildroot.git glog: fix atomic built-in problem glog can optionally use atomic __sync built-ins. However, its configure script only checks for the availability of the 4-byte variant, but the code also uses the 1-byte variant. While this works on most architectures, it does not on architectures that implement only the 4-byte variant, such as Microblaze. So if the architecture does not implement the 1-byte variant, we hint the configure script that atomic built-ins should not be used. Fixes: http://autobuild.buildroot.org/results/bedebd594c283adf39374c9a9f44432e5d6cbff7/ Signed-off-by: Thomas Petazzoni Reviewed-by: "Yann E. MORIN" --- diff --git a/package/glog/glog.mk b/package/glog/glog.mk index 8a8409487a..aa7cacbdd3 100644 --- a/package/glog/glog.mk +++ b/package/glog/glog.mk @@ -14,4 +14,15 @@ ifeq ($(BR2_PACKAGE_GFLAGS),y) GLOG_DEPENDENCIES = gflags endif +# glog can optionally use atomic __sync built-ins. However, its +# configure script only checks for the availability of the 4 bytes +# version, but the code also uses the 1 byte version. While this works +# on most architectures, it does not on architectures that implement +# only the 4 bytes version, such as Microblaze. So if the architecture +# does not implement the 1 byte version, we hint the configure script +# that atomic built-ins should not be used. +ifeq ($(BR2_TOOLCHAIN_HAS_SYNC_1),) +GLOG_CONF_ENV += ac_cv___sync_val_compare_and_swap=no +endif + $(eval $(autotools-package))