Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [toplevel patch] Remove future variable name conflict in configure.in
@ 2003-06-26 17:24 Nathanael Nerode
  2003-06-26 18:38 ` DJ Delorie
  0 siblings, 1 reply; 2+ messages in thread
From: Nathanael Nerode @ 2003-06-26 17:24 UTC (permalink / raw)
  To: gcc-patches, binutils, gdb-patches

This is the only variable name conflict with config-lang.in fragments.

It turns out that target_libs isn't used after its late assignment
from the config-lang.in fragments, which makes the assignment worthless,
so I just deleted it.

Tested on i686-pc-linux-gnu.

This is the precursor to simply sourcing the config-lang.in fragments
at the top level rather than running them through ungodly sed expressions.

	* configure.in: Rename 'target_libs' to 'target_libraries'.
	Remove useless reference to 'target_libs'.
	* configure: Regenerate.

Index: configure.in
===================================================================
RCS file: /cvs/gcc/gcc/configure.in,v
retrieving revision 1.241
diff -u -r1.241 configure.in
--- configure.in	23 Jun 2003 00:42:54 -0000	1.241
+++ configure.in	26 Jun 2003 17:19:29 -0000
@@ -136,7 +136,7 @@
 # these libraries are built for the target environment, and are built after
 # the host libraries and the host tools (which may be a cross compiler)
 #
-target_libs="target-libiberty \
+target_libraries="target-libiberty \
 		target-libgloss \
 		target-newlib \
 		target-libstdc++-v3 \
@@ -144,10 +144,10 @@
 		${libgcj} \
 		target-libobjc"
 
-# these tools are built using the target libs, and are intended to run only
-# in the target environment
+# these tools are built using the target libraries, and are intended to
+# run only in the target environment
 #
-# note: any program that *uses* libraries that are in the "target_libs"
+# note: any program that *uses* libraries that are in the "target_libraries"
 # list belongs in this list.  those programs are also very likely
 # candidates for the "native_only" list which follows
 #
@@ -162,7 +162,7 @@
 ## ${target_configdirs} is directories we build using the target tools.
 #
 configdirs=`echo ${host_libs} ${host_tools}`
-target_configdirs=`echo ${target_libs} ${target_tools}`
+target_configdirs=`echo ${target_libraries} ${target_tools}`
 
 # Only make build modules if build != host.
 # This should be done more generally, but at the moment it doesn't matter.
@@ -805,7 +805,7 @@
         *) add_this_lang=no ;;
       esac
       if test x"${add_this_lang}" = xyes; then
-        eval target_libs='"$target_libs "'\"$this_lang_libs\"
+        :
       else
         eval noconfigdirs='"$noconfigdirs "'\"$this_lang_libs $this_lang_dirs\"
       fi



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2003-06-26 18:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-06-26 17:24 [toplevel patch] Remove future variable name conflict in configure.in Nathanael Nerode
2003-06-26 18:38 ` DJ Delorie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox