Fix roll_dep.py to print the right latest SVN revision number on error.

Fix roll_dep.py to print the right latest SVN revision number on error.

TEST="roll_dep.py breakpad/src 10000" should print ~1387, not 4.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=292129

[Lei Zhang, 2014-09-24 18:56:36]

thestig@chromium.org changed reviewers:
+ szager@chromium.org

[szager1, 2014-09-24 23:36:33]

lgtm with nit, thanks.

https://codereview.chromium.org/604463002/diff/20001/roll_dep.py
File roll_dep.py (right):

https://codereview.chromium.org/604463002/diff/20001/roll_dep.py#newcode130
roll_dep.py:130: if not latest_svn_rev:
if not latest_svn_rev or int(svn_rev) > int(latest_svn_rev)

Necessary because this code loops over ('HEAD', 'origin/master')

[Lei Zhang (Do not use), 2014-09-24 23:45:38]

thestig@google.com changed reviewers:
+ thestig@google.com

[Lei Zhang (Do not use), 2014-09-24 23:45:39]

https://codereview.chromium.org/604463002/diff/20001/roll_dep.py
File roll_dep.py (right):

https://codereview.chromium.org/604463002/diff/20001/roll_dep.py#newcode130
roll_dep.py:130: if not latest_svn_rev:
On 2014/09/24 23:36:33, szager1 wrote:
> if not latest_svn_rev or int(svn_rev) > int(latest_svn_rev)
> 
> Necessary because this code loops over ('HEAD', 'origin/master')

Done.

[Lei Zhang (Do not use), 2014-09-24 23:45:43]

The CQ bit was checked by thestig@google.com

[commit-bot: I haz the power, 2014-09-24 23:46:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/604463002/40001

[commit-bot: I haz the power, 2014-09-24 23:48:36]

Committed patchset #3 (id:40001) as 292129