This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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] IS_ABSOLUTE_PATH to handle both DOS and POSIX path st yles


On Jan 6, 2008 11:37 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> [I added gdb-patches to the discussion.]
>
> > Date: Sun, 6 Jan 2008 00:40:30 -0500
> > From: Daniel Jacobowitz <drow@false.org>
> > Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
> >       Ryan Mansfield <RMansfield@qnx.com>
> >
> > On Sun, Jan 06, 2008 at 12:30:11AM -0500, Aleksandar Ristovski wrote:
> > > >
> > > > But the form of the file names is determined by the platform on which
> > > > you compiled the sources, right?
> > >
> > > Right and this is the problem: IS_ABSOLUTE_PATH is determined at compile
> > > time only by platform on which gdb will be running. Gdb, however needs to
> > > deal with binaries possibly built on a platform with different file system
> > > (we often have this case: gdb on POSIX file system debugging binary built on
> > > windows).
> >
> > I also regularly work with the other direction, POSIX build system and
> > Windows debug host.
>
> In that case, I think it's wrong to allow both styles of slashes and
> absolute file names.  Instead, we should have a user option to set the
> correct style, which would default to the convention of the platform
> where GDB was built to run.
>
> If you agree with this approach, then the _ANY version of the macros
> is not needed.

It seems like GDB should know what to use, or at least be able to
figure it out (in cases where it really matters).  Are we really sure
we need an option?  How would this option work in practice?  If one
ever wanted to set it to the non-default value it'd be rather
cumbersome to have to set it and then reset it for one-off uses of
"the other" path kind.


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