This is the mail archive of the gdb-patches@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: [RFA]: Fix for pending breakpoints in manually loaded/unloaded shlibs


Jeff Johnston <jjohnstn@redhat.com> wrote:
> Thanks.  Patches committed, including fix to the testcase.

Uh, I haven't approved that fix.

I wrote:

  If the test script accepts both old+new messages, and the new message is
  not wildly more complex than the old message, then testing with the old
  message alone is good enough for getting the test script approved.
  Just pop out the new patch and say how / what system you tested on.
   
You needed to send a fresh patch to gdb-patches, say how you tested it,
and then get it approved.  I'm sorry if this wasn't clear from my message.
Sometimes I do write too colloquially.

This is actually a live issue because the patch you committed has
a problem:

  -"Continuing.*y is 7.*warning: Temporarily disabling unloaded shared library breakpoints.*warning: breakpoint #.*Program exited normally." \
  -"continuing to end of program second time"
  +"Continuing.*y is 7.*warning: Temporarily disabling breakpoints for.*unloadshr.sl.*Program exited normally." \
  +"continuing to end of program"

Now there are two tests with the identical name "continuing to end
of program", which leads to confusion.

Can you please: fix that; say which system you tested it on;
and submit a patch to gdb-patches?

Michael


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