xapp_xload: fix musl build
PATCH 1/1 xapp_xload: fix musl build
xapp_xload tries to include protocols/rwhod.h, which isn't available on musl. Fortunately, the package also has stub code that can be enabled with -DRLOADSTUB (this is done e.g. on Cygwin).
This should probably be detected in configure, but that's a lot more work.
--- package/x11r7/xapp_xload/xapp_xload.mk | 5 +++++ 1 file changed, 5 insertions(+)
package/x11r7/xapp_xload/xapp_xload.mk +5 -0
@@ -11,4 +11,9 @@ XAPP_XLOAD_LICENSE = MIT XAPP_XLOAD_LICENSE_FILES = COPYING XAPP_XLOAD_DEPENDENCIES = xlib_libXaw +ifeq ($(BR2_TOOLCHAIN_USES_MUSL),y) +# musl doesn't have rwhod.h, but xload can replace it with stubs +XAPP_XLOAD_CONF_OPTS += CFLAGS="$(TARGET_CFLAGS) -DRLOADSTUB" +endif + $(eval $(autotools-package))
Signature · 1 line
2.10.2
Thomas Petazzoni
AppliedPATCH 1/1 xapp_xload: fix musl build
Hello,
Quoted 11 lines · Arnout Vandecappelle (Essensium/Mind)
xapp_xloadtries to includeprotocols/rwhod.h, which isn't available on musl. Fortunately, the package also has stub code that can be enabled with -DRLOADSTUB (this is done e.g. on Cygwin).This should probably be detected in configure, but that's a lot more work.
--- package/x11r7/xapp_xload/xapp_xload.mk | 5 +++++ 1 file changed, 5 insertions(+)
You forgot a reference to the autobuilder failure, and I forgot as well before pushing :/
Applied to master anyway, thanks!
Thomas
Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com