[sanvila@unex.es: Re: Bug#57136: gettext: please apply patch from GCC developers to support --keyword=WORD:3]

[Zack Weinberg <zack at wolery dot cumb dot org>]

I got this from the Debian gettext maintainer.  I had asked him to
apply the patch for --keyword=WORD:3 to their package, since the next
release of gettext is not happening anytime soon.  He did this, and
has now found a problem with the patch.

I think we need a ; after the 'done', as well as a backslash at the
end of that line.

----- Forwarded message from Santiago Vila <sanvila@unex.es> -----

Date: Mon, 17 Jul 2000 20:24:01 +0200 (CEST)
From: Santiago Vila <sanvila@unex.es>
To: Zack Weinberg <zack@wolery.cumb.org>
Subject: Re: Bug#57136: gettext: please apply patch from GCC developers to support --keyword=WORD:3

On Sat, 5 Feb 2000, Zack Weinberg wrote:

> Package: gettext
> Version: 0.10.35-13
> Severity: wishlist
> 
> The currently released version of gettext is not capable of regenerating
> the .po file for GCC.  A patch is included with the GCC development snapshots.
> It would be nice if this patch could be applied to the Debian package of
> gettext.

Hello.

I've received two reports saying there is a missing `\' here:

> diff -pu -r0.10.35.0 -r0.10.35.1
> --- intl/Makefile.in	1998/04/27 21:53:18	0.10.35.0
> +++ intl/Makefile.in	1998/07/09 21:39:18	0.10.35.1
> @@ -143,10 +143,14 @@ install-data: all
>  installcheck:
>  
>  uninstall:
> -	dists="$(DISTFILES.common)"; \
> -	for file in $$dists; do \
> -	  rm -f $(gettextsrcdir)/$$file; \
> -	done
> +	if test "$(PACKAGE)" = "gettext"; then \
> +	  dists="$(DISTFILES.common)"; \
> +	  for file in $$dists; do \
> +	    rm -f $(gettextsrcdir)/$$file; \
> +	  done
                ^^^^^
> +	else \
> +	  : ; \
> +	fi

So I'm going to add it to the Debian version.

Do you know whether or not GCC developers are aware of this?

Thanks.

----- End forwarded message -----