[Spice-devel] [PATCH spice-gtk 2/2] build-sys: modernize gettext

[Christophe Fergeau]

On Thu, Jul 13, 2017 at 08:58:42AM -0400, Marc-André Lureau wrote:
> Hi
> 
> ----- Original Message -----
> > On Wed, Jul 12, 2017 at 03:20:26PM +0000, Marc-André Lureau wrote:
> > > >
> > > > This is the latest gettext release, which is not even 1 year old, is
> > > > this the only one which is going to work? Or is it possible to relax
> > > > this version chekc?
> > > >
> > > 
> > > It's quite a mess to downgrade gettext or find the minimum required
> > > version. RHEL7 has 0.19.8.1 already. I suppose it should work with earlier
> > > 0.19 releases, perhaps even older..
> > 
> > RHEL7.x is using 0.18, and has been doing that for a few point releases.
> > RHEL7.4 will apparently have gettext 0.19, but that's not released yet..
> > Ubuntu 16.04 LTS has 0.19, Ubuntu 14.04 had 0.18. Debian
> > stable/oldstable are both on 0.19.
> > However, I was able to build spice-gtk git master on an up to date RHEL
> > 7.3 after changing the required gettext version to 0.18, so I'd just
> > lower the requirement.
> > 
> 
> ACK feel free to push the change if you tested it.

Building with AM_GNU_GETTEXT_VERSION([0.18]) works, except that this
triggers some autotools warnings about the use of a deprecated
AM_PROG_MKDIR_P macro, which then aborts as we are using
AM_INIT_AUTOMATKE([-Werror]). Did not have time to dig more into it yet.

I'd just revert this patch as what is meant as a build system cleanup
prevents building from git on platforms which are fine otherwise.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170713/2bcea868/attachment-0001.sig>