public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: Ian Roxborough <irox@evilpeople.com>
To: Fernando Nasser <fnasser@redhat.com>
Cc: fnasser@cygnus.com, keiths@cygnus.com, insight@sources.redhat.com
Subject: Re: Tcl/Tk 8.3 upgrade...
Date: Fri, 15 Jun 2001 10:59:00 -0000	[thread overview]
Message-ID: <20010615105717.1c988417.irox@evilpeople.com> (raw)
In-Reply-To: <3B2A0E6E.71C45F7E@redhat.com>

On Fri, 15 Jun 2001 09:32:30 -0400
Fernando Nasser <fnasser@redhat.com> wrote:

> Ian Roxborough wrote:
> > 
> > On Thu, 14 Jun 2001 18:49:36 -0400
> > Fernando Nasser <fnasser@cygnus.com> wrote:
> > > Also, how would this affect testsuite runs?
> > 
> > This shouldn't affect the testsuites. They run fine.
> 
> The gdb testsuites.  But I bet the gdbtk tests will not (will at least
> need some adjustments).

I bet you they do.  The only adjustment that I made to
get the gdbtk testsuit running was setting ITK_LIBRARY.
They are hacks in the current cvs version of Tcl which
munge out the ITCL/ITK pathes to find the source directory.

Just incase I've not been clear:

- running gdb and gdbtk from the install directory works fine.

- running gdb and gdbtk testsuites from the build directory
  works fine.

- running gdb from the build directory works.

The only think that didn't work is running gdbtk from the
build directory without setting the *_LIBRARY environment
variables.

> > > Note that if we make insight's own main.c we can add some code
> > > to handle these situations.
> > 
> > Yeah, I see that.  We can make this set TCL_LIBRARY, etc. before
> > it tries any *_inits and everything should work.
> > 
> 
> Exactly.

Yeah, I've done that. Still don't like it much, but that's
just me (chance to remove some code as well as keeping the
code to run gdb from the build directory in the build, slightly
smaller binary, etc.).  

What I've added will now check to see if gdb has been ran
from a "bin" directory.  If not and TCL_LIBRARY isn't set,
then it will set TCL_LIBRARY and then run a short embedded
Tcl script to set the other *_LIBRARY before going on to
start Insight.

You could also use this code to try and solve some of gdbtk's
location independance problems.


OK, so I've now got all the bits running the way you guys
have requested, I need some time to put this all together
and do some testing/code clean up/etc..  So hopefully
some time next week I'll submit a patch removing the need
from the Tcl local changes and renaming the gdbtk directory
to insight1.0.

Is there any chance of major changes over that time?
When are you planning to split gdb and insight in to
seperate modules?


Thanks,
  Ian.
-- 
What would Jenna Bush do?

  reply	other threads:[~2001-06-15 10:59 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-06-12 13:11 Ian Roxborough
2001-06-12 18:18 ` Keith Seitz
2001-06-12 22:11   ` Ian Roxborough
2001-06-13  8:08   ` Andrew Cagney
2001-06-13  8:12     ` Keith Seitz
2001-06-13 13:06   ` Mo DeJong
2001-06-14  1:30   ` Ian Roxborough
2001-06-14  8:27     ` Keith Seitz
2001-06-14 15:30       ` Ian Roxborough
2001-06-14 15:33         ` Keith Seitz
2001-06-14 15:49           ` Fernando Nasser
2001-06-14 20:03             ` Ian Roxborough
2001-06-15  6:35               ` Fernando Nasser
2001-06-15 10:59                 ` Ian Roxborough [this message]
2001-06-15 11:14                   ` Keith Seitz
2001-06-14 20:17           ` Ian Roxborough
2001-06-14 17:18         ` Mo DeJong

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=20010615105717.1c988417.irox@evilpeople.com \
    --to=irox@evilpeople.com \
    --cc=fnasser@cygnus.com \
    --cc=fnasser@redhat.com \
    --cc=insight@sources.redhat.com \
    --cc=keiths@cygnus.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).