Re: [spice-gtk] build-sys: Update gettext 0.18.2 -> 0.19.8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> From: Victor Toso <me@xxxxxxxxxxxxxx>
> 
> It was bumped to 0.19.0 by:
> 
>     commit 6195151f943480eecf38029ffad8a86844406659
>     Author: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
>     Date:   Tue Mar 28 14:51:04 2017 +0200
> 
>     build-sys: modernize gettext
>     (...)
> 
> And then got downgraded to 0.18.2 by:
> 
>     commit 20f717dac0962c2c52d0f17ba50556eb5aeffb0f
>     Author: Christophe Fergeau <cfergeau@xxxxxxxxxx>
>     Date:   Mon Jul 17 13:21:01 2017 +0200
> 
>     build-sys: Lower gettext requirement
> 
>     spice-gtk currently requires gettext 0.19 which is a fairly
>     recent release (not available in el7.3 and older, nor in
>     ubuntu 14.04 LTS).  They both have a gettext newer than
>     0.18.2, so we can require that.
> 
>     Requiring 0.18 or 0.18.1 is throwing automake errors because
>     of our use of -Werror, so we side-step this by requiring
>     0.18.2.
> 
> The 0.19 release is quite stable for a while now. Since RHEL 7.4
> we have 0.19.8 which this patch tries to have as well.
> 
> 0.19.8 was part of Fedora 25, considering that we are on Fedora
> 28, I hope its not a problem to bump this.
> 
> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index a9a7eb9..04dd8cb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -16,7 +16,7 @@ AM_MAINTAINER_MODE
>  GETTEXT_PACKAGE=AC_PACKAGE_TARNAME
>  AC_SUBST(GETTEXT_PACKAGE)
>  AC_DEFINE_UNQUOTED([GETTEXT_PACKAGE], "$GETTEXT_PACKAGE", [GETTEXT package
>  name])
> -AM_GNU_GETTEXT_VERSION([0.18.2])
> +AM_GNU_GETTEXT_VERSION([0.19.8])
>  AM_GNU_GETTEXT([external])
>  
>  

I miss the point. Why requiring a more recent version if we don't use
any new feature? The spice-gtk will pick up the new version if installed.
Looks like this patch only will fail to compile if you have an older version
despite it can work. Marc-Andre patch removed intltool usage as broken but
increasing the requirement too much while Christophe tested with a former
version still providing needed feature.
I upgraded to Ubuntu 18.04 LTS really recently, I suppose people will still
be using 14.04 LTS for a little longer.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]