Corinna Vinschen wrote:
> Hi,
> 
> I suggest the attached patch to ending-run.exp which fixes the following
> problems:
> 
> - The "step out of main (status wrapper)" check tests only for
>   __wrap_main but not for __wrap__main which is used in dejagnu's
>   testglue.c when UNDERSOCRES is defined.
> 
> - The "step to end of run (status wrapper)" test expects not only
>   the "EXIT code 0" but also a "Program exited normaly" string.
>   That's not ok for targets which never actually exit but are just
>   trapped in a special _exit() handler.
> 
> - Based on this, the expectation that the following "next" will print
>   the string "The program is not being run" is plain wrong.  By adding
>   a special case for these targets and setting the expected behaviour
>   to XFAIL, these targets get a fair chance ;-)
> 
> Corinna
> 
>         * gdb.base/ending-run.exp: Fix expected string for _wrap_main to
>         allow correct handling of #ifdef UNDERSCORES from dejagnu/testglue.c.
>         Fix handling of "step to end of run" for embedded targets which
>         never actually exit.
> 
> 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.



> @@ -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.



> +	    }
> +	    -re "Single.*EXIT code 0\r\n.*$gdb_prompt $" {
> +		pass "step to end of run (status wrapper)"
> +		setup_xfail "*-*-*"
>  	    }
>  	    -re ".*Single.*Program exited.*$gdb_prompt $" {
>  		pass "step to end of run"
> 
> 

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.


Regards,
Fernnado


-- 
Fernando Nasser
Red Hat Canada Ltd.                     E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9