* [patch] remove redundant tests
@ 2002-06-22 7:54 Nathanael Nerode
2002-06-22 9:19 ` Matt Kraai
0 siblings, 1 reply; 2+ messages in thread
From: Nathanael Nerode @ 2002-06-22 7:54 UTC (permalink / raw)
To: gcc-patches; +Cc: gdb-patches, binutils, dj
This removes some (now) redundant tests from toplevel configure.in.
2002-06-22 Nathanael Nerode <neroden@twcny.rr.com>
* configure.in: Remove some redundant tests.
Index: configure.in
===================================================================
RCS file: /cvsroot/gcc/gcc/configure.in,v
retrieving revision 1.162
diff -u -r1.162 configure.in
--- configure.in 21 Jun 2002 22:20:17 -0000 1.162
+++ configure.in 22 Jun 2002 14:52:31 -0000
@@ -1266,24 +1266,22 @@
# want to do that, then you should use the --without-gnu-as and
# --without-gnu-ld options for the configure script.
-if test x${use_gnu_as} = x ; then
- if test x${with_gnu_as} != xno && echo " ${configdirs} " | grep " gas " > /dev/null 2>&1 && test -d ${srcdir}/gas ; then
- with_gnu_as=yes
- withoptions="$withoptions --with-gnu-as"
- fi
+if test x${use_gnu_as} = x &&
+ echo " ${configdirs} " | grep " gas " > /dev/null 2>&1 ; then
+ with_gnu_as=yes
+ withoptions="$withoptions --with-gnu-as"
fi
-if test x${use_gnu_ld} = x ; then
- if test x${with_gnu_ld} != xno && echo " ${configdirs} " | grep " ld " > /dev/null 2>&1 && test -d ${srcdir}/ld ; then
- with_gnu_ld=yes
- withoptions="$withoptions --with-gnu-ld"
- fi
+if test x${use_gnu_ld} = x &&
+ echo " ${configdirs} " | grep " ld " > /dev/null 2>&1 ; then
+ with_gnu_ld=yes
+ withoptions="$withoptions --with-gnu-ld"
fi
# If using newlib, add --with-newlib to the withoptions so that gcc/configure
# can detect this case.
-if test x${with_newlib} != xno && echo " ${target_configdirs} " | grep " target-newlib " > /dev/null 2>&1 && test -d ${srcdir}/newlib ; then
+if test x${with_newlib} != xno && echo " ${target_configdirs} " | grep " target-newlib " > /dev/null 2>&1 ; then
with_newlib=yes
withoptions="$withoptions --with-newlib"
fi
@@ -1517,8 +1515,7 @@
fi
if test "x${use_gnu_ld}" = x && test "x${with_gnu_ld}" != xno &&
- echo " ${configdirs} " | grep " ld " > /dev/null &&
- test -d ${srcdir}/ld; then
+ echo " ${configdirs} " | grep " ld " > /dev/null ; then
# Arrange for us to find uninstalled linker scripts.
FLAGS_FOR_TARGET=$FLAGS_FOR_TARGET' -L$$r/ld'
fi
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [patch] remove redundant tests
2002-06-22 7:54 [patch] remove redundant tests Nathanael Nerode
@ 2002-06-22 9:19 ` Matt Kraai
0 siblings, 0 replies; 2+ messages in thread
From: Matt Kraai @ 2002-06-22 9:19 UTC (permalink / raw)
To: Nathanael Nerode; +Cc: gcc-patches, gdb-patches, binutils, dj
[-- Attachment #1: Type: text/plain, Size: 722 bytes --]
On Sat, Jun 22, 2002 at 10:53:47AM -0400, Nathanael Nerode wrote:
> This removes some (now) redundant tests from toplevel configure.in.
>
> 2002-06-22 Nathanael Nerode <neroden@twcny.rr.com>
> * configure.in: Remove some redundant tests.
>
> Index: configure.in
> ===================================================================
> RCS file: /cvsroot/gcc/gcc/configure.in,v
> retrieving revision 1.162
> diff -u -r1.162 configure.in
> +++ configure.in 22 Jun 2002 14:52:31 -0000
[snip]
> @@ -1517,8 +1515,7 @@
> fi
>
> if test "x${use_gnu_ld}" = x && test "x${with_gnu_ld}" != xno &&
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Isn't this test also redundant?
Matt
[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2002-06-22 16:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-06-22 7:54 [patch] remove redundant tests Nathanael Nerode
2002-06-22 9:19 ` Matt Kraai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox