----- "Garrett Cooper" <[email protected]> wrote:

> On Tue, Jul 20, 2010 at 8:40 PM, Caspar Zhang <[email protected]>
> wrote:
> > Hi all,
> >   In latest RHEL6, the mail format sometimes becomes "...Mail
> >   Delivery System..." instead of "...Mail Delivery Subsys...".
> >   This patch fixes this problem. Plus, corrected misspelling.
> >
> >   Signed-off-by: Caspar Zhang <[email protected]>
> >
>  ##################################################################
> 
> How about MailDeliverySub? It's a simpler version of the same string,
> so it's more resilient to change. 

I don't think so. The string is `Mail Delivery System' which doesn't
match `MailDeliverySub'. In fact, I get this different string in automated
testing framework, like this:

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Heirloom Mail version 12.4 7/29/08.  Type ? for help.
"/var/mail/root": 1 message 1 new
>N  1 Mail Delivery System  Thu Jul 15 01:21  77/3077  "Undelivered Mail Retu"
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

> Besides, the point isn't to check
> for the name being that, it's to verify that the the delivery failed
> :)..

Yes I see. The original codes tried to catch 'Mail Delivery Subsys' string 
and select the new item, then catch 'Undelivered Mail Retu' string to 
confirm it's a failed delivered mail. As I said before, in automated testing 
framework, the first catch will fail because the first string become 'Mail 
Delivery System' instead of 'Mail Delivery Subsys' and thus it cause the 
whole testing failed. 

Thanks,
Caspar

-- 
Quality Assurance Associate (Kernel) in
Red Hat Inc. (Beijing R&D Branch)

TEL: +86-10-62608150
Web: http://www.redhat.com/

------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to