On Wed, Feb 17, 2010 at 09:08:39AM -0800, Garrett Cooper wrote: > On Wed, Feb 17, 2010 at 12:09 AM, Rishikesh <[email protected]> > wrote: > > Hi Shi, > > > > Thanks for your patch . It will be merged in feb 2010 release. > > > > Acked-By: Rishikesh K Rajak <[email protected]> > > > > 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. 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 ------------------------------------------------------------------------------ Download Intel® 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 [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
