Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH next] package/bird: update to 2.15.1
@ 2024-08-11  7:16 Waldemar Brodkorb
  2024-08-12 20:22 ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 3+ messages in thread
From: Waldemar Brodkorb @ 2024-08-11  7:16 UTC (permalink / raw)
  To: buildroot

See here for a changelog:
https://gitlab.nic.cz/labs/bird/-/blob/master/NEWS

An uClibc-ng based system needs _GNU_SOURCE in the CFLAGS to
provide getrandom().

Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
---
 package/bird/bird.hash | 2 +-
 package/bird/bird.mk   | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/package/bird/bird.hash b/package/bird/bird.hash
index a62fbba630..a781e314df 100644
--- a/package/bird/bird.hash
+++ b/package/bird/bird.hash
@@ -1,2 +1,2 @@
-sha256  3ec462a237d06d1f4455d6ec00a42f0b1686061fc988e5c89a841d01dd753b53  bird-2.0.12.tar.gz
+sha256  48e85c622de164756c132ea77ad1a8a95cc9fd0137ffd0d882746589ce75c75d  bird-2.15.1.tar.gz
 sha256  94c53c84320078920ac1f0d49c81a4e9004512f534521a58bdf145acbcbc2cd2  README
diff --git a/package/bird/bird.mk b/package/bird/bird.mk
index f1b2ee6c5e..34d7912954 100644
--- a/package/bird/bird.mk
+++ b/package/bird/bird.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-BIRD_VERSION = 2.0.12
+BIRD_VERSION = 2.15.1
 BIRD_SITE = https://bird.network.cz/download
 BIRD_LICENSE = GPL-2.0+
 BIRD_LICENSE_FILES = README
@@ -12,6 +12,8 @@ BIRD_CPE_ID_VENDOR = nic
 BIRD_SELINUX_MODULES = bird
 BIRD_DEPENDENCIES = host-flex host-bison
 
+BIRD_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -D_GNU_SOURCE"
+
 # 0001-configure.ac-fix-build-with-autoconf-2.70.patch
 BIRD_AUTORECONF = YES
 
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH next] package/bird: update to 2.15.1
  2024-08-11  7:16 [Buildroot] [PATCH next] package/bird: update to 2.15.1 Waldemar Brodkorb
@ 2024-08-12 20:22 ` Thomas Petazzoni via buildroot
  2024-08-13  4:38   ` Waldemar Brodkorb
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni via buildroot @ 2024-08-12 20:22 UTC (permalink / raw)
  To: Waldemar Brodkorb; +Cc: Fabrice Fontaine, buildroot

Hello,

+Fabrice in Cc, as there is a request below :-)

On Sun, 11 Aug 2024 09:16:31 +0200
Waldemar Brodkorb <wbx@openadk.org> wrote:

> See here for a changelog:
> https://gitlab.nic.cz/labs/bird/-/blob/master/NEWS
> 
> An uClibc-ng based system needs _GNU_SOURCE in the CFLAGS to
> provide getrandom().
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>

Thanks, I've applied to next, but I have some comments below.

> +BIRD_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -D_GNU_SOURCE"

This seems to be a workaround for an issue in uClibc. Indeed, I don't
see in the manpage of getrandom() anything that indicates that defining
_GNU_SOURCE is needed to get the definition of getrandom(). Why uClibc
is guarding the definition of getrandom() on _GNU_SOURCE, and not glibc
or musl?

>  # 0001-configure.ac-fix-build-with-autoconf-2.70.patch
>  BIRD_AUTORECONF = YES

Could you work on upstreaming this patch perhaps? Or maybe convince
Fabrice to upstream it? :-)

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH next] package/bird: update to 2.15.1
  2024-08-12 20:22 ` Thomas Petazzoni via buildroot
@ 2024-08-13  4:38   ` Waldemar Brodkorb
  0 siblings, 0 replies; 3+ messages in thread
From: Waldemar Brodkorb @ 2024-08-13  4:38 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: Fabrice Fontaine, buildroot

Hi Thomas,
Thomas Petazzoni wrote,

> Hello,
> 
> +Fabrice in Cc, as there is a request below :-)
> 
> On Sun, 11 Aug 2024 09:16:31 +0200
> Waldemar Brodkorb <wbx@openadk.org> wrote:
> 
> > See here for a changelog:
> > https://gitlab.nic.cz/labs/bird/-/blob/master/NEWS
> > 
> > An uClibc-ng based system needs _GNU_SOURCE in the CFLAGS to
> > provide getrandom().
> > 
> > Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> 
> Thanks, I've applied to next, but I have some comments below.
> 
> > +BIRD_CONF_ENV += CFLAGS="$(TARGET_CFLAGS) -D_GNU_SOURCE"
> 
> This seems to be a workaround for an issue in uClibc. Indeed, I don't
> see in the manpage of getrandom() anything that indicates that defining
> _GNU_SOURCE is needed to get the definition of getrandom(). Why uClibc
> is guarding the definition of getrandom() on _GNU_SOURCE, and not glibc
> or musl?

I don't know :)
Hopefully fixed in uClibc-ng 1.0.50.

best regards
 Waldemar
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-08-13  4:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-11  7:16 [Buildroot] [PATCH next] package/bird: update to 2.15.1 Waldemar Brodkorb
2024-08-12 20:22 ` Thomas Petazzoni via buildroot
2024-08-13  4:38   ` Waldemar Brodkorb

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox