Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Bob Rossi <bob_rossi@cox.net>
To: David Carlton <carlton@math.stanford.edu>
Cc: gdb <gdb@sources.redhat.com>, Andrew Cagney <ac131313@redhat.com>,
	Elena Zannoni <ezannoni@redhat.com>
Subject: Re: bug in gdb.mi/mi-file.exp (or maybe in GDB)
Date: Wed, 09 Apr 2003 20:30:00 -0000	[thread overview]
Message-ID: <20030409203024.GA22063@white> (raw)
In-Reply-To: <ro1ptnvh23e.fsf@jackfruit.Stanford.EDU>

On Wed, Apr 09, 2003 at 01:01:41PM -0700, David Carlton wrote:
> When I run gdb.mi/mi-file.exp, I get this failure:
> 
> FAIL: gdb.mi/mi-file.exp: request path info of current source file (basics.c)
> 
> Presumably nobody else is seeing this because I'm the only weirdo who
> builds in source.  I think it's probably a testsuite bug instead of a
> GDB bug, but I don't know exactly what the relevant MI command is
> supposed to do, so I'll just show you what I'm seeing: gdb.log says:
> 
>   111^done,line="23",file="gdb.mi/basics.c",fullname="/extra/gdb/mirror/src/gdb/testsuite/gdb.mi/basics.c"
> 
> whereas mi-file.exp is looking for:
> 
>   "111\\\^done,line=\"23\",file=\"${srcfilepath}\",fullname=\"/.*/${srcfile}\""

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"^

> 
> where ${srcfilepath} is defined by:
> 
>   set srcfilepath [string_to_regexp ${srcdir}/${subdir}/${srcfile}]
> 
> So the test seems to expect 'file' to be the fully qualified path and
> for 'fullname' to be anything random that starts with a slash and ends
> right.  Whereas I'm not getting a fully qualified path for 'file'.
> So: is 'file' really supposed to be a fully qualified path?  If so,
> what's the difference between 'file' and 'fullname'? 

file should be the path that gdb has always produced. The relative path
to the source file given from where it was compiled. Its basically what
you should get when you do 'info sources'.

fullname should be the absolute path to the source file.
I only tested it for starting with a / and ending with basics.c because
that seemed to be a good and easy way to ensure the file was absolute.

> Should 'file' match .*${srcfile}
> (without any slashes) and 'fullname' match ${srcfilepath}?

I think your suggestion of what file should match is a good one.
The fullname match should probably not change. Since ${srcfilepath} is not
guaranteed to be the absolute path to the source file.

> Also, should gdb.mi have its own ChangeLog?  It's being used
> inconsistently, since most of the recent entries got put in
> testsuite/ChangeLog.
> 
> David Carlton
> carlton@math.stanford.edu


  reply	other threads:[~2003-04-09 20:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-09 20:01 David Carlton
2003-04-09 20:30 ` Bob Rossi [this message]
2003-04-09 20:38   ` David Carlton
2003-04-09 21:21     ` Elena Zannoni
2003-04-09 21:28       ` Bob Rossi
2003-04-09 22:09         ` Elena Zannoni
2003-04-09 20:54 ` Elena Zannoni

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=20030409203024.GA22063@white \
    --to=bob_rossi@cox.net \
    --cc=ac131313@redhat.com \
    --cc=carlton@math.stanford.edu \
    --cc=ezannoni@redhat.com \
    --cc=gdb@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