[MPlayer-dev-eng] [PATCH] fix to Makefile uninstall

[Paul TT]

On Tue, 15 Nov 2005 16:25:31 +0100
Diego Biurrun <diego at biurrun.de> wrote:

> On Tue, Nov 15, 2005 at 03:49:31PM +0100, Paul TT wrote:
> > 
> > cvs commit: [14:45:37] waiting for anonymous's lock in
> > /cvsroot/mplayer/main
> 
> Send Attila a mail so he can remove the lock.
iep

> > -	  if test "i$$l" != "en"; then \
> > -	    -rm -f $(MANDIR)/$$l/man1/mplayer.1
$(MANDIR)/$$l/man1/mencoder.1 \
> > -	    -rm -f $(MANDIR)/$$l/man1/gmplayer.1 \
> > -	  fi \
> > +	  if test "$$l" != "en"; then \
> > +	    rm -f $(MANDIR)/$$l/man1/mplayer.1 \
> > +	    $(MANDIR)/$$l/man1/mencoder.1   \
> > +	    $(MANDIR)/$$l/man1/gmplayer.1 ; \
> 
> OK, this is really just a nit, but I would indent this properly as
well,
> so either
> 
>     -rm -f $(MANDIR)/$$l/man1/mplayer.1 $(MANDIR)/$$l/man1/mencoder.1
\
>            $(MANDIR)/$$l/man1/gmplayer.1 \
> 
> or if you prefer
> 
>     -rm -f $(MANDIR)/$$l/man1/mplayer.1  \
>            $(MANDIR)/$$l/man1/mencoder.1 \
>            $(MANDIR)/$$l/man1/gmplayer.1 \
> 
> That should be more readable.

okee :-)