[Libguestfs] [PATCH 2/3] po: fix dependencies for libguestfs.pot extraction

[Richard W.M. Jones]

On Thu, Dec 05, 2013 at 04:30:05PM +0100, Pino Toscano wrote:
> Fix the dependencies of the libguestfs.pot target: other than using the
> right make variables holding the contents of the POTFILES, depend also
> on the POTFILES themselves.
> ---
>  po/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/po/Makefile.am b/po/Makefile.am
> index b0a8038..a8343ec 100644
> --- a/po/Makefile.am
> +++ b/po/Makefile.am
> @@ -65,7 +65,7 @@ XGETTEXT_ARGS = \
>  	--msgid-bugs-address="$(MSGID_BUGS_ADDRESS)" \
>  	--directory=$(top_srcdir)
>  
> -$(DOMAIN).pot: Makefile $(POTFILES) $(POTFILES-pl) $(POTFILES-ml)
> +$(DOMAIN).pot: Makefile POTFILES $(POTFILES) POTFILES-pl $(POTFILES_PL) POTFILES-ml $(POTFILES_ML)
>  	rm -f $@-t
>  if HAVE_OCAML_GETTEXT
>  	$(OCAML_GETTEXT) --action extract --extract-pot $@-t $(POTFILES_ML)

So I agree that $(POTFILES-pl) is definitely wrong.  Not sure exactly
what we were thinking about there ...

But, won't the addition of the literal file names break separate
compilation?  In particular, $(POTFILES_PL) is supposed to be the
correct path to the file POTFILES-pl (and correspondingly for the
other files), so it shouldn't be necessary to list both POTFILES-pl
and $(POTFILES_PL).

Separate compilation is something that Debian cares about a lot
because they have to rebuild libguestfs several times in series for
different python versions etc.

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW