Unify the version string to be displayed on "About Chromium" dialog. (4/6)

chrome/tools/build/win/version.bat

Index: chrome/tools/build/win/version.bat
diff --git a/chrome/tools/build/win/version.bat b/chrome/tools/build/win/version.bat
index c224e8aaaea9d053e53259510dd69ae32b23b6dd..5933cebadbb41fa8dacf3989d2d8b3fcb9d6890b 100644
--- a/chrome/tools/build/win/version.bat
+++ b/chrome/tools/build/win/version.bat
@@ -11,6 +11,7 @@ set InFile=%~1
 set SolutionDir=%~2
 set IntDir=%~3
 set OutFile=%~4
+set LastChangeDir=%~5
 set VarsBat=%IntDir%/vers-vars.bat
 
 :: Put cygwin in the path
@@ -34,26 +35,18 @@ call svn --version 2>nul 1>nul
 if errorlevel 1 goto :NO_SVN
 goto :SET_ENV
 
-
 :NO_SVN
 :: Not having svn makes it impossible to determine the current checkout revision
 :: number. On normal build, this is not an issue but for official builds, this
 :: *can't* be tolerated so issue an error instead. VS will pick it up corectly.
 set NO_SVN_LEVEL=error
 if "%OFFICIAL_BUILD%" == "0" set NO_SVN_LEVEL=warning
-echo %0(28) : %NO_SVN_LEVEL% : svn is not installed. Can't determine the revision number.
-echo set LASTCHANGE=0 >> %VarsBat%
-goto :GEN_FILE
-
+echo %0(28) : %NO_SVN_LEVEL% : svn is not installed.
 
 :SET_ENV
-call svn info | grep.exe "Revision:" | cut -d" " -f2- | sed "s/\(.*\)/set LASTCHANGE=\1/" >> %VarsBat%
-goto :GEN_FILE
-
+python %LastChangeDir%\lastchange.py | sed "s/\(.*\)/set \1/" >> %VarsBat%

[M-A Ruel, 2011/07/27 19:11:36]

is sed in path? It'd be nice to convert everything to python, in a follow up
patch.

[Mark Mentovai, 2011/07/27 19:14:03]

Marc-Antoine Ruel wrote:
This script already pipes things to sed, so I think we have to assume so.

 
-:GEN_FILE
 call %VarsBat%
-::echo LastChange: %LASTCHANGE%
 :: output file
 cat %InFile% | sed "s/@MAJOR@/%MAJOR%/" ^
                   | sed "s/@MINOR@/%MINOR%/" ^