This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[patch] Configure test failure - too many arguments


Hi,

Please find attached a patch to remove the configure error:

gcc/configure: test: too many arguments

Which is caused by a sed statement error when checking the version number of binutils.

I have tested by compiling from scratch with and without the patch, and ensuring that the new sed command works for both the old version string and the new version string of GAS. *note* I haven't done any regression testing with this patch as I don't have regression testing setup yet - but I don't see this adversley affecting anything.

Could someone please check and apply for me?

Changelog (${srcdir}/gcc/Changelog):

2007-08-01 Brian Sidebotham <brian.sidebotham@gmail.com>

	* configure.ac (leb128): Modify sed statement to work with newly formatted
	binutils version string.
	* configure: Regenerate

Best Regards,

Brian Sidebotham.
--- gcc/gcc/configure.ac	Mon Jul 30 10:18:27 2007
+++ gcc.patched/gcc/configure.ac	Mon Jul 30 10:45:48 2007
@@ -2105,7 +2105,7 @@ L2:],
   as_ver=`$gcc_cv_as --version 2>/dev/null | sed 1q`
   if echo "$as_ver" | grep GNU > /dev/null; then
 changequote(,)dnl
-    as_ver=`echo $as_ver | sed -e 's/GNU assembler \([0-9.][0-9.]*\).*/\1/'`
+    as_ver=`echo $as_ver | sed -e 's/GNU assembler \|GNU assembler (GNU Binutils) \([0-9.][0-9.]*\).*/\1/'`
     as_major=`echo $as_ver | sed 's/\..*//'`
     as_minor=`echo $as_ver | sed 's/[^.]*\.\([0-9]*\).*/\1/'`
 changequote([,])dnl

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]