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: law at cygnus dot com
- Subject: Re: Minor simplification for gcc/Makefile.in
- From: Gerald Pfeifer <pfeifer at dbai dot tuwien dot ac dot at>
- Date: Thu, 2 Jul 1998 14:17:20 +0200 (MET DST)
- cc: Dave Love <d dot love at dl dot ac dot uk>, Manfred Hollstein <manfred at s-direktnet dot de>, Manfred dot Hollstein at ks dot sel dot alcatel dot de, egcs-patches at cygnus dot com
On Wed, 1 Jul 1998, Jeffrey A Law wrote:
> Dave Love <d.love@dl.ac.uk> wrote:
>> It looks sensible to me and I was probably confused. However, why is
>> the sedantry necessary in the previous loop, doing the INSTALL_DATA?
> I believe it's redundant in both locations now.
and Manfred also agreed:
> Now that it's clear [...], the realfile stuff can be completely
> removed.
Indeed. Please find below an updated patch along these lines (which
happens to be my first non-doc egcs patch, btw :-) ).
Gerald
Thu Jul 2 14:10:14 1998 Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at>
* Makefile.in (install-info): No need to strip directory
information from a filename if there cannot be any such
information.
Index: Makefile.in
===================================================================
RCS file: /egcs/carton/cvsfiles/egcs/gcc/Makefile.in,v
retrieving revision 1.145
diff -r1.145 Makefile.in
2381,2382c2381
< realfile=`echo $$f | sed -e 's|.*/\([^/]*\)$$|\1|'`; \
< $(INSTALL_DATA) $$f $(infodir)/$$realfile; \
---
> $(INSTALL_DATA) $$f $(infodir)/$$f; \
2386,2387c2385
< realfile=`echo $$f | sed -e 's|.*/\([^/]*\)$$|\1|'`; \
< install-info --dir-file=$(infodir)/dir $(infodir)/$$realfile; \
---
> install-info --dir-file=$(infodir)/dir $(infodir)/$$f; \