roll-dep: If applicable, include svn revision range in commit message

roll-dep: If applicable, include svn revision range in commit message

This is analogous to the change made in
https://codereview.chromium.org/559243002/.  I want to change ARB to use the
commit message generated by roll-dep.

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

[borenet, 2014-09-15 17:05:50]

borenet@google.com changed reviewers:
+ szager@chromium.org

[borenet, 2014-09-15 17:05:51]

Sample commit messages generated after this change:

Skia:
"""
Roll src/third_party/skia from dc53701216c0 to a63d5df455e5

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/dc53701216c0..a63d5df455e5
"""

Blink:
"""
Roll src/third_party/WebKit from 110f8a11d9b4 to ff4e39b767aa (svn
180973:181036)
"""

The description is longer than 70 chars (81) now.  I could shorten by changing
to something like:

"""
Roll src/third_party/WebKit 110f8a11d9b4:ff4e39b767aa (180973:181036)
"""

Which is 69 chars.

[skobes, 2014-09-15 18:26:29]

LGTM

On 2014/09/15 17:05:51, borenet wrote:
> The description is longer than 70 chars (81) now.  I could shorten by changing
> to something like:
> 
> """
> Roll src/third_party/WebKit 110f8a11d9b4:ff4e39b767aa (180973:181036)
> """
> 
> Which is 69 chars.

You could also truncate the git hashes to 7 chars instead of 12, which would
match what ARB currently does (via git rev-parse --short).

[iannucci, 2014-09-15 18:45:14]

On 2014/09/15 18:26:29, skobes wrote:
> LGTM
> 
> On 2014/09/15 17:05:51, borenet wrote:
> > The description is longer than 70 chars (81) now.  I could shorten by
changing
> > to something like:
> > 
> > """
> > Roll src/third_party/WebKit 110f8a11d9b4:ff4e39b767aa (180973:181036)
> > """
> > 
> > Which is 69 chars.
> 
> You could also truncate the git hashes to 7 chars instead of 12, which would
> match what ARB currently does (via git rev-parse --short).

Definitely use `git rev-parse --short`, since it will return the shortest
(currently) non-ambiguous hash.

[borenet, 2014-09-15 18:51:15]

On 2014/09/15 18:45:14, iannucci wrote:
> On 2014/09/15 18:26:29, skobes wrote:
> > LGTM
> > 
> > On 2014/09/15 17:05:51, borenet wrote:
> > > The description is longer than 70 chars (81) now.  I could shorten by
> changing
> > > to something like:
> > > 
> > > """
> > > Roll src/third_party/WebKit 110f8a11d9b4:ff4e39b767aa (180973:181036)
> > > """
> > > 
> > > Which is 69 chars.
> > 
> > You could also truncate the git hashes to 7 chars instead of 12, which would
> > match what ARB currently does (via git rev-parse --short).
> 
> Definitely use `git rev-parse --short`, since it will return the shortest
> (currently) non-ambiguous hash.

Done in patch set 2.  New commit message looks like:

"""
Roll src/third_party/WebKit ff4e39b:ec72efd (svn 181036:181991)

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/ff4e39b..ec72efd
"""

63 chars in title line.

[borenet, 2014-09-15 20:01:31]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-09-15 20:02:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/564363003/20001

[commit-bot: I haz the power, 2014-09-15 20:03:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-15 20:03:54]

Presubmit check for 564363003-20001 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    depot_tools/roll_dep.py

Presubmit checks took 105.1s to calculate.

[szager1, 2014-09-15 20:57:43]

lgtm

[borenet, 2014-09-15 20:58:09]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-09-15 20:58:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/564363003/20001

[commit-bot: I haz the power, 2014-09-15 21:00:25]

Committed patchset #2 (id:20001) as 291975