-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Nodejs 8 (node:4376) [DEP0062] DeprecationWarning: node --debug and node --debug-brk are invalid. Please use node --inspect or node --inspect-brk instead. #220
Comments
I still have the same issue, that I found through issue #107 . I am using nodeclipse 1.0.2, which seems to be the latest version. Is the fix included in this version? |
I could really use this. Is there anything I can do to help? |
build, try that everything works fine then I will release or help to release |
Also need to know at what Node.js version this option where renamed, |
I am having the issue with Node.js v8.11.3. Haven't tested with Node.js 10 yet, but I plan to later. |
Unfortunately it's not a renamed option but a completely new replacement. According to the Node.js V6 change log the
So the issue is not that the the VM failed to launch because of the changed option, but that the debugger can't connect to the VM because the protocol has changed. I did go ahead and changing the option in LaunchConfigurationDelegate.java which made the depreciation warning go away, but it still thinks the VM failed to launch. |
So I'm guessing the continuation of this issue when running Node-v10.15.2 is expected, yet not desired? Is the only solution to downgrade Node.JS to pre-V8? |
I am not using Node for few years, so don't have real reason to follow node version changes. Some one should step in, willing to solve the issue with Java code. HELP WANTED. |
paulvi, it's been many years since I did Eclipse plugin development, so I'm not sure I can help. But isn't it just a matter of changing the nodeclipse plugin's launch configuration of the Node app (e.g., /usr/local/bin/node) to use --inspect or --inspect-brk instead of the deprecated --debug or --debug-brk parameter? |
Sorry, just seeing Fr33dan's comment re. change in protocol above. |
Mostly time is need for testing this change, I can help with the last. I don't expect someone is going to take over this project. Otherwise I would explain and help to do release by other person. |
Over a year and still no progress on this issue -- Wow! |
It will not be, until someone steps in to actually do it. I am not using Node.js any more, so not going to follow what changes with every version of Node |
Please fix this command for replace --debug to --inspect. The comment #220 (comment) is correct! C:\Users\muletto10>npm -v C:\Users\muletto10>node -v C:\Users\muletto10>java -version C:\Users\muletto10>node.exe --inspect-brk=1337 C:/Users/muletto10/eclipse-workspace-VA/first/hello-world-server.js |
Well , if I do "run Node Program" it work |
Initially raised as Nodeclipse/www.nodeclipse.org#77
It seems that Nodejs 8 change option names
The line in question is https://github.com/Nodeclipse/nodeclipse-1/blob/master/org.nodeclipse.debug/src/org/nodeclipse/debug/launch/LaunchConfigurationDelegate.java#L114
i.e. using
--debug
is hard-coded nowTODO test with Node.js 8, if using
--inspect
would work.HELP WANTED!!
The text was updated successfully, but these errors were encountered: