fix mail02,03 fail due to different sender names
PATCH 1/1 fix mail02,03 fail due to different sender names
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 Deliver System" instead of "Mail Delivery Subsys", which makes RC1 to be zero and fail the test. The patch fixes this issue.
--- testcases/commands/mail/mail_tests.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
(quoted diff) +0 -0
b/testcases/commands/mail/mail_tests.sh
index 53a5d90..c30e2c3 100755
testcases/commands/mail/mail_tests.sh +0 -1
@@ -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
##################################################################
Signature · 11 lines
1.7.4.rc2 -- Quality Assurance Engineer (Kernel) in Red Hat Software (Beijing) Co., R&D Branch TEL: +86-10-62608150 WEB: http://www.redhat.com/ -- C-A-S-P-A-R, Caspar /kæspɑ:/ ;-)
Subject: [PATCH] fix mail02,03 fail due to different sender names
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 Deliver System" instead of "Mail Delivery Subsys", which makes RC1 to be zero and fail the test. The patch fixes this issue.
--- testcases/commands/mail/mail_tests.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
testcases/commands/mail/mail_tests.sh +2 -2
@@ -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 ##################################################################
Signature · 1 line
1.7.4.rc2
------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)! Finally, a world-class log management solution at an even better price-free! Download using promo code Free_Logger_4_Dev2Dev. Offer expires February 28th, so secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Quoted 5 lines · Caspar Zhang
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 Deliver System" instead of "Mail Delivery Subsys", which makes RC1 to be zero and fail the test. The patch fixes this issue.
Garrett, any objection merging it to upstream?
Quoted 33 lines
Signed-off-by Caspar Zhang <[email protected]>--- testcases/commands/mail/mail_tests.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)testcases/commands/mail/mail_tests.sh +1
-1@@ -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)testcases/commands/mail/mail_tests.sh +1
-1fi@@ -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)testcases/commands/mail/mail_tests.sh +0
-0fi ##################################################################
Signature · 18 lines
Quality Assurance Engineer (Kernel) in Red Hat Software (Beijing) Co., R&D Branch TEL: +86-10-62608150 WEB: http://www.redhat.com/ -- C-A-S-P-A-R, Caspar /kæspɑ:/ ;-) ------------------------------------------------------------------------------ The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: Pinpoint memory and threading errors before they happen. Find and fix more than 250 security defects in the development cycle. Locate bottlenecks in serial and parallel code that limit performance. http://p.sf.net/sfu/intel-dev2devfeb _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
[LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
In reply to Caspar ZhangSorry for bothering, any comments on this patch?
Quoted 38 lines · Caspar Zhang
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 <[email protected]>--- testcases/commands/mail/mail_tests.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)testcases/commands/mail/mail_tests.sh +1
-1@@ -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)testcases/commands/mail/mail_tests.sh +1
-1fi@@ -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)testcases/commands/mail/mail_tests.sh +0
-0fi ##################################################################
Signature · 18 lines
Quality Assurance Engineer (Kernel) in Red Hat Software (Beijing) Co., R&D Branch TEL: +86-10-62608150 WEB: http://www.redhat.com/ -- C-A-S-P-A-R, Caspar /kæspɑ:/ ;-) ------------------------------------------------------------------------------ The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE: Pinpoint memory and threading errors before they happen. Find and fix more than 250 security defects in the development cycle. Locate bottlenecks in serial and parallel code that limit performance. http://p.sf.net/sfu/intel-dev2devfeb _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
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
Quoted 30 lines · Caspar Zhang
Sorry for bothering, any comments on this patch?
Quoted 27 lines · Caspar Zhang
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<[email protected]>--- testcases/commands/mail/mail_tests.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)testcases/commands/mail/mail_tests.sh +1
-1@@ -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
Quoted 12 lines
Quoted 11 lines
testcases/commands/mail/mail_tests.sh +1
-1@@ -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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Quoted 6 lines · Shubham Goyal
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?
I guess... there's no apparent rhyme nor reason to GNU/Linux
userland software development, so I'm not going to fight these issues anymore.
The LSB should maintain these tests and standardize things if
anyone really cares like the opengroup folks do for all of Unix. Thanks,
-Garrett
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Alright Garrett..., so how should we take the patches in userland arena in future (including this one from Caspar)? I mean should they be reviewed and merged upstream? Do LSB currently involved in maintaining them?
Thanks, Shubham
Quoted 8 lines · Garrett Cooper
Quoted 6 lines · Shubham Goyal
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?
Quoted 7 lines
I guess... there's no apparent rhyme nor reason to GNU/Linux
userland software development, so I'm not going to fight these issues anymore.
The LSB should maintain these tests and standardize things if
anyone really cares like the opengroup folks do for all of Unix. Thanks,
-Garrett
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Quoted 4 lines · Shubham Goyal
Alright Garrett..., so how should we take the patches in userland arena in future (including this one from Caspar)?
I mean should they be reviewed and merged upstream? Do LSB currently involved in maintaining them?
(this is going to seem like a bit of a rant, so please bear with me...)
I don't know if the LSB actually has a test suite, but they should really get into the business of producing a test suite, because the LSB is mocked (in some circles) for being an excuse for a standards committee.
FreeBSD devs (for sure) and many Linux devs know that there isn't a lot of decision making that goes into making Linux consistent across all platforms. Someone needs to herd all of the cats together and provide a means of testing standardization for these things (again, LSB) because then the quality element will properly trickle down in the GNU/Linux community.
Right now everything's too ad hoc and messy for anyone depending on behavior to actually depend on Linux to remain constant over an extended period of time. That's one reason why (apart from GPL licensing) that people stay away from Linux: it's consistently inconsistent.
Thanks,
-Garrett
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Alright Garrett.., I can understand your concerns on quality standardization and consistency across multiple platforms. But how do we conclude my below question for now?
@Caspar: Any comments from your side?
Thanks,
Shubham
Quoted 20 lines
Quoted 4 lines · Shubham Goyal
Alright Garrett..., so how should we take the patches in userland arena in future (including this one from Caspar)?
I mean should they be reviewed and merged upstream? Do LSB currently involved in maintaining them?(this is going to seem like a bit of a rant, so please bear with me...)
I don't know if the LSB actually has a test suite, but they should really get into the business of producing a test suite, because the LSB is mocked (in some circles) for being an excuse for a standards committee.
FreeBSD devs (for sure) and many Linux devs know that there isn't a lot of decision making that goes into making Linux consistent across all platforms. Someone needs to herd all of the cats together and provide a means of testing standardization for these things (again, LSB) because then the quality element will properly trickle down in the GNU/Linux community.
Quoted 8 lines
Right now everything's too ad hoc and messy for anyone depending on behavior to actually depend on Linux to remain constant over an extended period of time. That's one reason why (apart from GPL licensing) that people stay away from Linux: it's consistently inconsistent.
Thanks,
-Garrett
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
Commit "fix" now, beat standards committees into submission with a baseball bat later.
Thanks,
-Garrett
Quoted 38 lines · Shubham Goyal
Alright Garrett.., I can understand your concerns on quality standardization and consistency across multiple platforms. But how do we conclude my below question for now?
@Caspar: Any comments from your side?
Thanks,
ShubhamQuoted 28 lines
Quoted 4 lines · Shubham Goyal
Alright Garrett..., so how should we take the patches in userland arena in future (including this one from Caspar)?
I mean should they be reviewed and merged upstream? Do LSB currently involved in maintaining them?(this is going to seem like a bit of a rant, so please bear with me...)
I don't know if the LSB actually has a test suite, but they should really get into the business of producing a test suite, because the LSB is mocked (in some circles) for being an excuse for a standards committee.
FreeBSD devs (for sure) and many Linux devs know that there isn't a lot of decision making that goes into making Linux consistent across all platforms. Someone needs to herd all of the cats together and provide a means of testing standardization for these things (again, LSB) because then the quality element will properly trickle down in the GNU/Linux community.
Right now everything's too ad hoc and messy for anyone depending on behavior to actually depend on Linux to remain constant over an extended period of time. That's one reason why (apart from GPL licensing) that people stay away from Linux: it's consistently inconsistent.
Thanks,
-Garrett
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]
In reply to Shubham GoyalQuoted 5 lines · Shubham Goyal
Alright Garrett.., I can understand your concerns on quality standardization and consistency across multiple platforms. But how do we conclude my below question for now?
@Caspar: Any comments from your side?
I see Garrett committed my patch, it seems good for me. And I agree with that LSB should maintain a test suite to improve standardization.
Thanks,
Caspar
Quoted 35 lines
Thanks,
ShubhamQuoted 29 lines
Quoted 5 lines · Shubham Goyal
Alright Garrett..., so how should we take the patches in userland arena in
future (including this one from Caspar)?
I mean should they be reviewed and merged upstream? Do LSB currently involved in maintaining them?(this is going to seem like a bit of a rant, so please bear with me...)
I don't know if the LSB actually has a test suite, but they should really get into the business of producing a test suite, because the LSB is mocked (in some circles) for being an excuse for a standards committee.
FreeBSD devs (for sure) and many Linux devs know that there isn't a lot of decision making that goes into making Linux consistent across all platforms. Someone needs to herd all of the cats together and provide a means of testing standardization for these things (again, LSB) because then the quality element will properly trickle down in the GNU/Linux community.
Right now everything's too ad hoc and messy for anyone depending on behavior to actually depend on Linux to remain constant over an extended period of time. That's one reason why (apart from GPL licensing) that people stay away from Linux: it's consistently inconsistent.
Thanks,
-Garrett
Signature · 15 lines
Quality Engineer (Kernel) in Red Hat Software (Beijing) Co., R&D Branch http://www.cn.redhat.com/ TEL: +86-10-62608150 ------------------------------------------------------------------------------ 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 [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list