[Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sat Feb 10 17:45:52 UTC 2018
Hello Alex,
On Sat, 10 Feb 2018 17:19:16 +0100, Alex Baldwin wrote:
> I am not sure the best place to address this so please correct me :) I
> noticed that this patch failed with your submission as well as Matt
> Webber's, but for different reasons.
>
> It seems that on yours
> http://autobuild.buildroot.net/results/71a6d7e02a50f6cb53c70c38e82e2a2ae8af13ea/build-end.log
> there is a warning about having to redirect <sys/poll.h> to <poll.h> in the
> file "server.c:54". This could be sorted with a simple patch, is it
> acceptable to patch for this?
Yes, it is acceptable. In addition to fixing this particular problem,
it would be nice to avoid using -Werror. Indeed, the problem here is
that this warning is treated as an error, causing a build failure.
While -Werror makes sense during development, it doesn't make much
sense for releases.
> On Matt's
> http://autobuild.buildroot.net/results/c8b/c8bed3a3fa7d2b2258f573cbfcb01af07419e0bf/build-end.log
> we fail because undefined reference to __atomic_fetch_add_4 . I think a
> possible fix could be manually linking to libatomic. I am not entirely sure
> why, seems that some compilers are happy to find libatomic on their own and
> others aren't. Is this something that buildroot would normally patch for?
The libatomic situation depends on the architecture. On most
architectures, __atomic_fetch_add_4 is a compiler builtin, so you don't
need to link with libatomic.
However, on SPARC (which is the CPU architecture on which this build
failure occurs), libatomic is needed for __atomic_fetch_add_4.
You have two solutions to handle that:
(1) Just pass LIBS="-latomic" in LIBLO_CONF_ENV when
BR2_TOOLCHAIN_HAS_LIBATOMIC=y
(2) Patch the configure.ac script so that it tests if linking with
libatomic is needed or not, and links with it if needed.
In any case, if you do a patch against liblo, don't forget to submit it
upstream.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com
More information about the buildroot
mailing list