Date: Tue, 11 Jun 2002 14:34:03 +0100
From: "Richard Dawe" <rich AT phekda DOT freeserve DOT co DOT uk>
Sender: rich AT phekda DOT freeserve DOT co DOT uk
To: djgpp-workers AT delorie DOT com
X-Mailer: Emacs 21.3.50 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.6
Subject: automake 1.5 port: Bug in texinfos.am
Message-Id: <E17HlgW-0000gw-00@phekda.freeserve.co.uk>
Reply-To: djgpp-workers AT delorie DOT com

Hello.

There appears to be a bug in texinfos.am. The problem is underquoting
of $(PATH_SEPARATOR), so commands like:

    CDPATH=; && cd $$d && ...

are executed. Below is a patch to fix the underquoting.

doc/Makefile{,.in} from Fileutils 4.1 are bitten by this bug.
If I regenerate the Makefile with a patched automake, the problem is gone.

Bye, Rich =]

--- share/automake/am/texinfos.am.orig	2002-06-11 14:06:34.000000000 +0000
+++ share/automake/am/texinfos.am	2002-06-11 14:06:44.000000000 +0000
@@ -113,7 +113,7 @@ install-info-am: $(INFO_DEPS)
 ?!CYGNUS?	  d=$(srcdir); \
 ## We use these strange circumlocutions because we want the "ifile" to
 ## be relative, for the install.
-	  for ifile in `CDPATH=$(PATH_SEPARATOR) && cd $$d && echo $$file $$file-[0-9] $$file-[0-9][0-9]`; do \
+	  for ifile in `CDPATH="$(PATH_SEPARATOR)" && cd $$d && echo $$file $$file-[0-9] $$file-[0-9][0-9]`; do \
 	    if test -f $$d/$$ifile; then \
 	      echo " $(INSTALL_DATA) $$d/$$ifile $(DESTDIR)$(infodir)/$$ifile"; \
 	      $(INSTALL_DATA) $$d/$$ifile $(DESTDIR)$(infodir)/$$ifile; \
@@ -197,7 +197,7 @@ dist-info: $(INFO_DEPS)
 ## other modes, allow only source dir.
 ?!CYGNUS?	  d=$(srcdir); \
 ?CYGNUS?	  if test -f $$base; then d=.; else d=$(srcdir); fi; \
-	  for file in `CDPATH=$(PATH_SEPARATOR) && cd $$d && eval echo $$base*`; do \
+	  for file in `CDPATH="$(PATH_SEPARATOR)" && cd $$d && eval echo $$base*`; do \
 	    test -f $(distdir)/$$file \
 	    || cp -p $$d/$$file $(distdir)/$$file; \
 	  done; \