Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Vladimir Prus <vladimir@codesourcery.com>,
		gdb-patches@sources.redhat.com
Subject: Re: Better realpath
Date: Sun, 15 Jun 2008 17:43:00 -0000	[thread overview]
Message-ID: <20080615032653.GA5214@caradoc.them.org> (raw)
In-Reply-To: <ud4mjl71m.fsf@gnu.org>

On Sun, Jun 15, 2008 at 06:18:45AM +0300, Eli Zaretskii wrote:
> > > Alternatively, we could refrain from 
> > > using realpath, in which case we should consistently _not_ require
> > > that the file exists.
> > 
> > ... as I've said, I can modify gdb_realpath to check for file existance,
> > on Windows, which will make the behaviour of gdb_realpath the same
> > everywhere. 
> 
> The question is, what kind of consistent behavior do we want.
> Personally, I think realpath's behavior is wrong for most GDB usages,
> that's why I suggested the alternative.

It seems to me that the thing to do is eliminate gdb_realpath in favor
of lrealpath.  That's supposed to be a portable version of realpath,
so realpath semantics seem like the way to go.

Why do you think realpath's behavior is wrong for GDB?  The only times
we're interested in realpath should be when we have an actual on-disk
file.  If it doesn't exist, its expanded path is not important.

-- 
Daniel Jacobowitz
CodeSourcery


  reply	other threads:[~2008-06-15  3:27 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-14 11:09 Vladimir Prus
2008-06-14 11:30 ` Pierre Muller
2008-06-14 12:14   ` Vladimir Prus
2008-06-14 14:29 ` Eli Zaretskii
2008-06-14 15:10   ` Vladimir Prus
2008-06-14 22:05     ` Eli Zaretskii
2008-06-14 22:26       ` Vladimir Prus
2008-06-15 17:37         ` Eli Zaretskii
2008-06-15 17:43           ` Daniel Jacobowitz [this message]
2008-06-15 21:04             ` Eli Zaretskii
2008-06-16  3:17               ` Daniel Jacobowitz
2008-06-16  3:32                 ` Eli Zaretskii
2008-06-18 18:39               ` Stan Shebs
2008-06-18 20:47                 ` DJ Delorie
2008-06-18 15:22           ` Vladimir Prus
2008-06-18 21:08             ` Eli Zaretskii
2008-06-19  7:27               ` Vladimir Prus
2008-06-20  2:49                 ` Eli Zaretskii

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=20080615032653.GA5214@caradoc.them.org \
    --to=drow@false.org \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=vladimir@codesourcery.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