Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Alexandre Oliva <aoliva@redhat.com>
To: gdb-patches@sources.redhat.com
Subject: Re: patch for expect to build on Cygwin
Date: Tue, 15 Jul 2003 18:37:00 -0000	[thread overview]
Message-ID: <or7k6jab6i.fsf@free.redhat.lsd.ic.unicamp.br> (raw)
In-Reply-To: <orsmp89axf.fsf@free.redhat.lsd.ic.unicamp.br>

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

On Jul 14, 2003, Alexandre Oliva <aoliva@redhat.com> wrote:

> The problem was that, on Cygwin, expect/configure couldn't find
> the tk config file in the build tree

It would also find the wrong tcl config file: tcl/cygwin/tclConfig.sh
is full of variables that aren't substituted properly, and itcl/tcl.m4
indicates we should be using tcl/win/tclConfig.sh instead, so I'm
changing expect to that effect.  In fact, if you're unlucky to have
expect configured before itcl, it would pollute config.cache to the
point that itcl wouldn't build.

What I don't understand is why itcl has outdated code in its
aclocal.m4.  That code is not used, it is tcl.m4 that is used in
iwidgets.

Anyway, I'm going ahead and checking this in as well, which should get
s.r.c gdb/insight/etc back to a buildable state on Cygwin regardless
of the order in which expect and itcl are configured (or if
config.cache is disabled)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: expect-cygwin-tcl.patch --]
[-- Type: text/x-patch, Size: 804 bytes --]

Index: expect/ChangeLog
from  Alexandre Oliva  <aoliva@redhat.com>

	* aclocal.m4 (CY_AC_PATH_TCLCONFIG): Use .../tcl/win, not cygwin.
	* configure, Dbgconfigure, testsuite/configure: Rebuilt.

Index: expect/aclocal.m4
===================================================================
RCS file: /cvs/src/src/expect/aclocal.m4,v
retrieving revision 1.4
diff -u -p -r1.4 aclocal.m4
--- expect/aclocal.m4 14 Jul 2003 19:16:43 -0000 1.4
+++ expect/aclocal.m4 15 Jul 2003 18:36:58 -0000
@@ -201,8 +201,8 @@ changequote(,)
         ac_cv_c_tclconfig=`(cd $i/unix; pwd)`
 	break
       fi
-      if test -f "$i/cygwin/tclConfig.sh" ; then
-        ac_cv_c_tclconfig=`(cd $i/cygwin; pwd)`
+      if test -f "$i/win/tclConfig.sh" ; then
+        ac_cv_c_tclconfig=`(cd $i/win; pwd)`
 	break
       fi
     done

[-- Attachment #3: Type: text/plain, Size: 289 bytes --]


-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer

      parent reply	other threads:[~2003-07-15 18:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-07-14 19:16 Alexandre Oliva
2003-07-14 19:41 ` patch for expect to build on cygwin Christopher Faylor
2003-07-15  3:34   ` Alexandre Oliva
2003-07-15 18:37 ` Alexandre Oliva [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=or7k6jab6i.fsf@free.redhat.lsd.ic.unicamp.br \
    --to=aoliva@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox