Message ID | 1478441720-7653-1-git-send-email-romain.naour@gmail.com |
---|---|
State | Changes Requested |
Headers | show |
On 06-11-16 15:15, 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> > --- > package/qemu/Config.in | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/package/qemu/Config.in b/package/qemu/Config.in > index ea6b946..138d96a 100644 > --- a/package/qemu/Config.in > +++ b/package/qemu/Config.in > @@ -69,6 +69,8 @@ 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. > Shouldn't a comment be added? Regards, Arnout >
Hi Arnout, Le 06/11/2016 à 16:43, Arnout Vandecappelle a écrit : > > > On 06-11-16 15:15, 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> >> --- >> package/qemu/Config.in | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/package/qemu/Config.in b/package/qemu/Config.in >> index ea6b946..138d96a 100644 >> --- a/package/qemu/Config.in >> +++ b/package/qemu/Config.in >> @@ -69,6 +69,8 @@ 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. >> > > Shouldn't a comment be added? Yes you're right. Fixed in v2. Best regards, Romain > > Regards, > Arnout > >> >
diff --git a/package/qemu/Config.in b/package/qemu/Config.in index ea6b946..138d96a 100644 --- a/package/qemu/Config.in +++ b/package/qemu/Config.in @@ -69,6 +69,8 @@ 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.
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> --- package/qemu/Config.in | 2 ++ 1 file changed, 2 insertions(+)