Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Yao Qi <yao@codesourcery.com>
To: Hui Zhu <hui_zhu@mentor.com>
Cc: <gdb-patches@sourceware.org>
Subject: Re: [PATCH v2] Add autoload-breakpoints [2/6] ReportAsync-test
Date: Tue, 08 May 2012 08:39:00 -0000	[thread overview]
Message-ID: <4FA8DBB9.50505@codesourcery.com> (raw)
In-Reply-To: <4FA8CA98.6010503@mentor.com>

On 05/08/2012 03:26 PM, Hui Zhu wrote:
>> Remote feature should be tested in remote target or board file.
>> You are testing a remote feature GDB unconditionally, even in native
>> gdb.  It looks incorrect to me.
>>
> 
> This just a check for the this small gdbserver is OK.   I don't think this test will work with board or something.
> 

My point is this test case exercises some "undesired" features w.r.t.
testsuite and config.  I configure and build a native gdb, and type
`make -k check' in gdb build dir.  Then, I'll examine the FAILs in
gdb.sum to see the status of native gdb, what does the FAILs, if any, in
gdb.remote/reportasync-test.exp mean to me?  It sounds like "I did a
medical check-up to my eyes, but result tells me something wrong in my
ears".  However, I am not strongly against this test case.

We have a sub-dir gdb.server under testsuite/, *.exp are run even when
testing native gdb, but they are test cases to GDBserver, rather than
GDB, AFAICS.  When we see some FAILs in it, that means GDBserver has
something wrong, at least ideally :)

-- 
Yao (齐尧)


  reply	other threads:[~2012-05-08  8:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-11 10:54 Hui Zhu
2012-04-28  7:15 ` Hui Zhu
2012-04-28 20:37   ` Yao Qi
2012-05-08  7:26     ` Hui Zhu
2012-05-08  8:39       ` Yao Qi [this message]
2012-05-10 15:31         ` Hui Zhu

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=4FA8DBB9.50505@codesourcery.com \
    --to=yao@codesourcery.com \
    --cc=gdb-patches@sourceware.org \
    --cc=hui_zhu@mentor.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