[tools] Flexible perf runner path.

[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true

Committed: https://crrev.com/0eb083155c83e2272eb2946fc4bfe0b3eb400481
Cr-Commit-Position: refs/heads/master@{#33649}

[Mircea Trofin, 2016-02-01 16:04:56]

Description was changed from

==========
[tools] Flexible perf runner path.
Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

BUG=
==========

to

==========
Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

BUG=
==========

[Mircea Trofin, 2016-02-01 16:04:57]

mtrofin@chromium.org changed reviewers:
+ machenbach@chromium.org

[Michael Achenbach, 2016-02-01 19:59:57]

Description was changed from

==========
Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

BUG=
==========

to

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

BUG=
==========

[Michael Achenbach, 2016-02-01 20:02:24]

lgtm with comment:

https://codereview.chromium.org/1659483003/diff/1/tools/run_perf.py
File tools/run_perf.py (right):

https://codereview.chromium.org/1659483003/diff/1/tools/run_perf.py#newcode858
tools/run_perf.py:858: print "specify either binary-override-path or
shell-dir-no-patch"
nit: The option is called outdir-no-patch.

[Michael Achenbach, 2016-02-01 20:02:52]

Note, I copied the subject line to the first line of the CL desc.

[Michael Achenbach, 2016-02-01 20:03:01]

Description was changed from

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

BUG=
==========

to

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true
==========

[Mircea Trofin, 2016-02-01 20:25:45]

The CQ bit was checked by mtrofin@chromium.org

[Mircea Trofin, 2016-02-01 20:25:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
machenbach@chromium.org
Link to the patchset: https://codereview.chromium.org/1659483003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-02-01 20:25:51]

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

[Mircea Trofin, 2016-02-01 20:25:53]

https://codereview.chromium.org/1659483003/diff/1/tools/run_perf.py
File tools/run_perf.py (right):

https://codereview.chromium.org/1659483003/diff/1/tools/run_perf.py#newcode858
tools/run_perf.py:858: print "specify either binary-override-path or
shell-dir-no-patch"
On 2016/02/01 20:02:24, Michael Achenbach wrote:
> nit: The option is called outdir-no-patch.

Done.

[commit-bot: I haz the power, 2016-02-01 20:27:26]

Description was changed from

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true
==========

to

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true
==========

[commit-bot: I haz the power, 2016-02-01 20:27:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-01 20:27:57]

Description was changed from

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true
==========

to

==========
[tools] Flexible perf runner path.

Improved flexibility for the perf runner, by adding option to
specify precisely shell binary.

NOTRY=true

Committed: https://crrev.com/0eb083155c83e2272eb2946fc4bfe0b3eb400481
Cr-Commit-Position: refs/heads/master@{#33649}
==========

[commit-bot: I haz the power, 2016-02-01 20:27:58]

Patchset 2 (id:??) landed as
https://crrev.com/0eb083155c83e2272eb2946fc4bfe0b3eb400481
Cr-Commit-Position: refs/heads/master@{#33649}