Re: [LTP] [PATCH] rwho01: fix getting hostnames from command output and a typo

[Simon Xu <xu.simon@oracle.com>]

Could anyone review this?

Thanks
Simon

On 10/31/13 18:19, Simon Xu wrote:
> Fix the way the script gets hostnames from rwho/ruptime command output,
> and the mistake that $COUNT is used instead of $TST_COUNT in do_test().
> ---
>   testcases/network/tcp_cmds/rwho/rwho01 | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/testcases/network/tcp_cmds/rwho/rwho01 b/testcases/network/tcp_cmds/rwho/rwho01
> index 2c82ef1..eb31642 100755
> --- a/testcases/network/tcp_cmds/rwho/rwho01
> +++ b/testcases/network/tcp_cmds/rwho/rwho01
> @@ -106,18 +106,18 @@ do_test()
>   
>       while [ $TST_COUNT -le $NUMLOOPS ]; do
>           rwho -a > $OUTFILE
> -        HOST=`awk "\\$2 ~ /$LHOST>/" $OUTFILE | cut -f1 -d: | sort -u`
> +        HOST=`grep $LHOST $OUTFILE | sed 's/[^ ]* *//; s/:.*//' | uniq`
>           [ "$HOST" = "$LHOST" ] || end_testcase "$LHOST is not in rwho outfile"
> -        HOST=`awk "\\$2 ~ /$RHOSTNAME>/" $OUTFILE | cut -f1 -d: | sort -u`
> +        HOST=`grep $RHOSTNAME $OUTFILE | sed 's/[^ ]* *//; s/:.*//' | uniq`
>           [ "$HOST" = "$RHOSTNAME" ] || end_testcase "$RHOSTNAME is not in rwho outfile"
>   
>           ruptime -a > $OUTFILE
> -        HOST=`awk "\\$1 ~ /$LHOST>/" $OUTFILE | sort -u`
> +        HOST=`grep $LHOST $OUTFILE | sed 's/ .*//' | uniq`
>           [ "$HOST" = "$LHOST" ] || end_testcase "$LHOST is not in ruptime outfile"
> -        HOST=`awk "\\$1 ~ /$RHOSTNAME>/" $OUTFILE | sort -u`
> +        HOST=`grep $RHOSTNAME $OUTFILE | sed 's/ .*//' | uniq`
>           [ "$HOST" = "$RHOSTNAME" ] || end_testcase "$RHOSTNAME is not in ruptime outfile"
>   
> -        tst_resm TINFO "Test $COUNT of $NUMLOOPS complete"
> +        tst_resm TINFO "Test $TST_COUNT of $NUMLOOPS complete"
>           incr_tst_count
>       done
>   }


------------------------------------------------------------------------------
Android is increasing in popularity, but the open development platform that
developers love is also attractive to malware creators. Download this white
paper to learn more about secure code signing practices that can help keep
Android apps secure.
http://pubads.g.doubleclick.net/gampad/clk?id=65839951&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list