Re: [RFA] Fix for gdb-gnats bug #14

[Michael Snyder <msnyder@cygnus.com>]

Michael Elizabeth Chastain wrote:
> 
> Hi John,
> 
> As I understand it, HP and Cygnus did a lot of gdb development on a branch
> and then contributed big blocks of it back to the FSF code base.  I wasn't
> around at the time, so maybe somebody who was can comment on the history.

Close enough.  HP did the work separately, Cygnus attempted to merge it.

> This looks like someone was merging an HP chunk and saw that it was
> different from the baseline.  So they took the HP version and wrapped
> it in a test for 'istarget "hppa*-hp-hpux*"' test.

Probably right.

> But the hpux version is right (for all platforms) and the non-hpux
> version is wrong (for all platforms).

Correct.  In fact, this test has always been wrong, and AFAIK, 
it could never ever ever have passed.  I don't understand how
it has existed as such.


> > (and besides, I didn't have an hpux to try it out on).
> 
> I don't have access to an hpux at the moment either.  :(  But I do have
> recent log files from a Red Hat project.  Here are excerpts:
> 
>   # hppa1.1-hp-hpux10.20 native
>   (gdb) PASS: gdb.base/commands.exp: commands in infrun_breakpoint_command_test #2
>   continue^M
>   Continuing.^M
>   ^M
>   Breakpoint 7, factorial (value=5) at ***/run.c:77^M
>   77          if (value > 1) {^M
>   78              value *= factorial (value - 1);^M
>   (gdb) PASS: gdb.base/commands.exp: continue in infrun_breakpoint_command_test
> 
>   # solaris 2.6 native
>   (gdb) PASS: gdb.base/commands.exp: commands in infrun_breakpoint_command_test #2
>   continue^M
>   Continuing.^M
>   ^M
>   Breakpoint 7, factorial (value=5) at ***/run.c:77^M
>   77          if (value > 1) {^M
>   78              value *= factorial (value - 1);^M
>   (gdb) FAIL: gdb.base/commands.exp: continue in infrun_breakpoint_command_test
> 
> > I see no big deal (do more work) to eliminate the hpux exception.
> 
> That would be great.  If my archeological theory is right, you would be
> resolving a merge item that never got completely merged.

I don't understand why it isn't a FAIL on RH7.  The output is
identical to the failing output on Solaris.