Re: [LTP] [PATCH] mail_tests.sh: fix a version problem of Heirloom Mail

[Shi Weihua <shiwh@cn.fujitsu.com>]

at 2010-2-18 11:57, Rishikesh K Rajak wrote:
> On Wed, Feb 17, 2010 at 09:08:39AM -0800, Garrett Cooper wrote:
>> On Wed, Feb 17, 2010 at 12:09 AM, Rishikesh <risrajak@linux.vnet.ibm.com> wrote:
>>> Hi Shi,
>>>
>>> Thanks for your patch . It will be merged in feb 2010 release.
>>>
>>> Acked-By: Rishikesh K Rajak <risrajak@linux.vnet.ibm.com>
>>>
>>> http://marc.info/?l=ltp-list&m=126440546415559&w=2
>>
>> There's a lot of noise in the patch because of if-else behavior in
>> trying to determine the mail app. That's why I didn't commit this (it
>> requires verification of previous mail applications as well).
> 
> Thanks Garret, Still i have not committed this to CVS repo.
> 
> Shi, can you please verify if this patch does not break with any other
> mail application.

  In fact, i don't know any other mail application.
  When i tested RHEL's alpha version, this ltp bug catched. so tried to fix it.
  
  Maybe, you should knock FOSS community(?).

> 
> Thanks for your support.
> -Rishi
>>
>> The mail test should be resistant to changes like these as a lot of
>> developers in the FOSS community have different opinions on how to do
>> things and don't follow any particular standard. This `heirloom mailx'
>> support is just one said example that needs to be fixed.
>>
>> Thanks,
>> -Garrett
> 
> 

-- 
Shi Weihua

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list