This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH RFA: Correct toplevel configury
- From: Ian Lance Taylor <iant at google dot com>
- To: gcc-patches at gcc dot gnu dot org, Paolo Bonzini <bonzini at gnu dot org>, "ralf.wildenhues" <ralf dot wildenhues at gmx dot de>
- Date: Tue, 26 Jul 2011 11:00:36 -0700
- Subject: Re: PATCH RFA: Correct toplevel configury
- References: <mcrbowmvnu5.fsf@coign.corp.google.com>
Ping.
On Thu, Jul 21, 2011 at 11:20 PM, Ian Lance Taylor <iant@google.com> wrote:
> One of my recent patches broke the toplevel configury. ?I moved a test
> of $configdirs to a point before nonexistent directories have been
> removed from configdirs. ?The test was for whether gcc is being
> configured. ?The test is fine in the gcc repository, but not in the src
> repository.
>
> This patch fixes the problem. ?If the gcc directory exists, we assume
> that we are going to build it. ?This only matters for setting the
> default value for --enable-bootstrap.
>
> Bootstrapped on x86_64-unknown-linux-gnu. ?OK for mainline?
>
> Ian
>
>
> 2011-07-21 ?Ian Lance Taylor ?<iant@google.com>
>
> ? ? ? ?* configure.ac: Set have_compiler based on whether gcc directory
> ? ? ? ?exists, rather than on whether gcc is in configdirs.
>
>
>
Index: configure.ac
===================================================================
--- configure.ac (revision 176515)
+++ configure.ac (working copy)
@@ -1139,10 +1139,11 @@ AC_ARG_ENABLE([bootstrap],
enable_bootstrap=default)
# Issue errors and warnings for invalid/strange bootstrap combinations.
-case "$configdirs" in
- *gcc*) have_compiler=yes ;;
- *) have_compiler=no ;;
-esac
+if test -r $srcdir/gcc/configure; then
+ have_compiler=yes
+else
+ have_compiler=no ;;
+fi
case "$have_compiler:$host:$target:$enable_bootstrap" in
*:*:*:no) ;;