Patchwork [U-Boot] tools: fixed svn revision l10n-ism

login
register
mail settings
Submitter Stephane Fillod
Date June 1, 2011, 3:02 p.m.
Message ID <456755229.9445841306940550442.JavaMail.root@zimbra4-e1.priv.proxad.net>
Download mbox | patch
Permalink /patch/98206/
State Changes Requested
Headers show

Comments

Stephane Fillod - June 1, 2011, 3:02 p.m.
Fixed a l10n-ism in svn revision retrieval.

Signed-off-by: Stephane Fillod <fillods@users.sf.net>
---
Mike Frysinger - June 1, 2011, 3:19 p.m.
On Wednesday, June 01, 2011 11:02:30 Stephane Fillod wrote:
> -if rev=`svn info 2>/dev/null` ; then
> +if rev=`LANG=C svn info 2>/dev/null` ; then
>  	rev=`echo "${rev}" | grep '^Revision' | awk '{print $NF}'`

LANG=C wont fix the bug, just make it work in more cases.  you need LC_ALL=C.
-mike

Patch

--- a/tools/setlocalversion
+++ b/tools/setlocalversion
@@ -31,7 +31,7 @@ 
 fi

 # Check for svn and a svn repo.
-if rev=`svn info 2>/dev/null` ; then
+if rev=`LANG=C svn info 2>/dev/null` ; then
 	rev=`echo "${rev}" | grep '^Revision' | awk '{print $NF}'`
 	printf -- '-svn%s' $rev
 fi