Skip to content
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

Force unlock message in Windows shows \ instead of / in the path #1243

Closed
billdenney opened this issue Apr 22, 2020 · 2 comments · Fixed by #1244
Closed

Force unlock message in Windows shows \ instead of / in the path #1243

billdenney opened this issue Apr 22, 2020 · 2 comments · Fixed by #1244

Comments

@billdenney
Copy link
Contributor

In this line:

.self$path,

The error message is shown with backslashes () instead of forward slashes (/) in Windows, so it cannot be simply pasted.

The message looks like:

drake::drake_cache("C:\some_path\.drake")$unlock()

When it should look like:

drake::drake_cache("C:/some_path/.drake")$unlock()

Using normalizePath() should make it work as desired. (PR coming shortly.)

@wlandau
Copy link
Member

wlandau commented Apr 22, 2020

Both should work even on Windows, but the second version is clearer.

@billdenney
Copy link
Contributor Author

billdenney commented Apr 22, 2020

The first message gave an error like:

"\." is not a recognized escape sequence.

Thanks for merging!

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 a pull request may close this issue.

2 participants