Modify roll_dep's commit message

Modify roll_dep's commit message

This conforms to the convention of using a single-line short description
followed by an empty line followed by a long description. Shortened the
revisions to more easily fit on one line.

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

[borenet, 2014-08-28 20:07:53]

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

[szager1, 2014-08-28 21:23:15]

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

[szager1, 2014-08-28 21:23:15]

https://codereview.chromium.org/513383002/diff/1/roll_dep.py
File roll_dep.py (right):

https://codereview.chromium.org/513383002/diff/1/roll_dep.py#newcode256
roll_dep.py:256: dep_name, old_rev[:12], new_rev[:12], url))
Can we use ellipses for over-long dep names?  Something like:

while len(dep_name) > 40:
  dep_name = '.../' + dep_name.lstrip('./').partition('/')[2]

[szager1, 2014-08-28 21:24:52]

https://codereview.chromium.org/513383002/diff/1/roll_dep.py
File roll_dep.py (right):

https://codereview.chromium.org/513383002/diff/1/roll_dep.py#newcode32
roll_dep.py:32: Summary of changes available at: %s
Let's move the link to another line, to hopefully prevent overflow:

Summary of changes available at:
    %s

[borenet, 2014-08-29 13:32:02]

https://codereview.chromium.org/513383002/diff/1/roll_dep.py
File roll_dep.py (right):

https://codereview.chromium.org/513383002/diff/1/roll_dep.py#newcode32
roll_dep.py:32: Summary of changes available at: %s
On 2014/08/28 21:24:51, szager1 wrote:
> Let's move the link to another line, to hopefully prevent overflow:
> 
> Summary of changes available at:
>     %s

Done.

https://codereview.chromium.org/513383002/diff/1/roll_dep.py#newcode256
roll_dep.py:256: dep_name, old_rev[:12], new_rev[:12], url))
On 2014/08/28 21:23:15, szager1 wrote:
> Can we use ellipses for over-long dep names?  Something like:
> 
> while len(dep_name) > 40:
>   dep_name = '.../' + dep_name.lstrip('./').partition('/')[2]

Done.  Added a helper function since my intention is to have the AutoRoll Bot
import this file and use the ROLL_DESCRIPTION_STR directly when searching for
in-progress rolls.  Here's an example commit message where the dep_name was
shortened:

"""
Roll .../WebKit/LayoutTests/w3c/csswg-test from 0b12ea22689e to 198f08fb5364

Summary of changes available at:
   
https://chromium.googlesource.com/external/w3c/csswg-test/+log/0b12ea22689e.....
"""

The title line is under 80 chars, but the log link overflows.

[szager1, 2014-08-29 21:09:07]

lgtm % nits

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

https://codereview.chromium.org/513383002/diff/20001/roll_dep.py#newcode33
roll_dep.py:33: %s
Let's just get rid of the leading spaces, they serve no real purpose, and might
cause overflow when it need not happen:

Summary of changes:

%s

https://codereview.chromium.org/513383002/diff/20001/roll_dep.py#newcode39
roll_dep.py:39: while len(dep) > 40:
Let's make it 31 rather than 40; git commit topic lines are supposed to be <= 70
characters.

[borenet, 2014-08-29 21:14:23]

Uploaded patch set 3.

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

https://codereview.chromium.org/513383002/diff/20001/roll_dep.py#newcode33
roll_dep.py:33: %s
On 2014/08/29 21:09:07, szager1 wrote:
> Let's just get rid of the leading spaces, they serve no real purpose, and
might
> cause overflow when it need not happen:
> 
> Summary of changes:
> 
> %s

Done.

https://codereview.chromium.org/513383002/diff/20001/roll_dep.py#newcode39
roll_dep.py:39: while len(dep) > 40:
On 2014/08/29 21:09:07, szager1 wrote:
> Let's make it 31 rather than 40; git commit topic lines are supposed to be <=
70
> characters.

Done.

[borenet, 2014-08-29 21:25:03]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-08-29 22:00:55]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/borenet@google.com/513383002/40001

[commit-bot: I haz the power, 2014-08-29 22:03:09]

Committed patchset #3 (id:40001) as 291743