Hi!
> Hello Cyril Hrubis,

Sorry for the delay, I've been quite bussy with the release.

>     You are right! this error was found at lastest version(20140115).
>     I add 2 patch files:    
>     1. ltp-fix-sar-command-error-20140411.patch:
>         Fix sar command error with commit messages.

This one looks good to me.

>     2.ltp-fix-bad-variable-nameerr-20140411.patch:
>          Fix a bad variable name in runltp script for 
> DEFAULT_FILE_NAME_GENERATION_TIME when ltp running at
>          differenct language os(chinese).
>         This error was found in  my os, I use zh_CN for default language, the 
> `date +"%Y_%b_%d-%Hh_%Mm_%Ss"`
>         will get like "2014_ 4??_11-09h_36m_52s", it can not be a variable 
> name because include chinese characters.

Can you please move this detailed description to the patch itself so
that it gets included in the git repository?

Also please add a signed-of-by line to the patches (git can help create
these once you setup your name and email address) and send these to
mailing list one patch per email (ideally inline as well as attached).

There is a good howto for sumbitting patches to Linux kernel and most of
the information is applicable to other open source projects as well.
Have a look at:

https://www.kernel.org/doc/Documentation/SubmittingPatches

At least paragraphs 2, 3, 7, 11 are directly applicable to LTP as well.

-- 
Cyril Hrubis
chru...@suse.cz

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.  Get 
unparalleled scalability from the best Selenium testing platform available.
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to