This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Fix expect build under cygwin + Tcl/Tk 8.3


Hello.

I am just checking back on a patch I sent in:

http://sources.redhat.com/ml/gdb-patches/2002-05/msg00621.html

It got one response:

http://sources.redhat.com/ml/gdb-patches/2002-05/msg00654.html

> >The current CVS version of expect does not build correctly under
> >Cygwin.  The root of the problem is that expect is searching for a
> >tclConfig.sh file in tcl/unix when it should be looking in tcl/win.
> >The following patch fixes the problem by updating the m4 macros used by
> >the expect build.

> AFAIK, expect shouldn't be using the win version of anything.  It is
> supposed to be using a UNIX-only implementation of tcl.
> cgf

I guess it would have helped if I had included the more detailed description
from my earlier post to the insight list:

"This patch will get the configure script to pickup the tclConfig.sh
out of the tcl/win subdirectory instead of the fake one Tcl is
currently generating in the tcl/unix subdirectory. One possible
objection to this patch would be that a tclConfig.sh should be
created in the tcl/cygwin subdirectory and then found there before
checking tcl/win. That may be the "more correct" thing to do, but
this patch just tries to do the "less wrong" thing since it is
the minimal change that should get everything working again."

The other way to fix the problem would be to create a new tclConfig.sh
file in the tcl/cygwin sub and populate it with needed values and then
add support for this new subdirectory to any of the Tcl extensions
that needed it. That could end up being quite a bit of work. In the
end, I don't think it would matter since expect seems to ignore the
contents of the tclConfig.sh file, it just wants it to be there. Also,
the configure scripts in the expect subdir seem to be way out of sync
with the configures in the other extensions, so it seems like a big
rewrite would be needed to actually use the contents of the tclConfig.sh
file.

So, my point is that I think my patch should be applied since it fixes
the problem. It might be the most minimal fix, but I don't see how that
is any worse than what we have now. Currently, the Windows version
of Tcl/Tk/Itcl will not build because Tcl incorrectly emits a unix/tclConfig.sh
file just to make expect happy (and that hoses Itcl).

cheers
Mo DeJong


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]