Skip to content

Conversation

@rkeithhill
Copy link
Contributor

… from working.

Once I got breakpoints to be able to set for files like foo[1].ps1 and foo][.ps1 I noticed that the script named foo][.ps1 would not execute at all!! You get an error saying something like the specified wildcard character pattern is not valid. FWIW ISE fails with this same message. So PSES has one up on ISE. :-)

…reakpoints from working. Once I got breakpoints to be able to set for files like foo[1].ps1 and foo][.ps1 I noticed that the script named foo][.ps1 would not execute at all!! You get an error saying something like the specified wildcard character pattern is not valid. FWIW ISE fails with this same message. So PSES has one up on ISE. :-)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a big deal, but slight typo here: EscapeWildcardsInPath

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also I'd say this method probably belongs in PowerShellContext rather than DebugService since it may have more general utility

@rkeithhill
Copy link
Contributor Author

Thanks for catching that typo. I'll fix that. Yeah, I wondered where to put that static and dithered between PowerShellContext and DebugService. :-)

daviwil added a commit that referenced this pull request Feb 3, 2016
…litpath Fix for issue #123, wildcard chars in script path prevent breakpoints…
@daviwil daviwil merged commit 045c028 into PowerShell:master Feb 3, 2016
@daviwil
Copy link
Contributor

daviwil commented Feb 3, 2016

Awesome, thanks! Merged.

@rkeithhill rkeithhill deleted the rkeithhill/is123-breakpoints-litpath branch February 3, 2016 04:45
TylerLeonhardt pushed a commit to TylerLeonhardt/PowerShellEditorServices that referenced this pull request Feb 26, 2019
…yzer-warnings-buildfile Tweaks to PSake build script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants