public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: Mo DeJong <mdejong@cygnus.com>
To: insight@sources.redhat.com
Subject: Re: Tcl/Tk 8.3 upgrade...
Date: Thu, 14 Jun 2001 17:18:00 -0000	[thread overview]
Message-ID: <Pine.SOL.3.91.1010614171436.25913A-100000@cse.cygnus.com> (raw)
In-Reply-To: <20010614152905.67cf7aa6.irox@evilpeople.com>

On Thu, 14 Jun 2001, Ian Roxborough wrote:

> On Thu, 14 Jun 2001 08:27:52 -0700 (PDT) Keith Seitz <keiths@cygnus.com> wrote:
> > On Thu, 14 Jun 2001, Ian Roxborough wrote:
> > 
> > > Another fix is to switch and rename the hacked tcl_findLibrary with
> > > the real tcl_findLibrary, so we don't rely on the hack being in
> > > the Tcl version.
> > >
> > > Another fix is to "detect" (if <gdb executablr dir> + gdb-tk.o = true...?)
> > > if we are in the build directory then set the required
> > > *_LIBRARY variables as needed.
> > 
> > I'll vote for losing the hacked tcl_findLibrary, and using some heuristic
> > for determining if we are running in the build dir. Anything to prevent us
> > from modifying Tcl/Tk/tix/itcl/etc.
> 
> I'm finding yet more and more local changes to Tcl to make it play nicely
> with Insight running from the build directory (the ones for launching
> Tcl in tclInitScript.h).
> 
> Is anybody going to mind if I install a script at build time to run
> gdb out of the build directory.  Maybe call it run-insight.sh or something.
> Everything works other than typing ./gdb in the build directory,
> I'm not sure if this is worth all the code changes vs. a 7 line
> shell script.

Why don't we setup things so that running "make gdb" runs the
executable in the build directory. That way, we could set the
TCL_LIBRARY env var before running gdb in the build dir. That
is how tclsh is run out of the build directory. We should stick
with something simple instead of hacking the source code
to support running in the build dir (if possible).
 
Mo DeJong
Red Hat Inc

      parent reply	other threads:[~2001-06-14 17:18 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
2001-06-15 11:14                   ` Keith Seitz
2001-06-14 20:17           ` Ian Roxborough
2001-06-14 17:18         ` Mo DeJong [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=Pine.SOL.3.91.1010614171436.25913A-100000@cse.cygnus.com \
    --to=mdejong@cygnus.com \
    --cc=insight@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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).