[Buildroot] [PATCH] package/bison: disable libtextstyle

[Buildroot] [PATCH] package/bison: disable libtextstyle

[Baruch Siach]

Recent bison versions added support for fancy error output using gettext
provided libtextstyle. The Buildroot gettext version does not provided
all needed features. However host-bison might detect host installed
libtextstyle at configure time, but fail at run time because of missing
symbols in Buildroot provided libtextstyle.

We don't really need fancy output of host packages, so just disable
libtextstyle support.

Fixes (libpcap):
http://autobuild.buildroot.net/results/058e8ec90da3c06e31b31eb94541331cced44db3/
http://autobuild.buildroot.net/results/d99dda99722c53730b5964bd4ff1e1281c4ad759/
http://autobuild.buildroot.net/results/056fd1246554fe7def416429620175e86530c0a7/

Cc: Xogium <contact@xogium.me>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
Hi Xogium,

Can you verify that this indeed fixes libpcap configure on your builder?

Thanks,
baruch
---
 package/bison/bison.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/bison/bison.mk b/package/bison/bison.mk
index 45364e3cd5e6..70111a8d6297 100644
--- a/package/bison/bison.mk
+++ b/package/bison/bison.mk
@@ -13,5 +13,6 @@ BISON_LICENSE_FILES = COPYING
 BISON_MAKE = $(MAKE1)
 HOST_BISON_DEPENDENCIES = host-m4
 HOST_BISON_CONF_OPTS = --enable-relocatable
+HOST_BISON_CONF_ENV = ac_cv_libtextstyle=no
 
 $(eval $(host-autotools-package))
-- 
2.28.0

[Buildroot] [PATCH] package/bison: disable libtextstyle

[Baruch Siach]

Hi Buildroot list,

On Sun, Sep 27 2020, Baruch Siach wrote:
> Recent bison versions added support for fancy error output using gettext
> provided libtextstyle. The Buildroot gettext version does not provided

Typo: s/provided/provide/

> all needed features. However host-bison might detect host installed
> libtextstyle at configure time, but fail at run time because of missing
> symbols in Buildroot provided libtextstyle.
>
> We don't really need fancy output of host packages, so just disable
> libtextstyle support.
>
> Fixes (libpcap):
> http://autobuild.buildroot.net/results/058e8ec90da3c06e31b31eb94541331cced44db3/
> http://autobuild.buildroot.net/results/d99dda99722c53730b5964bd4ff1e1281c4ad759/
> http://autobuild.buildroot.net/results/056fd1246554fe7def416429620175e86530c0a7/
>
> Cc: Xogium <contact@xogium.me>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>

I'm adding Tested-by on behalf of Xogium, because he is not subscribed
to the list. Xogium verified that this patch fixes libpcap configure
step on his builder.

Tested-by: Xogium <contact@xogium.me>

baruch

> ---
> Hi Xogium,
>
> Can you verify that this indeed fixes libpcap configure on your builder?
>
> Thanks,
> baruch
> ---
>  package/bison/bison.mk | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/package/bison/bison.mk b/package/bison/bison.mk
> index 45364e3cd5e6..70111a8d6297 100644
> --- a/package/bison/bison.mk
> +++ b/package/bison/bison.mk
> @@ -13,5 +13,6 @@ BISON_LICENSE_FILES = COPYING
>  BISON_MAKE = $(MAKE1)
>  HOST_BISON_DEPENDENCIES = host-m4
>  HOST_BISON_CONF_OPTS = --enable-relocatable
> +HOST_BISON_CONF_ENV = ac_cv_libtextstyle=no
>  
>  $(eval $(host-autotools-package))


-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -

[Buildroot] [PATCH] package/bison: disable libtextstyle

[Thomas Petazzoni]

On Sun, 27 Sep 2020 08:08:29 +0300
Baruch Siach <baruch@tkos.co.il> wrote:

> Recent bison versions added support for fancy error output using gettext
> provided libtextstyle. The Buildroot gettext version does not provided
> all needed features. However host-bison might detect host installed
> libtextstyle at configure time, but fail at run time because of missing
> symbols in Buildroot provided libtextstyle.
> 
> We don't really need fancy output of host packages, so just disable
> libtextstyle support.
> 
> Fixes (libpcap):
> http://autobuild.buildroot.net/results/058e8ec90da3c06e31b31eb94541331cced44db3/
> http://autobuild.buildroot.net/results/d99dda99722c53730b5964bd4ff1e1281c4ad759/
> http://autobuild.buildroot.net/results/056fd1246554fe7def416429620175e86530c0a7/
> 
> Cc: Xogium <contact@xogium.me>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
> Hi Xogium,

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH] package/bison: disable libtextstyle

[Peter Korsgaard]

>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > Recent bison versions added support for fancy error output using gettext
 > provided libtextstyle. The Buildroot gettext version does not provided
 > all needed features. However host-bison might detect host installed
 > libtextstyle at configure time, but fail at run time because of missing
 > symbols in Buildroot provided libtextstyle.

 > We don't really need fancy output of host packages, so just disable
 > libtextstyle support.

 > Fixes (libpcap):
 > http://autobuild.buildroot.net/results/058e8ec90da3c06e31b31eb94541331cced44db3/
 > http://autobuild.buildroot.net/results/d99dda99722c53730b5964bd4ff1e1281c4ad759/
 > http://autobuild.buildroot.net/results/056fd1246554fe7def416429620175e86530c0a7/

 > Cc: Xogium <contact@xogium.me>
 > Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Committed to 2020.02.x, 2020.05.x and 2020.08.x, thanks.

-- 
Bye, Peter Korsgaard