Add script to apply a patch from rietveld.

Add script to apply a patch from rietveld.

At the moment it just fetches the patch. It's still useful to debug rietveld
issues.

Improve resiliency to invalid patches.

R=dpranke@chromium.org
BUG=
TEST=

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

[Dirk Pranke, 2011-04-12 20:44:28]

git-cl already has this logic. Perhaps it makes more sense to have this be a
command on gcl rather than a separate script?

[M-A Ruel, 2011-04-12 20:45:20]

On 2011/04/12 20:44:28, Dirk Pranke wrote:
> git-cl already has this logic. Perhaps it makes more sense to have this be a
> command on gcl rather than a separate script?

The try server won't use git-cl or gcl.

[Dirk Pranke, 2011-04-12 20:52:12]

LGTM.

http://codereview.chromium.org/6825085/diff/1/apply_issue.py
File apply_issue.py (right):

http://codereview.chromium.org/6825085/diff/1/apply_issue.py#newcode23
apply_issue.py:23: '-i', '--issue', type='int', help='Rietveld issue number')
Maybe this should be a positional parameter instead of optional, since it's
pretty much mandatory, right?

[M-A Ruel, 2011-04-12 20:58:23]

http://codereview.chromium.org/6825085/diff/1/apply_issue.py
File apply_issue.py (right):

http://codereview.chromium.org/6825085/diff/1/apply_issue.py#newcode23
apply_issue.py:23: '-i', '--issue', type='int', help='Rietveld issue number')
On 2011/04/12 20:52:12, Dirk Pranke wrote:
> Maybe this should be a positional parameter instead of optional, since it's
> pretty much mandatory, right?

Right. I've just set it this way since it's stricter and reduces potential
issues from changing build infrastructure. It eases the upgrade path when
necessary.

[Dirk Pranke, 2011-04-12 20:59:54]

I'm not sure how it's stricter, but the changing build infrastructure
part makes sense.

On Tue, Apr 12, 2011 at 1:58 PM,  <maruel@chromium.org> wrote:
>
> http://codereview.chromium.org/6825085/diff/1/apply_issue.py
> File apply_issue.py (right):
>
> http://codereview.chromium.org/6825085/diff/1/apply_issue.py#newcode23
> apply_issue.py:23: '-i', '--issue', type='int', help='Rietveld issue
> number')
> On 2011/04/12 20:52:12, Dirk Pranke wrote:
>>
>> Maybe this should be a positional parameter instead of optional, since
>
> it's
>>
>> pretty much mandatory, right?
>
> Right. I've just set it this way since it's stricter and reduces
> potential issues from changing build infrastructure. It eases the
> upgrade path when necessary.
>
> http://codereview.chromium.org/6825085/
>