* [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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.