Don't eat newlines in CL descriptions when merging changes using drover.

Don't eat newlines in CL descriptions when merging changes using drover.

BUG=80589
TEST=Merge a CL using drover and when prompted to upload, check that the original CL description still has its linebreaks.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=83561

[M-A Ruel, 2011-04-29 18:29:17]

http://codereview.chromium.org/6893095/diff/1/drover.py
File drover.py (right):

http://codereview.chromium.org/6893095/diff/1/drover.py#newcode121
drover.py:121: ['svn', 'log', url, '-r', str(revision)]).splitlines()
Replace this whole function with:

svn_log = subprocess2.check_output(
['svn', 'log', url, '-r', str(revision)],
universal_newlines=True
).splitlines(True)
return ''.join(svn_log[3:-1])

[Robert Sesek, 2011-04-29 18:38:41]

http://codereview.chromium.org/6893095/diff/1/drover.py
File drover.py (right):

http://codereview.chromium.org/6893095/diff/1/drover.py#newcode121
drover.py:121: ['svn', 'log', url, '-r', str(revision)]).splitlines()
On 2011/04/29 18:29:17, Marc-Antoine Ruel wrote:
> Replace this whole function with:
> 
> svn_log = subprocess2.check_output(
> ['svn', 'log', url, '-r', str(revision)],
> universal_newlines=True
> ).splitlines(True)
> return ''.join(svn_log[3:-1])

Done.

[M-A Ruel, 2011-04-29 18:39:20]

lgtm