This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Minor simplification for gcc/Makefile.in
- To: d dot love at dl dot ac dot uk
- Subject: Re: Minor simplification for gcc/Makefile.in
- From: Manfred Hollstein <manfred at s-direktnet dot de>
- Date: Thu, 2 Jul 1998 07:44:37 +0200 (MET DST)
- Cc: law at cygnus dot com, pfeifer at dbai dot tuwien dot ac dot at, egcs-patches at cygnus dot com
- References: <20222.899271989@hurl.cygnus.com> <rzqk95xh5pu.fsf@djlvig.dl.ac.uk>
- Reply-To: manfred at s-direktnet dot de, Manfred dot Hollstein at ks dot sel dot alcatel dot de
On nil, 1 July 1998, 17:54:37, d.love@dl.ac.uk wrote:
> >>>>> "Jeff" == Jeffrey A Law <law@hurl.cygnus.com> writes:
>
> Jeff> I leave this in Dave's hands. :-)
>
> A `courageous' decision!
>
> It looks sensible to me and I was probably confused. However, why is
> the sedantry necessary in the previous loop, doing the INSTALL_DATA?
> Isn't it actually redundant in both places below rather than just the
> lower one?
>
> for f in cpp.info* gcc.info*; do \
> realfile=`echo $$f | sed -e 's|.*/\([^/]*\)$$|\1|'`; \
> $(INSTALL_DATA) $$f $(infodir)/$$realfile; \
> done
> if $(SHELL) -c 'install-info --version' >/dev/null 2>&1; then \
> for f in cpp.info gcc.info; do \
> realfile=`echo $$f | sed -e 's|.*/\([^/]*\)$$|\1|'`; \
> install-info --dir-file=$(infodir)/dir $(infodir)/$$realfile; \
> done; \
> else true; fi;
>
Yes, I guess it's just a leftover from the time when the *.info*
files could have been either in the builddir or in srcdir, hence a
directory prefix needed to be removed. Now that it's clear that these
files will always be in srcdir, the realfile stuff can be completely
removed.
manfred