Improved git cl patch generated commit message to include patchset information

Improved git cl patch generated commit message to include patchset information
and a link to it.

BUG=

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

[carlosk, 2014-11-14 09:43:39]

carlosk@chromium.org changed reviewers:
+ brettw@chromium.org

[carlosk, 2014-11-14 09:43:40]

brettw@: PTAL?

Just a minor patch that I found useful while using git cl patch to get other
people CLs that were still being iterated on.

[Dirk Pranke, 2014-11-14 18:00:36]

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

[Dirk Pranke, 2014-11-14 18:00:36]

lgtm w/ nit.

https://codereview.chromium.org/731473002/diff/1/git_cl.py
File git_cl.py (right):

https://codereview.chromium.org/731473002/diff/1/git_cl.py#newcode2375
git_cl.py:2375: % {'i': issue, 'p': patchset}])
Nit: stylistically, we'd put the multiline string literal inside parens to make
it  a bit clearer that the strings should be treated as one an concatenated,
i.e.:

RunGit(['commit', '-m', ('...'
                         '...'
                         % { .. })])

[carlosk, 2014-11-14 18:10:24]

Thanks.

IDK if trybots are a thing for this project as I tried git cl try and got an
error message. I hope just pressing commit should be enough...

https://codereview.chromium.org/731473002/diff/1/git_cl.py
File git_cl.py (right):

https://codereview.chromium.org/731473002/diff/1/git_cl.py#newcode2375
git_cl.py:2375: % {'i': issue, 'p': patchset}])
On 2014/11/14 18:00:36, Dirk Pranke wrote:
> Nit: stylistically, we'd put the multiline string literal inside parens to
make
> it  a bit clearer that the strings should be treated as one an concatenated,
> i.e.:
> 
> RunGit(['commit', '-m', ('...'
>                          '...'
>                          % { .. })])

Done.

[carlosk, 2014-11-14 18:10:37]

The CQ bit was checked by carlosk@chromium.org

[commit-bot: I haz the power, 2014-11-14 18:10:46]

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

[commit-bot: I haz the power, 2014-11-14 18:12:52]

Committed patchset #2 (id:20001) as 292940

[Dirk Pranke, 2014-11-14 18:30:08]

On 2014/11/14 18:10:24, carlosk wrote:
> Thanks.
> 
> IDK if trybots are a thing for this project as I tried git cl try and got an
> error message. I hope just pressing commit should be enough...

Nope, there are no trybots, just the CQ, and that should be good enough.