Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(513)

Issue 808053006: Make --gdb work again by using gdbserver (Closed)

Created:
5 years, 11 months ago by eseidel
Modified:
5 years, 11 months ago
Reviewers:
qsr, abarth-chromium
CC:
abarth-chromium, esprehn, mojo-reviews_chromium.org, ojan
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Make --gdb work again by using gdbserver --gdb now starts gdbserver and I've added a new command gdb_attach which knows how to launch gdb and connect to the launched gdbserver This doesn't quite work on android yet, but it's very close. R=abarth@chromium.org, qsr@chromium.org BUG= Committed: https://chromium.googlesource.com/external/mojo/+/4656d85f590e4776eacd1b020b37d2df83a1e829

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+49 lines, -14 lines) Patch
M sky/tools/skydb View 7 chunks +49 lines, -14 lines 1 comment Download

Messages

Total messages: 4 (0 generated)
eseidel
5 years, 11 months ago (2015-01-13 20:10:38 UTC) #1
eseidel
https://codereview.chromium.org/808053006/diff/1/sky/tools/skydb File sky/tools/skydb (right): https://codereview.chromium.org/808053006/diff/1/sky/tools/skydb#newcode226 sky/tools/skydb:226: if 'remote_sky_server_port' in self.pids: This was just checking for ...
5 years, 11 months ago (2015-01-13 20:11:32 UTC) #2
abarth-chromium
lgtm
5 years, 11 months ago (2015-01-13 20:12:59 UTC) #3
eseidel
5 years, 11 months ago (2015-01-13 20:16:34 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
4656d85f590e4776eacd1b020b37d2df83a1e829 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698