diff mbox

[v2,1/1] package/exiv2: disable on musl

Message ID 20170605114703.12438-1-bernd.kuhls@t-online.de
State Accepted
Headers show

Commit Message

Bernd Kuhls June 5, 2017, 11:47 a.m. UTC
Musl build is broken:

/home/buildroot/build/instance-0/output/build/exiv2-910f3507795e1930ae216c9febee0bf9a88e99c0/src/actions.cpp:2052:32:
 error: 'PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP' was not declared in this scope

According to http://www.openwall.com/lists/musl/2017/02/20/3
the define is not available on musl.

Fixes
http://autobuild.buildroot.net/results/283/28334a8a0b23c7c8260023b9a3aa15e8dd7613d0/

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: added comment to Config.in (Baruch)

 package/exiv2/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni June 5, 2017, 12:53 p.m. UTC | #1
Hello,

On Mon,  5 Jun 2017 13:47:03 +0200, Bernd Kuhls wrote:
> Musl build is broken:
> 
> /home/buildroot/build/instance-0/output/build/exiv2-910f3507795e1930ae216c9febee0bf9a88e99c0/src/actions.cpp:2052:32:
>  error: 'PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP' was not declared in this scope
> 
> According to http://www.openwall.com/lists/musl/2017/02/20/3
> the define is not available on musl.
> 
> Fixes
> http://autobuild.buildroot.net/results/283/28334a8a0b23c7c8260023b9a3aa15e8dd7613d0/
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: added comment to Config.in (Baruch)

Applied to master, thanks.

Thomas
diff mbox

Patch

diff --git a/package/exiv2/Config.in b/package/exiv2/Config.in
index 7358283a6..acce81c59 100644
--- a/package/exiv2/Config.in
+++ b/package/exiv2/Config.in
@@ -1,12 +1,14 @@ 
-comment "exiv2 needs a toolchain w/ C++, wchar, dynamic library, threads"
+comment "exiv2 needs a uClibc or glibc toolchain w/ C++, wchar, dynamic library, threads"
 	depends on BR2_STATIC_LIBS || !BR2_INSTALL_LIBSTDCPP \
-		|| !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
+		|| !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS \
+		|| BR2_TOOLCHAIN_USES_MUSL
 
 config BR2_PACKAGE_EXIV2
 	bool "exiv2"
 	depends on !BR2_STATIC_LIBS
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_TOOLCHAIN_USES_MUSL # PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP
 	depends on BR2_USE_WCHAR
 	help
 	  Exiv2 is a C++ library and a command line utility to manage