Closed
Bug 1105812
Opened 11 years ago
Closed 10 years ago
run-gdb.sh should allow overriding GDB_PORT so as to more easily allow running gdb over ssh
Categories
(Firefox OS Graveyard :: GonkIntegration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
FxOS-S7 (18Sep)
People
(Reporter: kats, Assigned: kats)
Details
Attachments
(1 file)
Using jim chen's instructions at http://www.jnchen.com/blog/2012/11/tunnelling-adb I can easily use adb on a device that is connected to a remote machine, but gdb is not so easy. If the run-gdb.sh script allowed the user to force a well-known port to use for GDB_PORT then this would work. You wouldn't be able to run-gdb on multiple processes simultaneously, but it's better than nothing.
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8529817 -
Flags: review?(dhylands)
| Assignee | ||
Updated•11 years ago
|
Summary: run-gdb.sh should allow overriding GDB_PORT so as to more easily allow tunning gdb over ssh → run-gdb.sh should allow overriding GDB_PORT so as to more easily allow running gdb over ssh
Comment 2•11 years ago
|
||
Comment on attachment 8529817 [details] [review]
Github PR
Clearing review request for now (I'd like to see it after changes are made).
Attachment #8529817 -
Flags: review?(dhylands)
| Assignee | ||
Comment 3•11 years ago
|
||
I'll get to it eventually but feel free to steal it if you want it sooner.
| Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8529817 [details] [review]
Github PR
Finally got around to updating this; see the latest version of the patch in the pull request.
Attachment #8529817 -
Flags: review?(dhylands)
Comment 5•10 years ago
|
||
Comment on attachment 8529817 [details] [review]
Github PR
r=me if you don't remove the lines in README.md about "To restart B2G and run B2G under gdb" (since they're still correct).
Attachment #8529817 -
Flags: review?(dhylands) → review+
| Assignee | ||
Comment 6•10 years ago
|
||
When I run the command it usually attaches to B2G without restarting. I can leave that bit in though.
| Assignee | ||
Comment 7•10 years ago
|
||
Updated PR with the requested change. Flagging checkin-needed for merging the PR to the B2G repo.
Keywords: checkin-needed
Comment 8•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S7 (18Sep)
You need to log in
before you can comment on or make changes to this bug.
Description
•