public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* Re: [LTP] [PATCH] Fix an bad variable name erro in runltp script
       [not found] <47b112ce.152ba.145b070f2b0.Coremail.eeslook@163.com>
@ 2014-05-05 13:51 ` chrubis
  0 siblings, 0 replies; only message in thread
From: chrubis @ 2014-05-05 13:51 UTC (permalink / raw)
  To: look; +Cc: ltp-list

Hi!
>     I got an error for the latest version 20140115 of ltp project.
>     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_ 4yue_11-09h_36m_52s",
> it contain a space before "4yue" will get the bad variable name error.
> 
> 
>     I fix this error and change the date args,  see detail below:
> -    local DEFAULT_FILE_NAME_GENERATION_TIME=`date +"%Y_%b_%d-%Hh_%Mm_%Ss"`
> +    local DEFAULT_FILE_NAME_GENERATION_TIME=`date +"%Y_%m_%d-%Hh_%Mm_%Ss"`

I've reworded the summary a bit, added quotes around the variable and
pushed, thanks.

PS: I've been unable to reproduce the bug, for me date +%b does not add
    a space before the month even with LC_ALL=zh_CN, but IMHO number is
    better than month abbreviation that is different for each language.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
&#149; 3 signs your SCM is hindering your productivity
&#149; Requirements for releasing software faster
&#149; Expert tips and advice for migrating your SCM now
http://p.sf.net/sfu/perforce
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2014-05-05 13:52 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <47b112ce.152ba.145b070f2b0.Coremail.eeslook@163.com>
2014-05-05 13:51 ` [LTP] [PATCH] Fix an bad variable name erro in runltp script chrubis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox