Debugger requires path to program be correct casing for breakpoints to get hit about vscode-node-debug HOT 5 CLOSED

@weinand, thanks for porting over. I see you closed the issue, but the resolution is unclear.

from vscode-node-debug.

@martellaj my "Issue Mover" tool closed the issue in the vscode repo and moved it to the vscode-node-debug repo. But here the issue is still open and there is no resolution yet.

from vscode-node-debug.

Oh okay, I could've sworn it was closed here, too. Time to go get ☕. Thanks!

from vscode-node-debug.

@martellaj on a case-sensitive filesystem (e.g. on linux) your setup "foo/app.js" will not work if the file on disk actually lives in a folder "Foo". So I would call your setup at least 'dangerous' (and I was hit by this many times because I develop on a case insensitive file system but run builds on linux).

So from this perspective, I would rather try to flag the case-mismatch of the 'program' attribute as an error than make breakpoints work in this case. In addition another problem lurks around the corner: if you require('App.js')this will work on a case insensitive filesystem but again the breakpoints in 'app.js' will not be hit, because for node and the debugger the source lives in 'App.js'.

Bottom line: I will not make breakpoint matching case insensitive because it will hide potential problems. Instead I will make the initial case mismatch in your case an error because it will prevent subsequent breakpoints from being hit.

@martellaj what do you think?

from vscode-node-debug.

@weinand I think that's a good solution. Like I said initially, all or nothing is definitely better IMO. 👍

from vscode-node-debug.