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
prev 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