[patch] toplevel configure.in: some skipdirs->noconfigdirs

[patch] toplevel configure.in: some skipdirs->noconfigdirs

[Nathanael Nerode]

For some reason these targets were adding things to skipdirs rather than
noconfigdirs.  The only practical difference is that the warning message 
"This configuration is not supported in the following subdirectories" 
would not show up.

It's simpler for me to clean up the newlib-related code if they use
noconfigdirs the way every other target does.

So unless someone comes up with a good reason they should behave differently,
I suggest this patch.

2002-07-12  Nathanael Nerode  <neroden@gcc.gnu.org>

	* configure.in (*-*-netbsd*): Use noconfigdirs, not skipdirs.
	* configure.in (powerpc*-*-winnt*): Ditto.
	* configure.in (powerpc*-*-pe*): Ditto.
	* configure.in (ppc*-*-pe): Ditto.


Index: configure.in
===================================================================
RCS file: /cvs/gcc/gcc/configure.in,v
retrieving revision 1.173
diff -u -r1.173 configure.in
--- configure.in	11 Jul 2002 19:54:04 -0000	1.173
+++ configure.in	12 Jul 2002 19:35:42 -0000
@@ -277,7 +277,7 @@
     ;;
   *-*-netbsd*)
     # Skip some stuff on all NetBSD configurations.
-    skipdirs="$skipdirs target-newlib target-libiberty target-libgloss"
+    noconfigdirs="$noconfigdirs target-newlib target-libiberty target-libgloss"
 
     # Skip some stuff that's unsupported on some NetBSD configurations.
     case "${target}" in
@@ -333,7 +333,7 @@
     noconfigdirs="$noconfigdirs expect dejagnu"
     # the C++ libraries don't build on top of CE's C libraries
     noconfigdirs="$noconfigdirs ${libstdcxx_version}"
-    skipdirs="$skipdirs target-newlib"
+    noconfigdirs="$noconfigdirs target-newlib"
     case "${host}" in
       *-*-cygwin*) ;; # keep gdb and readline
       *) noconfigdirs="$noconfigdirs gdb readline ${libstdcxx_version}"

Re: [patch] toplevel configure.in: some skipdirs->noconfigdirs

[Matt Kraai]

[-- Attachment #1: Type: text/plain, Size: 1050 bytes --]

On Fri, Jul 12, 2002 at 03:44:45PM -0400, Nathanael Nerode wrote:
> 	* configure.in (powerpc*-*-winnt*): Ditto.
> 	* configure.in (powerpc*-*-pe*): Ditto.
> 	* configure.in (ppc*-*-pe): Ditto.
> 
> 
> Index: configure.in
> ===================================================================
> RCS file: /cvs/gcc/gcc/configure.in,v
> retrieving revision 1.173
> diff -u -r1.173 configure.in
> +++ configure.in	12 Jul 2002 19:35:42 -0000
> @@ -333,7 +333,7 @@
>      noconfigdirs="$noconfigdirs expect dejagnu"
>      # the C++ libraries don't build on top of CE's C libraries
>      noconfigdirs="$noconfigdirs ${libstdcxx_version}"
> -    skipdirs="$skipdirs target-newlib"
> +    noconfigdirs="$noconfigdirs target-newlib"
>      case "${host}" in
>        *-*-cygwin*) ;; # keep gdb and readline
>        *) noconfigdirs="$noconfigdirs gdb readline ${libstdcxx_version}"

The ChangeLog doesn't match: the patch modifies sh*-*-pe,
mips*-*-pe, and *arm-wince-pe, not powerpc*-*-winnt*,
powerpc*-*-pe*, and ppc*-*-pe.

Matt

[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]