[RFA]: configure.in: Fix Tcl/Tk tests on Cygwin

Corinna Vinschen vinschen@redhat.com
Tue Jun 8 15:39:00 GMT 2004


Hi,

the below patch fixes a build problem on Cygwin.  For some reason a
variable $configdir is always set to "unix".  The later tests for
tclConfig.sh and itkConfig.sh files are only testing in the unix subdir
of parallel tcl and tk build directories.  Fine on Unix. but these files
are in the "win" subdir on Cygwin.  Due to this bug, it's for example
impossible to build an i686-pc-cygwin-x-v850-elf gdb and insight.

The below patch fixes that and additionally uses $configdir now when
setting the paths for $tcldir and $tkdir a few lines later to simplify
the code.

Ok to check in?


Corinna


	* configure.in: Set $configdir to the right OS specific value.
	Use value when setting $tcldir and $tkdir.
	* configure: Regenerate.

Index: configure.in
===================================================================
RCS file: /cvs/src/src/gdb/configure.in,v
retrieving revision 1.155
diff -u -p -r1.155 configure.in
--- configure.in	23 May 2004 09:23:17 -0000	1.155
+++ configure.in	8 Jun 2004 15:23:03 -0000
@@ -1134,7 +1134,14 @@ WIN32LDAPP=
 AC_SUBST(WIN32LIBS)
 AC_SUBST(WIN32LDAPP)
 
-configdir="unix"
+case "${host}" in
+*-*-cygwin*)
+    configdir="win"
+    ;;
+*)
+    configdir="unix"
+    ;;
+esac
 
 GDBTKLIBS=
 if test "${enable_gdbtk}" = "yes"; then
@@ -1153,14 +1160,7 @@ if test "${enable_gdbtk}" = "yes"; then
 
 	# now look for Tcl library stuff
 
-	case "${host}" in
-	*-*-cygwin*)
-	    tcldir=../tcl/win/
-	    ;;
-	*)
-	    tcldir=../tcl/unix/
-	    ;;
-	esac
+	tcldir="../tcl/${configdir}/"
 
 	TCL_DEPS="${tcldir}${TCL_LIB_FILE}"
 
@@ -1176,14 +1176,7 @@ if test "${enable_gdbtk}" = "yes"; then
 
 	   # now look for Tk library stuff
 
-	   case "${host}" in
-	   *-*-cygwin*)
-	       tkdir=../tk/win/
-	       ;;
-	   *)
-	       tkdir=../tk/unix/
-	       ;;
-	   esac
+	   tkdir="../tk/${configdir}/"
 
 	   TK_DEPS="${tkdir}${TK_LIB_FILE}"
 

-- 
Corinna Vinschen
Cygwin Co-Project Leader
Red Hat, Inc.



More information about the Gdb-patches mailing list