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
fix scripts_installer handling user names with spaces on windows #15437
fix scripts_installer handling user names with spaces on windows #15437
Conversation
@wpferguson : Can you rebase on current master? The GitHub-CI issue should be fixed there. TIA. |
is happening when the scripts_installer doesn't work. Added a lua preference to turn the debugging messages on and off.
installer is visible.
windows by... adding the string sanitize functions from the lua scripts string library to handle spaces in argument strings. wrapping the os.execute() command with an os_execute() function that surrounds the command in quotes before sending to os.execute() if we are on windows. This is necessary if there is more than one quoted string in the command.
@TurboGit how do I do that? I did
|
You need to force push with option |
Thanks. That's what I thought but I didn't want to break anything. |
3ec4afe
to
eb64778
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Hi, FYI it's been reported the same issue on Mac when the username has a space in it. A workaround might be to replace on line 115 of luarc: |
Please open an issue at https://github.com/darktable-org/lua-scripts/issues |
Install the lua scripts on windows when the username has a space in it.
Just for reference the proper way to quote argument strings with spaces for windows is
""\Program Files\darktable\bin\darktable" "\Users\A User\Pictures\img1234.cr2""