Skip to contentSkip to conversation

fix mail02,03 fail due to different sender names

PATCH In review Caspar Zhang 3 participants 10 messages Jan 28, 2011 – Feb 28, 2011

lore
Share & tools
lore
#0 lore raw

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
index 53a5d90..c30e2c3 100755
@@ -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

#1 lore raw
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
---
 testcases/commands/mail/mail_tests.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
testcases/commands/mail/mail_tests.sh +1 -1
index 53a5d90..c30e2c3 100755
@@ -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 -1
 	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)
testcases/commands/mail/mail_tests.sh +0 -0
 	fi
 	##################################################################
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
#2 lore raw

[LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]

In reply to Caspar Zhang

Sorry 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.

---
 testcases/commands/mail/mail_tests.sh |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
testcases/commands/mail/mail_tests.sh +1 -1
index 53a5d90..c30e2c3 100755
@@ -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 -1
 	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)
testcases/commands/mail/mail_tests.sh +0 -0
 	fi
 	##################################################################
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
#3 lore raw

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.

---
  testcases/commands/mail/mail_tests.sh |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)
testcases/commands/mail/mail_tests.sh +1 -1
index 53a5d90..c30e2c3 100755
@@ -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

#4 lore raw

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

#5 lore raw

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

#6 lore raw

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

#7 lore raw

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

#8 lore raw

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,
Shubham

Quoted 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

#9 lore raw

Re: [LTP] Review Required: [Re: [PATCH] fix mail02, 03 fail due to different sender names]

In reply to Shubham Goyal
Quoted 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,
Shubham

Quoted 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