Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

BUG=

Committed: https://crrev.com/899f5b3f086923998716888ebd1d38168f4a25aa
Cr-Commit-Position: refs/heads/master@{#415623}

[cathiechentx, 2016-08-31 07:49:55]

Description was changed from

==========
Fixing the pushing gdbserver problem while seting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

to

==========
Fixing the pushing gdbserver problem while seting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

[cathiechentx, 2016-08-31 07:49:55]

cathiechen@tencent.com changed reviewers:
+ jamesxwei@tencent.com, jbudorick@chromium.org

[cathiechentx, 2016-08-31 07:55:28]

Description was changed from

==========
Fixing the pushing gdbserver problem while seting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

to

==========
Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

[cathiechentx, 2016-08-31 08:17:29]

hi jbudorick,
adb_gdb_content_shell won't work when su-prefix set. The COMMAND_SUFFIX has been
missed while pushing and removing gdbserver.

[jbudorick, 2016-08-31 13:01:41]

lgtm

[jbudorick, 2016-08-31 13:02:05]

On 2016/08/31 08:17:29, cathiechentx wrote:
> hi jbudorick,
> adb_gdb_content_shell won't work when su-prefix set. The COMMAND_SUFFIX has
been
> missed while pushing and removing gdbserver.

Thanks for the patch!

[wistoch.tencent, 2016-08-31 13:03:57]

The CQ bit was checked by wistoch.tencent@gmail.com

[commit-bot: I haz the power, 2016-08-31 13:04:05]

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-08-31 13:50:16]

Description was changed from

==========
Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

to

==========
Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

[commit-bot: I haz the power, 2016-08-31 13:50:17]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-31 13:52:55]

Description was changed from

==========
Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

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

to

==========
Fixing the pushing gdbserver problem while setting --su-prefix="su -c"

COMMAND_PREFIX and COMMAND_SUFFIX should be used simultaneously

BUG=

Committed: https://crrev.com/899f5b3f086923998716888ebd1d38168f4a25aa
Cr-Commit-Position: refs/heads/master@{#415623}
==========

[commit-bot: I haz the power, 2016-08-31 13:52:56]

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