public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH] commands/mail: Fix a mismatched sender name in mail0[23]
@ 2017-10-11  9:23 Xiao Yang
  2017-10-11 12:49 ` Cyril Hrubis
  2017-10-13  6:44 ` [LTP] [PATCH v2] " Xiao Yang
  0 siblings, 2 replies; 4+ messages in thread
From: Xiao Yang @ 2017-10-11  9:23 UTC (permalink / raw)
  To: ltp

In some systems (e.g. debain), both mail02 and mail03 fail because the
sender name of the returned letter is "Mail Deliver Syst" which does
not match existed keywords.  we add the 'Syst' keyword to fix it.

Signed-off-by: Shaoting Lei <leist.fnst@cn.fujitsu.com>
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
 testcases/commands/mail/mail_tests.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/testcases/commands/mail/mail_tests.sh b/testcases/commands/mail/mail_tests.sh
index 1652aa7..aaf73ef 100755
--- a/testcases/commands/mail/mail_tests.sh
+++ b/testcases/commands/mail/mail_tests.sh
@@ -169,8 +169,8 @@ else
 		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
 		    $LTPTMP/tst_mail.res)
 	else
-		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
-		    $LTPTMP/tst_mail.res)
+		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
+		    "MailDelivery(Subsys|Systemi|Syst)")}' $LTPTMP/tst_mail.res)
 	fi
 
 	##################################################################
@@ -250,8 +250,8 @@ else
 		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
 		    $LTPTMP/tst_mail.res)
 	else
-		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
-		    $LTPTMP/tst_mail.res)
+		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
+		    "MailDelivery(Subsys|System|Syst)")}' $LTPTMP/tst_mail.res)
 	fi
 	##################################################################
 	# In this testcase, mail will get "Returnedmail:", while mailx will
-- 
1.8.3.1




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [LTP] [PATCH] commands/mail: Fix a mismatched sender name in mail0[23]
  2017-10-11  9:23 [LTP] [PATCH] commands/mail: Fix a mismatched sender name in mail0[23] Xiao Yang
@ 2017-10-11 12:49 ` Cyril Hrubis
  2017-10-13  6:44 ` [LTP] [PATCH v2] " Xiao Yang
  1 sibling, 0 replies; 4+ messages in thread
From: Cyril Hrubis @ 2017-10-11 12:49 UTC (permalink / raw)
  To: ltp

Hi!
> In some systems (e.g. debain), both mail02 and mail03 fail because the
> sender name of the returned letter is "Mail Deliver Syst" which does
> not match existed keywords.  we add the 'Syst' keyword to fix it.
> 
> Signed-off-by: Shaoting Lei <leist.fnst@cn.fujitsu.com>
> Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
> ---
>  testcases/commands/mail/mail_tests.sh | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/testcases/commands/mail/mail_tests.sh b/testcases/commands/mail/mail_tests.sh
> index 1652aa7..aaf73ef 100755
> --- a/testcases/commands/mail/mail_tests.sh
> +++ b/testcases/commands/mail/mail_tests.sh
> @@ -169,8 +169,8 @@ else
>  		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
>  		    $LTPTMP/tst_mail.res)
>  	else
> -		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
> -		    $LTPTMP/tst_mail.res)
> +		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
> +		    "MailDelivery(Subsys|Systemi|Syst)")}' $LTPTMP/tst_mail.res)
                                               ^
					       Typo?
>  	fi
>  
>  	##################################################################
> @@ -250,8 +250,8 @@ else
>  		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
>  		    $LTPTMP/tst_mail.res)
>  	else
> -		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
> -		    $LTPTMP/tst_mail.res)
> +		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
> +		    "MailDelivery(Subsys|System|Syst)")}' $LTPTMP/tst_mail.res)
>  	fi
>  	##################################################################
>  	# In this testcase, mail will get "Returnedmail:", while mailx will
> -- 
> 1.8.3.1
> 
> 
> 
> 
> -- 
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [LTP] [PATCH v2] commands/mail: Fix a mismatched sender name in mail0[23]
  2017-10-11  9:23 [LTP] [PATCH] commands/mail: Fix a mismatched sender name in mail0[23] Xiao Yang
  2017-10-11 12:49 ` Cyril Hrubis
@ 2017-10-13  6:44 ` Xiao Yang
  2017-10-13 11:47   ` Cyril Hrubis
  1 sibling, 1 reply; 4+ messages in thread
From: Xiao Yang @ 2017-10-13  6:44 UTC (permalink / raw)
  To: ltp

In some systems (e.g. debain), both mail02 and mail03 fail because the
sender name of the returned letter is "Mail Deliver Syst" which does
not match existed keywords.  we add the 'Syst' keyword to fix it.

Signed-off-by: Shaoting Lei <leist.fnst@cn.fujitsu.com>
Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com>
---
 testcases/commands/mail/mail_tests.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/testcases/commands/mail/mail_tests.sh b/testcases/commands/mail/mail_tests.sh
index 1652aa7..7cd5531 100755
--- a/testcases/commands/mail/mail_tests.sh
+++ b/testcases/commands/mail/mail_tests.sh
@@ -169,8 +169,8 @@ else
 		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
 		    $LTPTMP/tst_mail.res)
 	else
-		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
-		    $LTPTMP/tst_mail.res)
+		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
+		    "MailDelivery(Subsys|System|Syst)")}' $LTPTMP/tst_mail.res)
 	fi
 
 	##################################################################
@@ -250,8 +250,8 @@ else
 		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3, "Mailer-Daemon")}' \
 		    $LTPTMP/tst_mail.res)
 	else
-		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5, "MailDelivery(Subsys|System)")}' \
-		    $LTPTMP/tst_mail.res)
+		RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
+		    "MailDelivery(Subsys|System|Syst)")}' $LTPTMP/tst_mail.res)
 	fi
 	##################################################################
 	# In this testcase, mail will get "Returnedmail:", while mailx will
-- 
1.8.3.1




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [LTP] [PATCH v2] commands/mail: Fix a mismatched sender name in mail0[23]
  2017-10-13  6:44 ` [LTP] [PATCH v2] " Xiao Yang
@ 2017-10-13 11:47   ` Cyril Hrubis
  0 siblings, 0 replies; 4+ messages in thread
From: Cyril Hrubis @ 2017-10-13 11:47 UTC (permalink / raw)
  To: ltp

Hi!
Pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-10-13 11:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-11  9:23 [LTP] [PATCH] commands/mail: Fix a mismatched sender name in mail0[23] Xiao Yang
2017-10-11 12:49 ` Cyril Hrubis
2017-10-13  6:44 ` [LTP] [PATCH v2] " Xiao Yang
2017-10-13 11:47   ` Cyril Hrubis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox