Re: other/5515: ./configure fails: sed: Function s%@subdirs@% cp cannot be parsed.

[Craig Rodrigues <rodrigc@mediaone.net>]

The following reply was made to PR other/5515; it has been noted by GNATS.

From: Craig Rodrigues <rodrigc@mediaone.net>
To: Carlo Wood <carlo@alinoe.com>
Cc: Edwin Bates <ebates@Ebix.com>,
   "'rodrigc@gcc.gnu.org'" <rodrigc@gcc.gnu.org>,
   "'gcc-bugs@gcc.gnu.org'" <gcc-bugs@gcc.gnu.org>,
   "'gcc-prs@gcc.gnu.org'" <gcc-prs@gcc.gnu.org>,
   "'nobody@gcc.gnu.org'" <nobody@gcc.gnu.org>,
   "'gcc-gnats@gcc.gnu.org'" <gcc-gnats@gcc.gnu.org>,
   Steve Moore <SMoore@Ebix.com>
Subject: Re: other/5515: ./configure fails: sed: Function s%@subdirs@% cp cannot be parsed.
Date: Mon, 28 Jan 2002 22:54:29 -0500

 On Tue, Jan 29, 2002 at 04:00:09AM +0100, Carlo Wood wrote:
 > <guru mode on>
 > Maybe I can shine a light on this.
 > One of the reasons that configure fails to substitue
 > is when
 > 1) A substitution contains new-lines
 > 2) The substitution is exactly overlapping
 >    the border at which the substitution command
 >    is cut into multiple sed commands.
 > 
 > Obviously this a coincidence, but in fact only
 > truely safe when none of the substitution contain
 > new-lines.
 > 
 > What the patch below might change is replacing
 > new-lines in ${subdirs} into spaces.
 > </guru mode off>
 
 OK.  Zack Weinberg applied a patch which does a similar
 thing to subdirs:
 http://subversions.gnu.org/cgi-bin/viewcvs/gcc/gcc/gcc/configure.in.diff?r1=text&tr1=1.573&r2=text&tr2=1.574&diff_format=u
 
 and:
 http://subversions.gnu.org/cgi-bin/viewcvs/gcc/gcc/gcc/configure.diff?r1=text&tr1=1.583&r2=text&tr2=1.584&diff_format=u
 
 So maybe that is the right thing to do here?
 
 Edwin, can you try changing your fix to:
 subdirs='$subdirs'
 
 and see if that works?
 
 -- 
 Craig Rodrigues        
 http://www.gis.net/~craigr    
 rodrigc@mediaone.net