This is the mail archive of the gdb@sources.redhat.com mailing list for the GDB 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: bug in gdb.mi/mi-file.exp (or maybe in GDB)


On Wed, Apr 09, 2003 at 05:25:52PM -0400, Elena Zannoni wrote:
> David Carlton writes:
>  > On Wed, 9 Apr 2003 16:30:24 -0400, Bob Rossi <bob_rossi at cox dot net> said:
>  > > On Wed, Apr 09, 2003 at 01:01:41PM -0700, David Carlton wrote:
>  > 
>  > >> gdb.log says:
>  > >> 
>  > >> 111^done,line="23",file="gdb.mi/basics.c",fullname="/extra/gdb/mirror/src/gdb/testsuite/gdb.mi/basics.c"
>  > 
>  > > This is what I get, which passes
>  > > 111^done,line="23",file="./gdb.mi/basics.c",fullname="/home/bob/cvs/src/gdb/testsuite/gdb.mi/basics.c"^
>  > 
>  > Ah. I guess maybe different versions of GCC differ as to whether or
>  > not they include the './' in the filename in the debug info, or
>  > something.
>  > 
> 
> Compiling and building everything with absolute paths in build!=src, I
> get absolute paths in both.
> 
> 111^done,line="23",file="/home/ezannoni/sources/src/gdb/testsuite/gdb.mi/basics.c",fullname="/home/ezannoni/sources/src/gdb/testsuite/gdb.mi/basics.c"
> 
>  > >> Should 'file' match .*${srcfile}
>  > >> (without any slashes) and 'fullname' match ${srcfilepath}?
>  > 
>  > > I think your suggestion of what file should match is a good one.
>  > 
>  > Elena just raised a similar issue in annota1.exp.  So maybe we should
>  > wait for the resolution there and then adopt the same solution for
>  > 'file' here.  (Though the format of 'file' is less crucial here, since
>  > that's what 'fullname' is for.)
>  >  
> 
> All I care about it to match correctly with '+' in the directory
> names. So maybe all I need is to use string_to_regexp and leave the
> full path in there.
> 
>  > > The fullname match should probably not change. Since ${srcfilepath}
>  > > is not guaranteed to be the absolute path to the source file.
>  > 
>  > Oh, right, duh.  I should have tested that before suggesting it!
>  > 
> 
> Ah right, ${srcfilepath} is set up by the testsuite machinery, not
> gdb, so it is not guaranteed to match fullname.  I thought you meant
> that fullname could not be absolute, and that got me puzzled.  Bottom
> line, the testsuite doesn't know what the absolute path is.  We could
> pull the dirty trick of asking gdb for it, with an 'info source' and
> use that output to match.

I think if we did that, they might break consistently, and the test case
could still pass. It seems like maybe we should allow the file to match
the same thing we allow fullname to match. "/.*$(srcfile}" Since it is legal.

Bob Rossi

> 
> 
> elena
> 
> 
>  > David Carlton
>  > carlton at math dot stanford dot edu


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