Re: intltool rule patch

[Sander Vesik <Sander Vesik Sun COM>]

On Fri, 15 Mar 2002, Jody Goldberg wrote:

> On Fri, Mar 15, 2002 at 03:12:30PM -0500, jacob berkman wrote:
> >  
> > -  INTLTOOL_DESKTOP_RULE='%.desktop:   %.desktop.in   $(INTLTOOL_MERGE) $(wildcard $(top_srcdir)/po/*.po) ; $(INTLTOOL_MERGE) $(top_srcdir)/po $< [$]@ -d -u -c $(top_builddir)/po/.intltool-merge-cache'
> > +  INTLTOOL_DESKTOP_RULE='%.desktop:   %.desktop.in   ; $(INTLTOOL_MERGE) $(wildcard $(top_srcdir)/po/*.po) ; $(INTLTOOL_MERGE) $(top_srcdir)/po $< [$]@ -d -u -c $(top_builddir)/po/.intltool-merge-cache'
> 
> Ok so you are proposing to replace piles of
> 
>     %.a : %.b    commands
> with
>     %.a : %.b  ; commands
> 
> That is correct for gmake, but I don't know how portable it is.
> The existing rules are not terribly portable though.  Why use
> $(wildcard) ? The command is run in a shell.  Just use the builtin
> wildcarding.
> 

Doing
	
target: dependency<newline>
<tab>commands

Should work for all makes, including all gmakes old and new, whetever the
target and dependency are wildcarded or no.

> eg
> INTLTOOL_DESKTOP_RULE='%.desktop:   %.desktop.in   ; $(INTLTOOL_MERGE) $(top_srcdir)/po/*.po ; $(INTLTOOL_MERGE) $(top_srcdir)/po $< [$]@ -d -u -c $(top_builddir)/po/.intltool-merge-cache'
> 

	Sander

	I see a dark sail on the horizon
	Set under a dark cloud that hides the sun
	Bring me my Broadsword and clear understanding