Skip to contentSkip to conversation

log4cplus: add C++11 dependencies

PATCH New Fabrice Fontaine 2 participants 2 messages May 9, 2018 – May 11, 2018

lore
Share & tools
lore

Fabrice Fontaine

author
#0 lore raw

PATCH 1/1 log4cplus: add C++11 dependencies

log4cplus uses std::future which is a C++11 feature affected by the gcc bug 64735

Fixes:
- http://autobuild.buildroot.net/results/4f4e9ed5dd5bad76eeee5d151e958f22603615b3

Signed-off-by Fabrice Fontaine <[email protected]>
---
 package/log4cplus/Config.in | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)
package/log4cplus/Config.in +8 -2
index 8a112a4e8f..fd224dd270 100644
@@ -3,6 +3,8 @@ config BR2_PACKAGE_LOG4CPLUS
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_USE_WCHAR
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_TOOLCHAIN_GCC_AT_LEAST_4_7 # C++11
+	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_64735 # std::future
 	help
 	  log4cplus is a simple to use C++ logging API providing
 	  thread-safe, flexible, and arbitrarily granular control

@@ -11,5 +13,9 @@ config BR2_PACKAGE_LOG4CPLUS
 
 	  http://sourceforge.net/projects/log4cplus
 
-comment "log4cplus needs a toolchain w/ C++, wchar, threads"
-	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS)
+comment "log4cplus needs a toolchain w/ C++, wchar, threads, gcc >= 4.7"
+	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
+		!BR2_TOOLCHAIN_HAS_THREADS || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_7
+
+comment "log4cplus needs a toolchain not affected by GCC bug 64735"
+	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_64735
Signature · 1 line
2.14.1
Applied
#1 lore raw

PATCH 1/1 log4cplus: add C++11 dependencies

Hello,

Quoted 10 lines · Fabrice Fontaine

log4cplus uses std::future which is a C++11 feature affected by the gcc bug 64735

Fixes:
- http://autobuild.buildroot.net/results/4f4e9ed5dd5bad76eeee5d151e958f22603615b3

Signed-off-by Fabrice Fontaine <[email protected]>
---
 package/log4cplus/Config.in | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com