public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
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

  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