Fix for issue 9685:

Fix for issue 9685:
Use git log | grep .. instead of git svn info

[tedoc2000, 2009-04-03 15:48:18]

Here is a fix for issue 9685 that we discussed on IRC two nights ago.

[tedoc2000, 2009-04-03 15:57:57]

Switch to sgk

[sgk, 2009-04-03 16:22:33]

http://codereview.chromium.org/60094/diff/1/2
File chrome/tools/build/linux/version.sh (right):

http://codereview.chromium.org/60094/diff/1/2#newcode30
Line 30: LASTCHANGE=$(git log | grep '^\s*git-svn-id:' 2>/dev/null | head -1 |
perl -pi -e 's/^\s*git-svn-id:\s+(.*)\@(\d+)\s([a-f\d\-]+)$/\2/;')
Please simplify this.  If you're invoking Perl to pick the change number out of
the line, then we shouldn't need grep and head to select the first line, but can
just exit after the match.  Also, the first and third () groups in the regular
expression aren't contributing anything and should be dropped.

[tedoc2000, 2009-04-03 17:24:39]

Fixed up based on sgk feedback

[sgk, 2009-04-03 17:31:42]

lgtm

[tedoc2000, 2009-04-03 17:31:48]

Got an improvement on the perl from ion_ on IRC

[tedoc2000, 2009-04-03 17:46:57]

Missed $1 substitution.

[tony, 2009-04-03 17:48:41]

I'm not opposed to this, but realize that the comment in the header says that we
should replace this with a python script and merge with the windows .bat script.
 It would probably be better to focus on that (most of our scripts are already
in python).

[Michael Moss, 2009-04-03 17:51:11]

>  It would probably be better to focus on that (most of our scripts are already
> in python).

This is fairly minor, so I'm fine with it until I get a chance to rewrite
things.

[sgk, 2009-04-03 18:09:41]

still lgtm w/$1 subst

[sgk, 2009-04-03 20:20:35]

Committed as r13101.  Thanks,