Hi!
> Hi Cyril
> 
> I sent this as new message since it's a revision of the patch as you
> suggested, which is a better way to do it. I wasn't sure whether it
> would be proper procedure to send a revised patch in a reply that
> contained the original. Anyway I modified some of the configuration
> settings of the mail client I'm using (Thunderbird) which unfortunately
> I need to run on a windows machine. Hopefully the configuration changes
> will prevent distortion of the patch. Please let me know if it does not.
> 
>  Signed-off-by: James Czyzak <[email protected]>

The patch is OK. But please write description for the code change into
the mail body (instead of how Thunderbird tends to mangle patches ;).

Also there shouldn't be a whitespace before the Signed-off-by line.

> ---
>  testcases/network/tcp_cmds/ssh/ssh03 |    4 +---
>  1 files changed, 1 insertions(+), 3 deletions(-)
> 
> diff --git a/testcases/network/tcp_cmds/ssh/ssh03
> b/testcases/network/tcp_cmds/ssh/ssh03
> index 8fbca34..798d385 100755
> --- a/testcases/network/tcp_cmds/ssh/ssh03
> +++ b/testcases/network/tcp_cmds/ssh/ssh03
> @@ -39,10 +39,8 @@ do_setup()
>      export TEST_USER_HOMEDIR="/home/$TEST_USER"
> 
>      # erase user if he/she already exists, so we can have a clean env
> -    TCtmp=/home/$TEST_USER
> -
> -    rm -Rf $TCtmp
> 
> +    rm -Rf /home/$TEST_USER
>      tst_setup
> 
>      exists expect ssh ssh03_s1 useradd userdel
> --
> 1.7.4.1

-- 
Cyril Hrubis
[email protected]

------------------------------------------------------------------------------
Special Offer -- Download ArcSight Logger for FREE!
Finally, a world-class log management solution at an even better 
price-free! And you'll get a free "Love Thy Logs" t-shirt when you
download Logger. Secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsisghtdev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to