* [Buildroot] [PATCH] package/openntpd: needs host-bison @ 2020-11-16 11:18 Baruch Siach 2020-11-16 16:19 ` Peter Korsgaard 2020-11-16 21:25 ` Peter Korsgaard 0 siblings, 2 replies; 7+ messages in thread From: Baruch Siach @ 2020-11-16 11:18 UTC (permalink / raw) To: buildroot Build fails when no yacc alternative is installed. Fixes: http://autobuild.buildroot.net/results/1ba8e339cbb5646663d0bf4e158d89e54433b242/ http://autobuild.buildroot.net/results/a00a53d6635c64e72c50d4841658155de5380110/ Signed-off-by: Baruch Siach <baruch@tkos.co.il> --- package/openntpd/openntpd.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk index f2eaee7efd46..9076f3ddda4e 100644 --- a/package/openntpd/openntpd.mk +++ b/package/openntpd/openntpd.mk @@ -8,6 +8,7 @@ OPENNTPD_VERSION = 6.2p3 OPENNTPD_SITE = http://ftp.openbsd.org/pub/OpenBSD/OpenNTPD OPENNTPD_LICENSE = MIT-like, BSD-2-Clause, BSD-3-Clause OPENNTPD_LICENSE_FILES = COPYING +OPENNTPD_DEPENDENCIES = host-bison # Need to autoreconf for our libtool patch to apply properly OPENNTPD_AUTORECONF = YES -- 2.29.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 11:18 [Buildroot] [PATCH] package/openntpd: needs host-bison Baruch Siach @ 2020-11-16 16:19 ` Peter Korsgaard 2020-11-16 17:09 ` Baruch Siach 2020-11-16 21:25 ` Peter Korsgaard 1 sibling, 1 reply; 7+ messages in thread From: Peter Korsgaard @ 2020-11-16 16:19 UTC (permalink / raw) To: buildroot >>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes: > Build fails when no yacc alternative is installed. > Fixes: > http://autobuild.buildroot.net/results/1ba8e339cbb5646663d0bf4e158d89e54433b242/ > http://autobuild.buildroot.net/results/a00a53d6635c64e72c50d4841658155de5380110/ > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > --- > package/openntpd/openntpd.mk | 1 + > 1 file changed, 1 insertion(+) > diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk > index f2eaee7efd46..9076f3ddda4e 100644 > --- a/package/openntpd/openntpd.mk > +++ b/package/openntpd/openntpd.mk > @@ -8,6 +8,7 @@ OPENNTPD_VERSION = 6.2p3 > OPENNTPD_SITE = http://ftp.openbsd.org/pub/OpenBSD/OpenNTPD > OPENNTPD_LICENSE = MIT-like, BSD-2-Clause, BSD-3-Clause > OPENNTPD_LICENSE_FILES = COPYING > +OPENNTPD_DEPENDENCIES = host-bison Can we not use $(BR2_BISON_HOST_DEPENDENCY) instead? -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 16:19 ` Peter Korsgaard @ 2020-11-16 17:09 ` Baruch Siach 2020-11-16 17:42 ` Peter Korsgaard 2020-11-16 18:07 ` Yann E. MORIN 0 siblings, 2 replies; 7+ messages in thread From: Baruch Siach @ 2020-11-16 17:09 UTC (permalink / raw) To: buildroot Hi Peter, +Yann. On Mon, Nov 16 2020, Peter Korsgaard wrote: >>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes: > > > Build fails when no yacc alternative is installed. > > Fixes: > > http://autobuild.buildroot.net/results/1ba8e339cbb5646663d0bf4e158d89e54433b242/ > > http://autobuild.buildroot.net/results/a00a53d6635c64e72c50d4841658155de5380110/ > > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > > --- > > package/openntpd/openntpd.mk | 1 + > > 1 file changed, 1 insertion(+) > > > diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk > > index f2eaee7efd46..9076f3ddda4e 100644 > > --- a/package/openntpd/openntpd.mk > > +++ b/package/openntpd/openntpd.mk > > @@ -8,6 +8,7 @@ OPENNTPD_VERSION = 6.2p3 > > OPENNTPD_SITE = http://ftp.openbsd.org/pub/OpenBSD/OpenNTPD > > OPENNTPD_LICENSE = MIT-like, BSD-2-Clause, BSD-3-Clause > > OPENNTPD_LICENSE_FILES = COPYING > > +OPENNTPD_DEPENDENCIES = host-bison > > Can we not use $(BR2_BISON_HOST_DEPENDENCY) instead? We surly can. But for some reason no other package under package/ uses BR2_BISON_HOST_DEPENDENCY. Plain host-bison is used there instead. Only Barebox, U-Boot and the kernel use it. Why is that? Commit b01100cc901a83 ("linux: kconfig may need host-{flex, bison} to build the configurators") mentions kconfig. Is that related? I guess that BR2_BISON_HOST_DEPENDENCY is more susceptible to incompatibility due to different host installed bison versions (and same for flex). baruch -- ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 17:09 ` Baruch Siach @ 2020-11-16 17:42 ` Peter Korsgaard 2020-11-16 18:07 ` Yann E. MORIN 1 sibling, 0 replies; 7+ messages in thread From: Peter Korsgaard @ 2020-11-16 17:42 UTC (permalink / raw) To: buildroot >>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes: Hi, >> > +OPENNTPD_DEPENDENCIES = host-bison >> >> Can we not use $(BR2_BISON_HOST_DEPENDENCY) instead? > We surly can. But for some reason no other package under package/ uses > BR2_BISON_HOST_DEPENDENCY. Plain host-bison is used there instead. Only > Barebox, U-Boot and the kernel use it. Why is that? Your tree is not uptodate, we also have it in linux-backports ;) But yes, it is a bit odd that we are not using it more often. Probably it is mostly just because it is fairly new (~2 years) and not really documented anywhere. Yann, you added it, anything to add? > Commit b01100cc901a83 ("linux: kconfig may need host-{flex, bison} to > build the configurators") mentions kconfig. Is that related? > I guess that BR2_BISON_HOST_DEPENDENCY is more susceptible to > incompatibility due to different host installed bison versions (and same > for flex). Correct, similar to the other components where we can use what is available on the host if new enough. For bison/flex, I would imagine that the version incompatibilities are fairly small (even though we do have some patches for building with bison 3.7.1). -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 17:09 ` Baruch Siach 2020-11-16 17:42 ` Peter Korsgaard @ 2020-11-16 18:07 ` Yann E. MORIN 2020-11-16 18:54 ` Peter Korsgaard 1 sibling, 1 reply; 7+ messages in thread From: Yann E. MORIN @ 2020-11-16 18:07 UTC (permalink / raw) To: buildroot Baruch, Peter, All, On 2020-11-16 19:09 +0200, Baruch Siach spake thusly: > On Mon, Nov 16 2020, Peter Korsgaard wrote: > >>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes: > > > > > Build fails when no yacc alternative is installed. > > > Fixes: > > > http://autobuild.buildroot.net/results/1ba8e339cbb5646663d0bf4e158d89e54433b242/ > > > http://autobuild.buildroot.net/results/a00a53d6635c64e72c50d4841658155de5380110/ > > > > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > > > --- > > > package/openntpd/openntpd.mk | 1 + > > > 1 file changed, 1 insertion(+) > > > > > diff --git a/package/openntpd/openntpd.mk b/package/openntpd/openntpd.mk > > > index f2eaee7efd46..9076f3ddda4e 100644 > > > --- a/package/openntpd/openntpd.mk > > > +++ b/package/openntpd/openntpd.mk > > > @@ -8,6 +8,7 @@ OPENNTPD_VERSION = 6.2p3 > > > OPENNTPD_SITE = http://ftp.openbsd.org/pub/OpenBSD/OpenNTPD > > > OPENNTPD_LICENSE = MIT-like, BSD-2-Clause, BSD-3-Clause > > > OPENNTPD_LICENSE_FILES = COPYING > > > +OPENNTPD_DEPENDENCIES = host-bison > > > > Can we not use $(BR2_BISON_HOST_DEPENDENCY) instead? > > We surly can. But for some reason no other package under package/ uses > BR2_BISON_HOST_DEPENDENCY. Plain host-bison is used there instead. Only > Barebox, U-Boot and the kernel use it. Why is that? > > Commit b01100cc901a83 ("linux: kconfig may need host-{flex, bison} to > build the configurators") mentions kconfig. Is that related? > > I guess that BR2_BISON_HOST_DEPENDENCY is more susceptible to > incompatibility due to different host installed bison versions (and same > for flex). The rule is that we can depend on $(BR2_BISON_HOST_DEPENDENCY) to build host tools, but we really want to depend on host-bison when we generate code that is built for the target. Indeed, different versions of bison will generate different C code, and thus the resulting binary will change. This is not very reproducible. (Yes, I've already experienced that, and finding the root cause of the delta cost me about two weeks full-time...) AFAICS, the openntd code will be running on the target, so we do want the host-bison dependency. Acked-by: Yann E. MORIN <yann.morin.1998@free.fr> Regards, Yann E. MORIN. -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 18:07 ` Yann E. MORIN @ 2020-11-16 18:54 ` Peter Korsgaard 0 siblings, 0 replies; 7+ messages in thread From: Peter Korsgaard @ 2020-11-16 18:54 UTC (permalink / raw) To: buildroot >>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes: Hi, >> > Can we not use $(BR2_BISON_HOST_DEPENDENCY) instead? >> >> We surly can. But for some reason no other package under package/ uses >> BR2_BISON_HOST_DEPENDENCY. Plain host-bison is used there instead. Only >> Barebox, U-Boot and the kernel use it. Why is that? >> >> Commit b01100cc901a83 ("linux: kconfig may need host-{flex, bison} to >> build the configurators") mentions kconfig. Is that related? >> >> I guess that BR2_BISON_HOST_DEPENDENCY is more susceptible to >> incompatibility due to different host installed bison versions (and same >> for flex). > The rule is that we can depend on $(BR2_BISON_HOST_DEPENDENCY) to build host > tools, but we really want to depend on host-bison when we generate code > that is built for the target. > Indeed, different versions of bison will generate different C code, and > thus the resulting binary will change. This is not very reproducible. > (Yes, I've already experienced that, and finding the root cause of the > delta cost me about two weeks full-time...) > AFAICS, the openntd code will be running on the target, so we do want > the host-bison dependency. > Acked-by: Yann E. MORIN <yann.morin.1998@free.fr> Ok, we should probably add a sensible note about that in check-host-bison-flex.mk to make that clear. Committed, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] package/openntpd: needs host-bison 2020-11-16 11:18 [Buildroot] [PATCH] package/openntpd: needs host-bison Baruch Siach 2020-11-16 16:19 ` Peter Korsgaard @ 2020-11-16 21:25 ` Peter Korsgaard 1 sibling, 0 replies; 7+ messages in thread From: Peter Korsgaard @ 2020-11-16 21:25 UTC (permalink / raw) To: buildroot >>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes: > Build fails when no yacc alternative is installed. > Fixes: > http://autobuild.buildroot.net/results/1ba8e339cbb5646663d0bf4e158d89e54433b242/ > http://autobuild.buildroot.net/results/a00a53d6635c64e72c50d4841658155de5380110/ > Signed-off-by: Baruch Siach <baruch@tkos.co.il> Committed to 2020.02.x and 2020.08.x, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-11-16 21:25 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-11-16 11:18 [Buildroot] [PATCH] package/openntpd: needs host-bison Baruch Siach 2020-11-16 16:19 ` Peter Korsgaard 2020-11-16 17:09 ` Baruch Siach 2020-11-16 17:42 ` Peter Korsgaard 2020-11-16 18:07 ` Yann E. MORIN 2020-11-16 18:54 ` Peter Korsgaard 2020-11-16 21:25 ` Peter Korsgaard
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.