* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 @ 2018-02-08 19:37 Alex B 2018-02-08 19:37 ` Alex B 2018-02-08 22:04 ` Thomas Petazzoni 0 siblings, 2 replies; 7+ messages in thread From: Alex B @ 2018-02-08 19:37 UTC (permalink / raw) To: buildroot From: Alex <alexbaldwinmusic@gmail.com> Starting from version 0.28, C++ wrappers are provided as well. The configure script appropriately detects the availability of a C++ compiler so no Buildroot changes are needed for this. Building fails with --enable-ipv6 so --disable-ipv6 is needed in CONF_OPTS. This is known by liblo developers and building without ipv6 support is recomennded. Included hash for license files. Signed-off-by: Alex Baldwin <alexbaldwinmusic@gmail.com> --- CHANGELOG No longer including a patch for liblo/configure. This was the wrong way of dealing with --enable-ipv6. SHA256 for COPYING file instead of sha1. Include # License files comment for clarity. --- package/liblo/liblo.hash | 10 +++++++--- package/liblo/liblo.mk | 3 ++- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/package/liblo/liblo.hash b/package/liblo/liblo.hash index a4e01a6..862b6fb 100644 --- a/package/liblo/liblo.hash +++ b/package/liblo/liblo.hash @@ -1,3 +1,7 @@ -# From http://sourceforge.net/projects/liblo/files/liblo/0.26/ -sha1 21942c8f19e9829b5842cb85352f98c49dfbc823 liblo-0.26.tar.gz -md5 5351de14262560e15e7f23865293b16f liblo-0.26.tar.gz +# From http://sourceforge.net/projects/liblo/files/liblo/0.29/ +sha1 6aa69456787d3d6ef915281b4a0f8f2c79548ce3 liblo-0.29.tar.gz +md5 b0e70bc0fb2254addf94adddf85cffd3 liblo-0.29.tar.gz + +# Locally computed +sha1 caeb68c46fa36651acf592771d09de7937926bb3 COPYING +md5 fbc093901857fcd118f065f900982c24 COPYING diff --git a/package/liblo/liblo.mk b/package/liblo/liblo.mk index 3c852be..856b437 100644 --- a/package/liblo/liblo.mk +++ b/package/liblo/liblo.mk @@ -4,11 +4,12 @@ # ################################################################################ -LIBLO_VERSION = 0.26 +LIBLO_VERSION = 0.29 LIBLO_SITE = http://downloads.sourceforge.net/project/liblo/liblo/$(LIBLO_VERSION) LIBLO_LICENSE = LGPL-2.1+ LIBLO_LICENSE_FILES = COPYING LIBLO_INSTALL_STAGING = YES +LIBLO_CONF_OPTS += --disable-ipv6 $(eval $(autotools-package)) -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-08 19:37 [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 Alex B @ 2018-02-08 19:37 ` Alex B 2018-02-08 22:04 ` Thomas Petazzoni 1 sibling, 0 replies; 7+ messages in thread From: Alex B @ 2018-02-08 19:37 UTC (permalink / raw) To: buildroot From: Alex <alexbaldwinmusic@gmail.com> Starting from version 0.28, C++ wrappers are provided as well. The configure script appropriately detects the availability of a C++ compiler so no Buildroot changes are needed for this. Building fails with --enable-ipv6 so --disable-ipv6 is needed in CONF_OPTS. This is known by liblo developers and building without ipv6 support is recomennded. Included hash for license files. Signed-off-by: Alex Baldwin <alexbaldwinmusic@gmail.com> --- CHANGELOG No longer including a patch for liblo/configure. This was the wrong way of dealing with --enable-ipv6. SHA256 for COPYING file instead of sha1. Include # License files comment for clarity. Signed-off-by: Alex <alexbaldwinmusic@gmail.com> --- package/liblo/liblo.hash | 10 +++++++--- package/liblo/liblo.mk | 3 ++- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/package/liblo/liblo.hash b/package/liblo/liblo.hash index a4e01a6..862b6fb 100644 --- a/package/liblo/liblo.hash +++ b/package/liblo/liblo.hash @@ -1,3 +1,7 @@ -# From http://sourceforge.net/projects/liblo/files/liblo/0.26/ -sha1 21942c8f19e9829b5842cb85352f98c49dfbc823 liblo-0.26.tar.gz -md5 5351de14262560e15e7f23865293b16f liblo-0.26.tar.gz +# From http://sourceforge.net/projects/liblo/files/liblo/0.29/ +sha1 6aa69456787d3d6ef915281b4a0f8f2c79548ce3 liblo-0.29.tar.gz +md5 b0e70bc0fb2254addf94adddf85cffd3 liblo-0.29.tar.gz + +# Locally computed +sha1 caeb68c46fa36651acf592771d09de7937926bb3 COPYING +md5 fbc093901857fcd118f065f900982c24 COPYING diff --git a/package/liblo/liblo.mk b/package/liblo/liblo.mk index 3c852be..856b437 100644 --- a/package/liblo/liblo.mk +++ b/package/liblo/liblo.mk @@ -4,11 +4,12 @@ # ################################################################################ -LIBLO_VERSION = 0.26 +LIBLO_VERSION = 0.29 LIBLO_SITE = http://downloads.sourceforge.net/project/liblo/liblo/$(LIBLO_VERSION) LIBLO_LICENSE = LGPL-2.1+ LIBLO_LICENSE_FILES = COPYING LIBLO_INSTALL_STAGING = YES +LIBLO_CONF_OPTS += --disable-ipv6 $(eval $(autotools-package)) -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-08 19:37 [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 Alex B 2018-02-08 19:37 ` Alex B @ 2018-02-08 22:04 ` Thomas Petazzoni 2018-02-10 16:19 ` Alex Baldwin 1 sibling, 1 reply; 7+ messages in thread From: Thomas Petazzoni @ 2018-02-08 22:04 UTC (permalink / raw) To: buildroot Hello, On Thu, 8 Feb 2018 20:37:48 +0100, Alex B wrote: > diff --git a/package/liblo/liblo.hash b/package/liblo/liblo.hash > index a4e01a6..862b6fb 100644 > --- a/package/liblo/liblo.hash > +++ b/package/liblo/liblo.hash > @@ -1,3 +1,7 @@ > -# From http://sourceforge.net/projects/liblo/files/liblo/0.26/ > -sha1 21942c8f19e9829b5842cb85352f98c49dfbc823 liblo-0.26.tar.gz > -md5 5351de14262560e15e7f23865293b16f liblo-0.26.tar.gz > +# From http://sourceforge.net/projects/liblo/files/liblo/0.29/ > +sha1 6aa69456787d3d6ef915281b4a0f8f2c79548ce3 liblo-0.29.tar.gz > +md5 b0e70bc0fb2254addf94adddf85cffd3 liblo-0.29.tar.gz > + > +# Locally computed > +sha1 caeb68c46fa36651acf592771d09de7937926bb3 COPYING > +md5 fbc093901857fcd118f065f900982c24 COPYING A single sha256 hash was sufficient here. > diff --git a/package/liblo/liblo.mk b/package/liblo/liblo.mk > index 3c852be..856b437 100644 > --- a/package/liblo/liblo.mk > +++ b/package/liblo/liblo.mk > @@ -4,11 +4,12 @@ > # > ################################################################################ > > -LIBLO_VERSION = 0.26 > +LIBLO_VERSION = 0.29 > LIBLO_SITE = http://downloads.sourceforge.net/project/liblo/liblo/$(LIBLO_VERSION) > > LIBLO_LICENSE = LGPL-2.1+ > LIBLO_LICENSE_FILES = COPYING > LIBLO_INSTALL_STAGING = YES > +LIBLO_CONF_OPTS += --disable-ipv6 A comment above this line (in addition to the commit log) would have been nice. I've fixed both issues, and applied your patch to next. Thanks! Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-08 22:04 ` Thomas Petazzoni @ 2018-02-10 16:19 ` Alex Baldwin 2018-02-10 17:45 ` Thomas Petazzoni 0 siblings, 1 reply; 7+ messages in thread From: Alex Baldwin @ 2018-02-10 16:19 UTC (permalink / raw) To: buildroot Hi Thomas, 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? 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? Thanks, Alex On 8 February 2018 at 23:04, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > > Hello, > > On Thu, 8 Feb 2018 20:37:48 +0100, Alex B wrote: > > > diff --git a/package/liblo/liblo.hash b/package/liblo/liblo.hash > > index a4e01a6..862b6fb 100644 > > --- a/package/liblo/liblo.hash > > +++ b/package/liblo/liblo.hash > > @@ -1,3 +1,7 @@ > > -# From http://sourceforge.net/projects/liblo/files/liblo/0.26/ > > -sha1 21942c8f19e9829b5842cb85352f98c49dfbc823 liblo-0.26.tar.gz > > -md5 5351de14262560e15e7f23865293b16f liblo-0.26.tar.gz > > +# From http://sourceforge.net/projects/liblo/files/liblo/0.29/ > > +sha1 6aa69456787d3d6ef915281b4a0f8f2c79548ce3 liblo-0.29.tar.gz > > +md5 b0e70bc0fb2254addf94adddf85cffd3 liblo-0.29.tar.gz > > + > > +# Locally computed > > +sha1 caeb68c46fa36651acf592771d09de7937926bb3 COPYING > > +md5 fbc093901857fcd118f065f900982c24 COPYING > > A single sha256 hash was sufficient here. > > > diff --git a/package/liblo/liblo.mk b/package/liblo/liblo.mk > > index 3c852be..856b437 100644 > > --- a/package/liblo/liblo.mk > > +++ b/package/liblo/liblo.mk > > @@ -4,11 +4,12 @@ > > # > > ################################################################################ > > > > -LIBLO_VERSION = 0.26 > > +LIBLO_VERSION = 0.29 > > LIBLO_SITE = http://downloads.sourceforge.net/project/liblo/liblo/$(LIBLO_VERSION) > > > > LIBLO_LICENSE = LGPL-2.1+ > > LIBLO_LICENSE_FILES = COPYING > > LIBLO_INSTALL_STAGING = YES > > +LIBLO_CONF_OPTS += --disable-ipv6 > > A comment above this line (in addition to the commit log) would have > been nice. > > I've fixed both issues, and applied your patch to next. Thanks! > > Thomas > -- > Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) > Embedded Linux and Kernel engineering > http://bootlin.com -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180210/87f2c79d/attachment.html> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-10 16:19 ` Alex Baldwin @ 2018-02-10 17:45 ` Thomas Petazzoni 2018-02-11 16:25 ` Alex Baldwin 0 siblings, 1 reply; 7+ messages in thread From: Thomas Petazzoni @ 2018-02-10 17:45 UTC (permalink / raw) To: buildroot 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-10 17:45 ` Thomas Petazzoni @ 2018-02-11 16:25 ` Alex Baldwin 2018-02-12 20:05 ` Thomas Petazzoni 0 siblings, 1 reply; 7+ messages in thread From: Alex Baldwin @ 2018-02-11 16:25 UTC (permalink / raw) To: buildroot Hi Thomas (sorry about the double message, forgot to include the mailing list!) On 10 February 2018 at 18:45, Thomas Petazzoni <thomas.petazzoni@bootlin.com> wrote: > 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. When submitting this patch should I include it as V3 (as V2 failed), with all the original patches and ammend the changlog or simply just submit the new changes? > > Thanks! > > Thomas > -- > Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) > Embedded Linux and Kernel engineering > http://bootlin.com Thanks, Alex ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 2018-02-11 16:25 ` Alex Baldwin @ 2018-02-12 20:05 ` Thomas Petazzoni 0 siblings, 0 replies; 7+ messages in thread From: Thomas Petazzoni @ 2018-02-12 20:05 UTC (permalink / raw) To: buildroot Hello, On Sun, 11 Feb 2018 17:25:25 +0100, Alex Baldwin wrote: > > In any case, if you do a patch against liblo, don't forget to submit it > > upstream. > > When submitting this patch should I include it as V3 (as V2 failed), > with all the original patches and ammend the changlog or simply just > submit the new changes? As replied to your new submission: you should have sent just the new changes, because the liblo package has been applied to the next branch already (which is why it was tested by our autobuilders, and why you received e-mails about build failures). Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering http://bootlin.com ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-02-12 20:05 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-02-08 19:37 [Buildroot] [PATCH v2 1/1] LIBLO : bump version to 0.29 Alex B 2018-02-08 19:37 ` Alex B 2018-02-08 22:04 ` Thomas Petazzoni 2018-02-10 16:19 ` Alex Baldwin 2018-02-10 17:45 ` Thomas Petazzoni 2018-02-11 16:25 ` Alex Baldwin 2018-02-12 20:05 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox