MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0001353XdebugFeature/Change requestpublic2016-10-12 05:592016-12-13 21:42
Reportersidharth_k 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityalways
StatusacknowledgedResolutionopen 
PlatformOSOS Version
Product Version2.4.1 
Target VersionFixed in Version 
Summary0001353: Make static function xdebug_dbgp_parse_option() extern
DescriptionDontbug (see https://github.com/sidkshatriya/dontbug [^] ) is a reversible debugger for PHP

Dontbug depends on Xdebug ( see Credits at https://github.com/sidkshatriya/dontbug#credits [^] )

Currently a very small patch needs to be applied to Xdebug to get it to work with Dontbug (see https://github.com/sidkshatriya/dontbug/blob/14ddeda6f527b399c6653e7e41291ccbd28a8c4b/ext/dontbug/xdebug_2_4.make_extern.dontbug.patch [^] )

Basically the patch makes the static function xdebug_dbgp_parse_option() extern.

The rationale for this patch is described here:

https://github.com/sidkshatriya/dontbug/wiki/How-the-Dontbug-Debugger-works#getting-the-value-of-variables-and-inspecting-the-call-stack-1 [^]

I was wondering if we could incorporate the patch into the Xdebug master & 2.4.x branches so that end users will not need to build a patched Xdebug locally in order to use Dontbug. (The extraneous C comments in the patch should be removed, of course, while incorporating it into Xdebug sources)

I would be very grateful if you considered this patch for inclusion. I think this will be a great help in reducing entry barriers to reverse debugging on PHP. Thank you for all the help Xdebug has already given Dontbug.
TagsNo tags attached.
Operating System
PHP Version7.0.10-7.0.14
Attached Files

- Relationships

-  Notes
(0003730)
sidharth_k (reporter)
2016-10-19 15:40
edited on: 2016-10-19 15:44

@derrick Hope you didn't miss this ticket. Would greatly appreciate if you could have a quick look at this! :-)

(0003731)
sidharth_k (reporter)
2016-10-28 19:42
edited on: 2016-10-28 19:42

Filed a corresponding pull request https://github.com/xdebug/xdebug/pull/288 [^]

(0004057)
derick (administrator)
2016-12-13 21:42

I might want to implement this, but it's not high on my list of things to do.

- Issue History
Date Modified Username Field Change
2016-10-12 05:59 sidharth_k New Issue
2016-10-19 15:40 sidharth_k Note Added: 0003730
2016-10-19 15:44 sidharth_k Note Edited: 0003730 View Revisions
2016-10-28 19:42 sidharth_k Note Added: 0003731
2016-10-28 19:42 sidharth_k Note Edited: 0003731 View Revisions
2016-12-13 21:42 derick Status new => acknowledged
2016-12-13 21:42 derick Note Added: 0004057


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker