Remove trailing garbage from bisect-builds.py

Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true

Committed: https://crrev.com/a898d221e7a213f2b011c1751276a55da6f5a0c4
Cr-Commit-Position: refs/heads/master@{#404847}

[f(malita), 2016-07-12 17:40:31]

Description was changed from

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

TBR=dimu@google.com
==========

to

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
==========

[f(malita), 2016-07-12 17:40:31]

fmalita@chromium.org changed reviewers:
+ dpranke@chromium.org, thakis@chromium.org
- dimu@google.com

[Nico, 2016-07-12 17:47:29]

lgtm

dimu, how come https://codereview.chromium.org/2136353002/ was landed without
review?

[f(malita), 2016-07-12 17:48:04]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-07-12 17:48:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dimu, 2016-07-12 18:00:16]

On 2016/07/12 17:47:29, Nico wrote:
> lgtm
> 
> dimu, how come https://codereview.chromium.org/2136353002/ was landed without
> review?

Sry I was having some authentication issues committing to src.git yesterday, and
I created several CLs to retry and it turned out that I submitted the wrong one.

The real reviewed CL is
https://codereview.chromium.org/2136143002/patch/20001/30001

[Nico, 2016-07-12 18:01:19]

On 2016/07/12 18:00:16, dimu wrote:
> On 2016/07/12 17:47:29, Nico wrote:
> > lgtm
> > 
> > dimu, how come https://codereview.chromium.org/2136353002/ was landed
without
> > review?
> 
> Sry I was having some authentication issues committing to src.git yesterday,
and
> I created several CLs to retry and it turned out that I submitted the wrong
one.
> 
> The real reviewed CL is
> https://codereview.chromium.org/2136143002/patch/20001/30001

Ah cool. What I'd recommend to do in this case is to say "Reviewed at <link>" in
the newly-uploaded CL.

[dimu, 2016-07-12 18:10:48]

On 2016/07/12 18:01:19, Nico wrote:
> On 2016/07/12 18:00:16, dimu wrote:
> > On 2016/07/12 17:47:29, Nico wrote:
> > > lgtm
> > > 
> > > dimu, how come https://codereview.chromium.org/2136353002/ was landed
> without
> > > review?
> > 
> > Sry I was having some authentication issues committing to src.git yesterday,
> and
> > I created several CLs to retry and it turned out that I submitted the wrong
> one.
> > 
> > The real reviewed CL is
> > https://codereview.chromium.org/2136143002/patch/20001/30001
> 
> Ah cool. What I'd recommend to do in this case is to say "Reviewed at <link>"
in
> the newly-uploaded CL.

sure I've just modified the description of the uploaded CL. Thanks!

[f(malita), 2016-07-12 21:01:18]

The CQ bit was unchecked by fmalita@chromium.org

[f(malita), 2016-07-12 21:01:31]

Description was changed from

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
==========

to

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true
==========

[f(malita), 2016-07-12 21:01:36]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-07-12 21:04:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-12 22:29:17]

Description was changed from

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true
==========

to

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true
==========

[commit-bot: I haz the power, 2016-07-12 22:29:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-12 22:31:35]

Description was changed from

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true
==========

to

==========
Remove trailing garbage from bisect-builds.py

  File "tools/bisect-builds.py", line 897
    return m.group(1)]
                     ^
SyntaxError: invalid syntax

(introduced in http://crrev.com/2136353002)

R=dpranke@chromium.org,thakis@chromium.org
NOTRY=true

Committed: https://crrev.com/a898d221e7a213f2b011c1751276a55da6f5a0c4
Cr-Commit-Position: refs/heads/master@{#404847}
==========

[commit-bot: I haz the power, 2016-07-12 22:31:36]

Patchset 1 (id:??) landed as
https://crrev.com/a898d221e7a213f2b011c1751276a55da6f5a0c4
Cr-Commit-Position: refs/heads/master@{#404847}