Re: [PATCH] COPYING and copying.c

[Andrew Cagney <ac131313 at cygnus dot com>]

Eli Zaretskii wrote:
> 
> > > +# When DJGPP Make runs on MS-DOS, it downcases all file names, so
> > > +# it doesn't find COPYING, and wants to make it...
> > > +copying.txt:
> > > +       test -f copying.txt || ln COPYING copying.txt
> >
> > This bit won't work when building in a separate directory.
> > I'm also not sure about the ``ln'' command. configure.in contains stuff
> > to set ``LN_S''.
> 
> Then how about the following changes?  They seem to work for me.
> 
> 2000-03-03  Eli Zaretskii  <eliz@is.elta.co.il>
> 
>         * Makefile.in (copying.c): Depend on copying.txt, not COPYING.
>         (copying.txt): New target, a link to COPYING.
>         (LN_S): Define.

(covers eyes, looks other way ... :-)
I don't have any problems with this.

> +LN_S = @LN_S@

Not needed, recently added.

> +         { test "$$LN_S" = "ln -s" && \
> +           ln -s $(srcdir)/COPYING copying.txt; } || \
> +         cp -p $(srcdir)/COPYING copying.txt 2>/dev/null || \
> +         ln $(srcdir)/COPYING copying.txt 2>/dev/null || \
> +         cp $(srcdir)/COPYING copying.txt

People tend to use ``( ... )'' instead of  ``{ ... }''.

	Andrew