-
Notifications
You must be signed in to change notification settings - Fork 511
Scripts fail to launch in debugger (or open in ISE) if working directory path contains spaces #94
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
Comments
Perfect, thanks for filing this! Should be an easy fix. |
We should switch to using PathIntrinsics to set the working dir and when we escape the script path, we also need to escape spaces (not just "program": "d:\\powershell` code\\Get-OutlookPublicFolders.ps1" |
Ah yes! The back tick is the trick. |
BTW this issue also impacts the "Open file in ISE" command. :-( |
Ouch! That's OK, should be relatively easy to make a backtick escaper in JavaScript. |
Hmm, I was planning on fixing it in PSES. Do you think it would be better to do in the extension JavaScript? |
The debug program path issue will need to be fixed in PSES but the 'Open in ISE' command is done purely in TypeScript (for now) |
…paces Fixes #94 - specifically the Open in ISE issue.
Awesome, thank you for the quick turn around on this! |
Keith is the man! We're planning to ship an update sometime next week with this fix and a few others. |
Example: D:\powershell code\Get-Something.ps1 will fail to launch with the following error:
d:\powershell : The term 'd:\powershell' is not recognized as the name of a cmdlet, function, script file, or operable
program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
launch.json contains the default settings (as of 0.4.0 release of Powershell extension):
I have also tried placing the full path name in the cwd parameter, using escaped double quotes, single quotes, etc.
The text was updated successfully, but these errors were encountered: