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 intomicrosoft:masterfrom
bemon:master
Closed

fix missing --debug-brk command when trying to debug nodejs with insp…#99
bemon wants to merge 1 commit intomicrosoft:masterfrom
bemon:master

Conversation

@bemon
Copy link
Copy Markdown

@bemon bemon commented Apr 12, 2017

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
Copy link
Copy Markdown

refack commented Apr 23, 2017

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

refack commented Apr 24, 2017

@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

refack commented May 8, 2017

@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