From: Shubham Goyal <shubham@linux.vnet.ibm.com>
To: Caspar Zhang <czhang@redhat.com>
Cc: LTP list <ltp-list@lists.sourceforge.net>
Subject: Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Date: Wed, 23 Feb 2011 00:45:33 +0530 [thread overview]
Message-ID: <4D640B55.1050300@linux.vnet.ibm.com> (raw)
In-Reply-To: <4D58DD0A.2050403@redhat.com>
Hi Caspar,
I reviewed/verified your patch and it seems to be working fine. I verified it on RHEL6 and both mail02 and mail03 tests are passing now. I believe it can be merged upstream.
Garrett, Any comments?
Thanks,
Shubham
On Monday 14 February 2011 01:13 PM, Caspar Zhang wrote:
> Sorry for bothering, any comments on this patch?
>
> On 01/29/2011 01:41 AM, Caspar Zhang wrote:
>> Hi all, mail02 and mail03 tests non-existed host and non-existed
>> username in the test. However in some systems (at least in RHEL6),
>> they both fails because of the sender name of the returned letter is
>> "Mail Delivery System" instead of "Mail Delivery Subsys", which makes RC1
>> to be zero and fail the test. The patch fixes this issue.
>>
>> Signed-off-by: Caspar Zhang<czhang@redhat.com>
>> ---
>> testcases/commands/mail/mail_tests.sh | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/testcases/commands/mail/mail_tests.sh
>> b/testcases/commands/mail/mail_tests.sh
>> index 53a5d90..c30e2c3 100755
>> --- a/testcases/commands/mail/mail_tests.sh
>> +++ b/testcases/commands/mail/mail_tests.sh
>> @@ -169,7 +169,7 @@ 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,
>> "MailDeliverySubsys")}' \
>> + RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
>> "MailDelivery(Subsys|System)")}' \
>> $LTPTMP/tst_mail.res)
>> fi
>>
>> @@ -250,7 +250,7 @@ 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,
>> "MailDeliverySubsys")}' \
>> + RC1=$(awk '/^>N/ {IGNORECASE=1; print match($3 $4 $5,
>> "MailDelivery(Subsys|System)")}' \
>> $LTPTMP/tst_mail.res)
>> fi
>> ##################################################################
>
------------------------------------------------------------------------------
Free Software Download: Index, Search & Analyze Logs and other IT data in
Real-Time with Splunk. Collect, index and harness all the fast moving IT data
generated by your applications, servers and devices whether physical, virtual
or in the cloud. Deliver compliance at lower cost and gain new business
insights. http://p.sf.net/sfu/splunk-dev2dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2011-02-22 19:15 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-28 17:41 [LTP] [PATCH] fix mail02,03 fail due to different sender names Caspar Zhang
2011-02-11 15:45 ` Caspar Zhang
2011-02-14 7:43 ` [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names] Caspar Zhang
2011-02-22 19:15 ` Shubham Goyal [this message]
2011-02-22 21:08 ` Garrett Cooper
2011-02-24 18:31 ` Shubham Goyal
2011-02-24 18:37 ` Garrett Cooper
2011-02-28 7:41 ` Shubham Goyal
2011-02-28 7:43 ` Garrett Cooper
2011-02-28 7:55 ` Caspar Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4D640B55.1050300@linux.vnet.ibm.com \
--to=shubham@linux.vnet.ibm.com \
--cc=czhang@redhat.com \
--cc=ltp-list@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox