Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

fix missing --debug-brk command when trying to debug nodejs with insp… - #99

Closed
bemon wants to merge 1 commit into
microsoft:masterfrom
bemon:master
Closed

bemon wants to merge 1 commit into
microsoft:masterfrom
bemon:master

Conversation

@bemon

@bemon bemon commented Apr 12, 2017

Copy link
Copy Markdown

fix missing --debug-brk command when trying to debug nodejs with inspector.

related to:
nodejs/node#12197
and
nodejs/node#12360 (comment)

@roblourens

Copy link
Copy Markdown
Member

Thanks for noticing this, but --inspect-brk was only introduced in 7.6.0 and I need to support Node versions earlier than that. This is actually really annoying, because I'll have to either detect the Node version before spawning it and decide which argument to use, or detect the 'bad option' error so I can try again with a different argument.

@roblourens

Copy link
Copy Markdown
Member

Filed nodejs/node#12364

@refack

Refael Ackermann (refack) commented Apr 23, 2017

Copy link
Copy Markdown

Hi roblourens as I said in nodejs/node#12364, I'd rather help you solve the issue here, then perpetuate the interim solution of --inspect --debug-brk.
Could you give me a starting point?

@refack

Copy link
Copy Markdown

roblourens if we get newer v6 version to support --inspect-brk:port will that solve this issue? You already have a big please update to v6.8 notice in your readme, you could update it to v6.11?

@refack

Copy link
Copy Markdown

Workaround for the meanwhile
https://gist.github.com/refack/ab07a28580672a67f8e1e0adde359aba

@roblourens roblourens closed this May 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants