Add a safeguard because I always use -p 8000 by error when I mean --port 8000.

Add a safeguard because I always use -p 8000 by error when I mean --port 8000.

R=dpranke@chromium.org
BUG=
TEST=


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

[Dirk Pranke, 2011-10-13 00:46:33]

lgtm.

http://codereview.chromium.org/8261001/diff/3001/trychange.py
File trychange.py (right):

http://codereview.chromium.org/8261001/diff/3001/trychange.py#newcode662
trychange.py:662: 'Have you tried --port instead? You probably confused -p and
-P.')
Nit: I'd probably use something like 100 instead of 10. I've certainly seen more
than 10 patches on some issues.

[M-A Ruel, 2011-10-13 00:47:32]

http://codereview.chromium.org/8261001/diff/3001/trychange.py
File trychange.py (right):

http://codereview.chromium.org/8261001/diff/3001/trychange.py#newcode662
trychange.py:662: 'Have you tried --port instead? You probably confused -p and
-P.')
On 2011/10/13 00:46:33, Dirk Pranke wrote:
> Nit: I'd probably use something like 100 instead of 10. I've certainly seen
more
> than 10 patches on some issues.

No, it's patchlevel, like patch -p1, not the number of patchset.

[Dirk Pranke, 2011-10-13 00:48:34]

Ah. Right then.

On Wed, Oct 12, 2011 at 5:47 PM,  <maruel@chromium.org> wrote:
>
> http://codereview.chromium.org/8261001/diff/3001/trychange.py
> File trychange.py (right):
>
> http://codereview.chromium.org/8261001/diff/3001/trychange.py#newcode662
> trychange.py:662: 'Have you tried --port instead? You probably confused
> -p and -P.')
> On 2011/10/13 00:46:33, Dirk Pranke wrote:
>>
>> Nit: I'd probably use something like 100 instead of 10. I've certainly
>
> seen more
>>
>> than 10 patches on some issues.
>
> No, it's patchlevel, like patch -p1, not the number of patchset.
>
> http://codereview.chromium.org/8261001/
>

[commit-bot: I haz the power, 2011-10-13 00:51:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/8261001/3001

[commit-bot: I haz the power, 2011-10-13 00:58:09]

Change committed as 105226