Skip to content

Fix for issue #123, wildcard chars in script path prevent breakpoints… #126

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

Merged

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.  :-)
/// </summary>
/// <param name="path">The path to process.</param>
/// <returns>The path with [ and ] escaped.</returns>
public static string EscapeWilcardsInPath(string path)
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants