Re: [RFA] testsuite/gdb.base/ending-run.exp

[Corinna Vinschen <vinschen@redhat.com>]

On Wed, Sep 18, 2002 at 11:44:49AM -0400, Fernando Nasser wrote:
> >Index: gdb.base/ending-run.exp
> >===================================================================
> >RCS file: /cvs/src/src/gdb/testsuite/gdb.base/ending-run.exp,v
> >retrieving revision 1.16
> >diff -u -p -r1.16 ending-run.exp
> >--- gdb.base/ending-run.exp	19 Jul 2002 00:02:34 -0000	1.16
> >+++ gdb.base/ending-run.exp	18 Sep 2002 11:21:22 -0000
> >@@ -195,7 +195,7 @@ gdb_expect {
> > 	    -re ".*in.*\\\$START\\\$.*from.*dld.sl.*$gdb_prompt $" { 
> > 		pass "step out of main"
> > 	    }
> >-	    -re ".*in __wrap_main ().*$gdb_prompt $" {
> >+	    -re ".*in __wrap__?main ().*$gdb_prompt $" {
> > 		pass "step out of main (status wrapper)"
> > 	    }
> >             -re ".*$gdb_prompt $" { fail "step out of main (at end 2)" }
> 
> This part is OK.  You can check it in immediately.

Applied.

> >@@ -245,8 +245,12 @@ if {! [target_info exists use_gdb_stub]
> > 		# exit" message.
> > 		pass "step to end of run"
> > 	    }
> >-	    -re "Single.*EXIT code 0.*Program exited normally.*$gdb_prompt $" {
> >+	    -re "Single.*EXIT code 0\r\n.*Program exited normally.*$gdb_prompt $" 
> >{
> > 		pass "step to end of run (status wrapper)"
> 
> You forgot to mention this change (which seems correct).  It also can be 
> checked in providing you mention it in the ChangeLog.

Applied w/ ChangeLog entry.

> The code for xstormy was actually not correct.  Please look at the 
> attached (untested) patch, add your case to it setting the variables 
> accordingly and give it a spin.  If it works for you I will check it in 
> and then you can add the case above to it.

Works fine.  My code then adds the new case as unsupported.  I'll
check it in as soon as you checked your patch in.

Thanks,
Corinna