Spaces in command arguments break things about braid HOT 5 CLOSED

I'm working on getting the Braid::Operations::Git class statically typed, which is going to require extensive rework and review of that class and all calls to it anyway, and I'm hoping it will be efficient to fix the problems with spaces at the same time.

from braid.

Without testing, it looks like the problem is that you have a space in your username and Braid joins command arguments with spaces and passes the result to the shell without any quoting. ☹️ Hopefully we can fix this by moving everything to the forms of Open3.capture3, etc. that take a list of arguments.

However, it's generally advisable (even on unix-like OSes) to avoid spaces in file paths for software development work because lots of less-used development tools have such bugs, although as they get more used, the bugs tend to get fixed (as is happening here with Braid). Indeed, in some programming environments such as makefiles, trying to defend against all characters treated specially by the shell entails so much clutter in the code that it's considered better to require users not to use those characters; maybe we should be moving off of those environments, but realistically we will be stuck with them for many years. In fact, I think the removal of spaces from a lot of default folder names in Windows Vista (e.g., Documents and Settings -> Users) was at least partly for this reason, though I haven't confirmed this.

from braid.

Your are right, the problem was the space in my username.
After renaming my username, it went smooth as expected.

from braid.

I'm glad to have confirmation of what the problem was and that you worked around it. This is still a bug in Braid.

from braid.

I'll be glad to see this issue fixed. In my scenario, it was just one folder, that I had to rename and add an _ instead of space, but on some cases, there are several directories one inside the other where there are spaces in few of them.
On existing projects, renaming a folder (In my case, the solution has over 110 projects), might break things.

from braid.