* [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static
@ 2019-10-19 8:34 Fabrice Fontaine
2019-10-25 18:23 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Fabrice Fontaine @ 2019-10-19 8:34 UTC (permalink / raw)
To: buildroot
Fixes:
- http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
...x-build-without-babel-bgp-ospf-radv-.patch | 47 +++++++++++++++++++
1 file changed, 47 insertions(+)
create mode 100644 package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
diff --git a/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
new file mode 100644
index 0000000000..d27b87ca3f
--- /dev/null
+++ b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
@@ -0,0 +1,47 @@
+From 16d4ba934e4e7458d9437f45faef5774c358e979 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Sat, 19 Oct 2019 10:16:23 +0200
+Subject: [PATCH] nest/config.Y: fix build without babel, bgp, ospf, radv, rip
+ or static
+
+Build without babel, bgp, ospf, radv, rip or static fails on:
+
+bison -Dparse.lac=full -Dparse.error=verbose -dv -pcf_ -b
+obj/conf/cf-parse obj/conf/cf-parse.y
+nest/config.Y:381.20-23: error: symbol LINK is used, but is not
+defined as a token and has no rules
+ 381 | | dev_proto CHECK LINK bool ';' { DIRECT_CFG->check_link = $4; }
+ | ^~~~
+make[1]: *** [obj/conf/cf-parse.tab.c] Error 1
+
+This error is raised because of the following commit:
+https://gitlab.labs.nic.cz/labs/bird/commit/e90dd656cc9126e1fbcc45fb77a10bf1baa2a1b5
+
+This commit assumes that LINK is defined which is not always the case.
+Indeed, it seems that LINK is not defined by BFD, MRT, PERF, PIPE, RPKI
+
+To fix this issue, add CF_KEYWORDS(LINK) in nest/config.py
+
+Fixes:
+ - http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+---
+ nest/config.Y | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/nest/config.Y b/nest/config.Y
+index c62501a3..44f392dd 100644
+--- a/nest/config.Y
++++ b/nest/config.Y
+@@ -75,6 +75,7 @@ CF_KEYWORDS(BGP, PASSWORDS, DESCRIPTION, SORTED)
+ CF_KEYWORDS(RELOAD, IN, OUT, MRTDUMP, MESSAGES, RESTRICT, MEMORY, IGP_METRIC, CLASS, DSCP)
+ CF_KEYWORDS(TIMEFORMAT, ISO, SHORT, LONG, ROUTE, PROTOCOL, BASE, LOG, S, MS, US)
+ CF_KEYWORDS(GRACEFUL, RESTART, WAIT, MAX, FLUSH, AS)
++CF_KEYWORDS(LINK)
+
+ /* For r_args_channel */
+ CF_KEYWORDS(IPV4, IPV4_MC, IPV4_MPLS, IPV6, IPV6_MC, IPV6_MPLS, IPV6_SADR, VPN4, VPN4_MC, VPN4_MPLS, VPN6, VPN6_MC, VPN6_MPLS, ROA4, ROA6, FLOW4, FLOW6, MPLS, PRI, SEC)
+--
+2.23.0
+
--
2.23.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static
2019-10-19 8:34 [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static Fabrice Fontaine
@ 2019-10-25 18:23 ` Thomas Petazzoni
2019-10-25 18:48 ` Fabrice Fontaine
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2019-10-25 18:23 UTC (permalink / raw)
To: buildroot
Hello Adrien,
On Sat, 19 Oct 2019 10:34:04 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
> Fixes:
> - http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Adrien, do you have some feedback on the below patch from Fabrice
fixing build issues in bird ?
Thanks!
Thomas
> ---
> ...x-build-without-babel-bgp-ospf-radv-.patch | 47 +++++++++++++++++++
> 1 file changed, 47 insertions(+)
> create mode 100644 package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
>
> diff --git a/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> new file mode 100644
> index 0000000000..d27b87ca3f
> --- /dev/null
> +++ b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> @@ -0,0 +1,47 @@
> +From 16d4ba934e4e7458d9437f45faef5774c358e979 Mon Sep 17 00:00:00 2001
> +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +Date: Sat, 19 Oct 2019 10:16:23 +0200
> +Subject: [PATCH] nest/config.Y: fix build without babel, bgp, ospf, radv, rip
> + or static
> +
> +Build without babel, bgp, ospf, radv, rip or static fails on:
> +
> +bison -Dparse.lac=full -Dparse.error=verbose -dv -pcf_ -b
> +obj/conf/cf-parse obj/conf/cf-parse.y
> +nest/config.Y:381.20-23: error: symbol LINK is used, but is not
> +defined as a token and has no rules
> + 381 | | dev_proto CHECK LINK bool ';' { DIRECT_CFG->check_link = $4; }
> + | ^~~~
> +make[1]: *** [obj/conf/cf-parse.tab.c] Error 1
> +
> +This error is raised because of the following commit:
> +https://gitlab.labs.nic.cz/labs/bird/commit/e90dd656cc9126e1fbcc45fb77a10bf1baa2a1b5
> +
> +This commit assumes that LINK is defined which is not always the case.
> +Indeed, it seems that LINK is not defined by BFD, MRT, PERF, PIPE, RPKI
> +
> +To fix this issue, add CF_KEYWORDS(LINK) in nest/config.py
> +
> +Fixes:
> + - http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +---
> + nest/config.Y | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/nest/config.Y b/nest/config.Y
> +index c62501a3..44f392dd 100644
> +--- a/nest/config.Y
> ++++ b/nest/config.Y
> +@@ -75,6 +75,7 @@ CF_KEYWORDS(BGP, PASSWORDS, DESCRIPTION, SORTED)
> + CF_KEYWORDS(RELOAD, IN, OUT, MRTDUMP, MESSAGES, RESTRICT, MEMORY, IGP_METRIC, CLASS, DSCP)
> + CF_KEYWORDS(TIMEFORMAT, ISO, SHORT, LONG, ROUTE, PROTOCOL, BASE, LOG, S, MS, US)
> + CF_KEYWORDS(GRACEFUL, RESTART, WAIT, MAX, FLUSH, AS)
> ++CF_KEYWORDS(LINK)
> +
> + /* For r_args_channel */
> + CF_KEYWORDS(IPV4, IPV4_MC, IPV4_MPLS, IPV6, IPV6_MC, IPV6_MPLS, IPV6_SADR, VPN4, VPN4_MC, VPN4_MPLS, VPN6, VPN6_MC, VPN6_MPLS, ROA4, ROA6, FLOW4, FLOW6, MPLS, PRI, SEC)
> +--
> +2.23.0
> +
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static
2019-10-25 18:23 ` Thomas Petazzoni
@ 2019-10-25 18:48 ` Fabrice Fontaine
2019-10-25 22:25 ` Adrien Gallouët
0 siblings, 1 reply; 4+ messages in thread
From: Fabrice Fontaine @ 2019-10-25 18:48 UTC (permalink / raw)
To: buildroot
Hello,
Le ven. 25 oct. 2019 ? 20:23, Thomas Petazzoni
<thomas.petazzoni@bootlin.com> a ?crit :
>
> Hello Adrien,
>
> On Sat, 19 Oct 2019 10:34:04 +0200
> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
>
> > Fixes:
> > - http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>
> Adrien, do you have some feedback on the below patch from Fabrice
> fixing build issues in bird ?
For information, I submitted this patch and it was merged a few days ago:
https://gitlab.labs.nic.cz/labs/bird/commit/f9eb9b4cab65b850c61738dd56632bae0e7329ca
I can send a v2 if needed.
>
> Thanks!
>
> Thomas
>
> > ---
> > ...x-build-without-babel-bgp-ospf-radv-.patch | 47 +++++++++++++++++++
> > 1 file changed, 47 insertions(+)
> > create mode 100644 package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> >
> > diff --git a/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> > new file mode 100644
> > index 0000000000..d27b87ca3f
> > --- /dev/null
> > +++ b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> > @@ -0,0 +1,47 @@
> > +From 16d4ba934e4e7458d9437f45faef5774c358e979 Mon Sep 17 00:00:00 2001
> > +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > +Date: Sat, 19 Oct 2019 10:16:23 +0200
> > +Subject: [PATCH] nest/config.Y: fix build without babel, bgp, ospf, radv, rip
> > + or static
> > +
> > +Build without babel, bgp, ospf, radv, rip or static fails on:
> > +
> > +bison -Dparse.lac=full -Dparse.error=verbose -dv -pcf_ -b
> > +obj/conf/cf-parse obj/conf/cf-parse.y
> > +nest/config.Y:381.20-23: error: symbol LINK is used, but is not
> > +defined as a token and has no rules
> > + 381 | | dev_proto CHECK LINK bool ';' { DIRECT_CFG->check_link = $4; }
> > + | ^~~~
> > +make[1]: *** [obj/conf/cf-parse.tab.c] Error 1
> > +
> > +This error is raised because of the following commit:
> > +https://gitlab.labs.nic.cz/labs/bird/commit/e90dd656cc9126e1fbcc45fb77a10bf1baa2a1b5
> > +
> > +This commit assumes that LINK is defined which is not always the case.
> > +Indeed, it seems that LINK is not defined by BFD, MRT, PERF, PIPE, RPKI
> > +
> > +To fix this issue, add CF_KEYWORDS(LINK) in nest/config.py
> > +
> > +Fixes:
> > + - http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
> > +
> > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > +---
> > + nest/config.Y | 1 +
> > + 1 file changed, 1 insertion(+)
> > +
> > +diff --git a/nest/config.Y b/nest/config.Y
> > +index c62501a3..44f392dd 100644
> > +--- a/nest/config.Y
> > ++++ b/nest/config.Y
> > +@@ -75,6 +75,7 @@ CF_KEYWORDS(BGP, PASSWORDS, DESCRIPTION, SORTED)
> > + CF_KEYWORDS(RELOAD, IN, OUT, MRTDUMP, MESSAGES, RESTRICT, MEMORY, IGP_METRIC, CLASS, DSCP)
> > + CF_KEYWORDS(TIMEFORMAT, ISO, SHORT, LONG, ROUTE, PROTOCOL, BASE, LOG, S, MS, US)
> > + CF_KEYWORDS(GRACEFUL, RESTART, WAIT, MAX, FLUSH, AS)
> > ++CF_KEYWORDS(LINK)
> > +
> > + /* For r_args_channel */
> > + CF_KEYWORDS(IPV4, IPV4_MC, IPV4_MPLS, IPV6, IPV6_MC, IPV6_MPLS, IPV6_SADR, VPN4, VPN4_MC, VPN4_MPLS, VPN6, VPN6_MC, VPN6_MPLS, ROA4, ROA6, FLOW4, FLOW6, MPLS, PRI, SEC)
> > +--
> > +2.23.0
> > +
>
>
>
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best Regards,
Fabrice
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static
2019-10-25 18:48 ` Fabrice Fontaine
@ 2019-10-25 22:25 ` Adrien Gallouët
0 siblings, 0 replies; 4+ messages in thread
From: Adrien Gallouët @ 2019-10-25 22:25 UTC (permalink / raw)
To: buildroot
Hi all,
Le ven. 25 oct. 2019 ? 20:47, Fabrice Fontaine <fontaine.fabrice@gmail.com>
a ?crit :
> Hello,
>
> Le ven. 25 oct. 2019 ? 20:23, Thomas Petazzoni
> <thomas.petazzoni@bootlin.com> a ?crit :
> >
> > Hello Adrien,
> >
> > On Sat, 19 Oct 2019 10:34:04 +0200
> > Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
> >
> > > Fixes:
> > > -
> http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
> > >
> > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> >
> > Adrien, do you have some feedback on the below patch from Fabrice
> > fixing build issues in bird ?
> For information, I submitted this patch and it was merged a few days ago:
>
> https://gitlab.labs.nic.cz/labs/bird/commit/f9eb9b4cab65b850c61738dd56632bae0e7329ca
> I can send a v2 if needed.
> >
> > Thanks!
> >
> > Thomas
> >
> > > ---
> > > ...x-build-without-babel-bgp-ospf-radv-.patch | 47 +++++++++++++++++++
> > > 1 file changed, 47 insertions(+)
> > > create mode 100644
> package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> > >
> > > diff --git
> a/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> > > new file mode 100644
> > > index 0000000000..d27b87ca3f
> > > --- /dev/null
> > > +++
> b/package/bird/0001-nest-config.Y-fix-build-without-babel-bgp-ospf-radv-.patch
> > > @@ -0,0 +1,47 @@
> > > +From 16d4ba934e4e7458d9437f45faef5774c358e979 Mon Sep 17 00:00:00 2001
> > > +From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > > +Date: Sat, 19 Oct 2019 10:16:23 +0200
> > > +Subject: [PATCH] nest/config.Y: fix build without babel, bgp, ospf,
> radv, rip
> > > + or static
> > > +
> > > +Build without babel, bgp, ospf, radv, rip or static fails on:
> > > +
> > > +bison -Dparse.lac=full -Dparse.error=verbose -dv -pcf_ -b
> > > +obj/conf/cf-parse obj/conf/cf-parse.y
> > > +nest/config.Y:381.20-23: error: symbol LINK is used, but is not
> > > +defined as a token and has no rules
> > > + 381 | | dev_proto CHECK LINK bool ';' { DIRECT_CFG->check_link =
> $4; }
> > > + | ^~~~
> > > +make[1]: *** [obj/conf/cf-parse.tab.c] Error 1
> > > +
> > > +This error is raised because of the following commit:
> > > +
> https://gitlab.labs.nic.cz/labs/bird/commit/e90dd656cc9126e1fbcc45fb77a10bf1baa2a1b5
> > > +
> > > +This commit assumes that LINK is defined which is not always the case.
> > > +Indeed, it seems that LINK is not defined by BFD, MRT, PERF, PIPE,
> RPKI
> > > +
> > > +To fix this issue, add CF_KEYWORDS(LINK) in nest/config.py
> > > +
> > > +Fixes:
> > > + -
> http://autobuild.buildroot.org/results/0b00948eed9bb8405b70f3f9112ecce99b365f35
> > > +
> > > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > > +---
> > > + nest/config.Y | 1 +
> > > + 1 file changed, 1 insertion(+)
> > > +
> > > +diff --git a/nest/config.Y b/nest/config.Y
> > > +index c62501a3..44f392dd 100644
> > > +--- a/nest/config.Y
> > > ++++ b/nest/config.Y
> > > +@@ -75,6 +75,7 @@ CF_KEYWORDS(BGP, PASSWORDS, DESCRIPTION, SORTED)
> > > + CF_KEYWORDS(RELOAD, IN, OUT, MRTDUMP, MESSAGES, RESTRICT, MEMORY,
> IGP_METRIC, CLASS, DSCP)
> > > + CF_KEYWORDS(TIMEFORMAT, ISO, SHORT, LONG, ROUTE, PROTOCOL, BASE,
> LOG, S, MS, US)
> > > + CF_KEYWORDS(GRACEFUL, RESTART, WAIT, MAX, FLUSH, AS)
> > > ++CF_KEYWORDS(LINK)
> > > +
> > > + /* For r_args_channel */
> > > + CF_KEYWORDS(IPV4, IPV4_MC, IPV4_MPLS, IPV6, IPV6_MC, IPV6_MPLS,
> IPV6_SADR, VPN4, VPN4_MC, VPN4_MPLS, VPN6, VPN6_MC, VPN6_MPLS, ROA4, ROA6,
> FLOW4, FLOW6, MPLS, PRI, SEC)
> > > +--
> > > +2.23.0
> > > +
> >
> >
> >
> > --
> > Thomas Petazzoni, CTO, Bootlin
> > Embedded Linux and Kernel engineering
> > https://bootlin.com
> Best Regards,
>
> Fabrice
>
Thanks a lot for taking the time to investigate and fix this issue Fabrice.
I would just recommend a v2 to use the exact same patch to be consistent
with the upstream code.
Best regards,
Adrien
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20191026/26c3a3c8/attachment.html>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-25 22:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-19 8:34 [Buildroot] [PATCH 1/1] package/bird: fix build without babel, bgp, ospf, radv, rip or static Fabrice Fontaine
2019-10-25 18:23 ` Thomas Petazzoni
2019-10-25 18:48 ` Fabrice Fontaine
2019-10-25 22:25 ` Adrien Gallouët
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.