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

[Edwin Bates <ebates at Ebix dot com>]

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

From: Edwin Bates <ebates@Ebix.com>
To: 'Zack Weinberg' <zack@codesourcery.com>, Craig Rodrigues
	 <rodrigc@mediaone.net>
Cc: Carlo Wood <carlo@alinoe.com>, 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: Tue, 29 Jan 2002 10:02:58 -0500

 Thank you all for your input.  Carlo was correct: it's inserting a newline,
 but only after the first directory (cp), and, as Craig said, it shouldn't be
 doing it.  (What are you doing, AIX?)  Here are the results of the test:
 
 ------------------------ Begin test results ----------------------------
 checking assembler eh_frame optimization... no
 checking assembler dwarf2 debug_line support... no
 subdirs=
 + sed -n -e s,^language=['"]\(.*\)["'].*$,\1,p -e s,^language=\([^
 ]*\).*$,\1,p ./cp/config-lang.in 
 lang_alias=c++
 + sed -n -e s,^target_libs=['"]\(.*\)["'].*$,\1,p -e s,^target_libs=\([^
 ]*\).*$,\1,p ./cp/config-lang.in 
 this_lang_libs=${libstdcxx_version}
 + sed -n -e s,^build_by_default=['"]\(.*\)["'].*$,\1,p -e
 s,^build_by_default=\([^ 	]*\).*$,\1,p ./cp/config-lang.in 
 build_by_default=
 + test xc++ = x 
 add_this_lang=yes
 + test xyes = xyes 
 + echo ./cp/config-lang.in 
 + sed -e s,^.*/\([^/]*\)/config-lang.in$,\1, 
 subdirs= cp
 
 + sed -n -e s,^language=['"]\(.*\)["'].*$,\1,p -e s,^language=\([^
 ]*\).*$,\1,p ./f/config-lang.in 
 lang_alias=f77
 + sed -n -e s,^target_libs=['"]\(.*\)["'].*$,\1,p -e s,^target_libs=\([^
 ]*\).*$,\1,p ./f/config-lang.in 
 this_lang_libs=target-libf2c
 + sed -n -e s,^build_by_default=['"]\(.*\)["'].*$,\1,p -e
 s,^build_by_default=\([^ 	]*\).*$,\1,p ./f/config-lang.in 
 build_by_default=
 + test xf77 = x 
 add_this_lang=yes
 + test xyes = xyes 
 + echo ./f/config-lang.in 
 + sed -e s,^.*/\([^/]*\)/config-lang.in$,\1, 
 subdirs= cp
  f
 + sed -n -e s,^language=['"]\(.*\)["'].*$,\1,p -e s,^language=\([^
 ]*\).*$,\1,p ./java/config-lang.in 
 lang_alias=java
 + sed -n -e s,^target_libs=['"]\(.*\)["'].*$,\1,p -e s,^target_libs=\([^
 ]*\).*$,\1,p ./java/config-lang.in 
 this_lang_libs=${libgcj_saved}
 + sed -n -e s,^build_by_default=['"]\(.*\)["'].*$,\1,p -e
 s,^build_by_default=\([^ 	]*\).*$,\1,p ./java/config-lang.in 
 build_by_default=
 + test xjava = x 
 add_this_lang=yes
 + test xyes = xyes 
 + sed -e s,^.*/\([^/]*\)/config-lang.in$,\1, 
 + echo ./java/config-lang.in 
 subdirs= cp
  f java
 + sed -n -e s,^language=['"]\(.*\)["'].*$,\1,p -e s,^language=\([^
 ]*\).*$,\1,p ./objc/config-lang.in 
 lang_alias=objc
 + sed -n -e s,^target_libs=['"]\(.*\)["'].*$,\1,p -e s,^target_libs=\([^
 ]*\).*$,\1,p ./objc/config-lang.in 
 this_lang_libs=target-libobjc
 + sed -n -e s,^build_by_default=['"]\(.*\)["'].*$,\1,p -e
 s,^build_by_default=\([^ 	]*\).*$,\1,p ./objc/config-lang.in 
 build_by_default=
 + test xobjc = x 
 add_this_lang=yes
 + test xyes = xyes 
 + echo ./objc/config-lang.in 
 + sed -e s,^.*/\([^/]*\)/config-lang.in$,\1, 
 subdirs= cp
  f java objc
 + set +x
 ------------------------ End test results ----------------------------
 
 Yes, it appears to be an AIX anomaly.
 
 -- Ed
 
  -----Original Message-----
 From: 	Zack Weinberg [mailto:zack@codesourcery.com] 
 Sent:	Monday, January 28, 2002 8:09 PM
 To:	Craig Rodrigues
 Cc:	Carlo Wood; Edwin Bates; 'rodrigc@gcc.gnu.org';
 'gcc-bugs@gcc.gnu.org'; 'gcc-prs@gcc.gnu.org'; 'nobody@gcc.gnu.org';
 'gcc-gnats@gcc.gnu.org'; Steve Moore
 Subject:	Re: other/5515: ./configure fails: sed: Function
 s%@subdirs@% cp cannot be parsed.
 
 On Mon, Jan 28, 2002 at 10:54:29PM -0500, Craig Rodrigues wrote:
 > 
 > OK.  Zack Weinberg applied a patch which does a similar
 > thing to subdirs:
 
 That patch has nothing to do with this problem.
 
 > Edwin, can you try changing your fix to:
 > subdirs='$subdirs'
 
 The line that does that, from my patch, is executed in an unusual
 context; it's dead wrong anywhere else.
 
 What I want to know is how we got newlines in $subdirs in the first
 place.  That should be impossible.  Edwin, would you mind applying
 this patch to configure.in, regenerating configure, running it, and
 reporting the output (just the lines beginning with a plus sign should
 be enough)?
 
 zw
 
 ===================================================================
 Index: configure.in
 --- configure.in	2002/01/10 22:21:36	1.576
 +++ configure.in	2002/01/29 04:07:52
 @@ -2032,6 +2032,7 @@ changequote([,])dnl
     esac
  done
  
 +set -x
  subdirs=
  for lang in ${srcdir}/*/config-lang.in ..
  do
 @@ -2072,6 +2073,7 @@ changequote(,)dnl
  changequote([,])dnl
  	esac
  done
 +set +x
  
  # Make gthr-default.h if we have a thread file.
  gthread_flags=