Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew STUBBS <andrew.stubbs@st.com>
To: Daniel Jacobowitz <drow@false.org>
Cc: Eli Zaretskii <eliz@gnu.org>, gdb-patches@sources.redhat.com
Subject: Re: [PATCH] Use search path for scripts
Date: Fri, 25 Nov 2005 20:12:00 -0000	[thread overview]
Message-ID: <4387585E.9050302@st.com> (raw)
In-Reply-To: <20051125181735.GF736@nevyn.them.org>

Daniel Jacobowitz wrote:
> ... what?
> 
> It's the source search path because that's what it's used for.  It's
> also documented that way:
> 
> `-directory DIRECTORY'
> `-d DIRECTORY'
>      Add DIRECTORY to the path to search for source files.

My patch includes a modification to this documentation to reflect it's 
update purpose.

> Most users don't ever have to add a source path.  If you're using a
> modern compiler, and debugging something built locally, and not using
> certain kinds of include path constructs (which is unambiguously an
> open, and recently discussed, bug in GDB) then GDB will Just Find
> Things.

That hasn't changed. The compiler can't tell the debugger where to find 
its script files though.

> I don't find it intuitive that the same path should be overloaded this
> way, and it makes changing the source searching algorithm even more
> complicated.  However, I'd like to know if any other GDB maintainers
> have an opinion on this before I ask you to modify the patch.

I'm not sure a users find it intuitive when they tell a tool to search a 
directory and then it ignores that directory because there was another 
way to tell the tool to search that directory, IYSWIM.

I see your point about changing the search algorithm though. Although 
even then, it might be more intuitive to always use the same algorithm 
for both. Is it likely to change?

Whatever though, as long as I get a way to search for script files I'm 
not that fussed.

Andrew


  reply	other threads:[~2005-11-25 18:33 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-16 19:36 Andrew STUBBS
2005-11-17  0:04 ` Eli Zaretskii
2005-11-17 12:41   ` Andrew STUBBS
2005-11-17 19:42     ` Eli Zaretskii
2005-11-18 13:23       ` Andrew STUBBS
2005-11-18 14:53         ` Eli Zaretskii
2005-11-18 15:54           ` Andrew STUBBS
2005-11-25 18:02             ` Andrew STUBBS
2005-11-25 18:12             ` Daniel Jacobowitz
2005-11-25 18:33               ` Andrew STUBBS
2005-11-25 18:39                 ` Daniel Jacobowitz
2005-11-25 20:12                   ` Andrew STUBBS [this message]
2005-11-25 21:26                     ` Daniel Jacobowitz
2005-11-25 22:33                   ` Eli Zaretskii
2005-11-28 19:34                     ` Andrew STUBBS
2005-11-25 21:56                 ` Eli Zaretskii
2005-11-25 21:43               ` Eli Zaretskii
2006-01-12  0:09             ` Andrew STUBBS
2006-01-22 20:40               ` Daniel Jacobowitz
2006-01-23 16:29                 ` Andrew STUBBS
2006-01-23 17:41                   ` Andrew STUBBS

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=4387585E.9050302@st.com \
    --to=andrew.stubbs@st.com \
    --cc=drow@false.org \
    --cc=eliz@gnu.org \
    --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