[Buildroot] [PATCH v2] package/qemu: disable Qemu user-land emulation for musl toolchains

[Buildroot] [PATCH v2] package/qemu: disable Qemu user-land emulation for musl toolchains

[Romain Naour]

Like for cyclictest (rt-test)[1], linux-user/syscall.c use the
definition of the "struct sigevent" from Glibc/uClibc which doesn't
build with musl libc.

So, disable Qemu user-land emulation for musl toolchains.

Fixes:
http://autobuild.buildroot.net/results/b27/b277e668d9e22b624416bf46f6ccba802acd257e

[1] 43bffa59e7359610c8f00038ddd97795c0ca35e1

Signed-off-by: Romain Naour <romain.naour@gmail.com>
Cc: Francois Perrad <francois.perrad@gadz.org>
---
v2: add a comment for BR2_TOOLCHAIN_USES_MUSL case (Arnout)
---
 package/qemu/Config.in | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/package/qemu/Config.in b/package/qemu/Config.in
index ea6b946..8bae5f4 100644
--- a/package/qemu/Config.in
+++ b/package/qemu/Config.in
@@ -69,11 +69,16 @@ comment "systems emulation needs a toolchain w/ dynamic library"
 
 config BR2_PACKAGE_QEMU_LINUX_USER
 	bool "Enable all Linux user-land emulation"
+	# Incompatible "struct sigevent" definition on musl
+	depends on !BR2_TOOLCHAIN_USES_MUSL
 	help
 	  Say 'y' to build all Linux user-land emulators that QEMU supports.
 
 # Note: bsd-user can not be build on Linux
 
+comment "Linux user-land emulation needs a glibc or uClibc toolchain"
+	depends on BR2_TOOLCHAIN_USES_MUSL
+
 endif # BR2_PACKAGE_QEMU_CUSTOM_TARGETS == ""
 
 config BR2_PACKAGE_QEMU_HAS_EMULS
-- 
2.5.5

[Buildroot] [PATCH v2] package/qemu: disable Qemu user-land emulation for musl toolchains

[Thomas Petazzoni]

Hello,

On Sun,  6 Nov 2016 17:04:48 +0100, Romain Naour wrote:
> Like for cyclictest (rt-test)[1], linux-user/syscall.c use the
> definition of the "struct sigevent" from Glibc/uClibc which doesn't
> build with musl libc.
> 
> So, disable Qemu user-land emulation for musl toolchains.
> 
> Fixes:
> http://autobuild.buildroot.net/results/b27/b277e668d9e22b624416bf46f6ccba802acd257e
> 
> [1] 43bffa59e7359610c8f00038ddd97795c0ca35e1
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> Cc: Francois Perrad <francois.perrad@gadz.org>
> ---
> v2: add a comment for BR2_TOOLCHAIN_USES_MUSL case (Arnout)
> ---
>  package/qemu/Config.in | 5 +++++
>  1 file changed, 5 insertions(+)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com