`dvc add` print not quote-wrapped file-paths if they contain spaces

[byaka]

dvc 0.82.8
installed via yum
centos 7

Some of our, khm, windows-users put files in project with spaces in names.
In this case dvc add, when prints hint-command git add blah-blah, not wrap path with quotes so copy-paste this hint-command not working ofc.

[efiop]

Off topic: Makes me think if we should re-consider auto git adding files. 🤔

[byaka]

Off topic: Makes me think if we should re-consider auto git adding files.

think it good idea if it will not break old setups after upgrade :)

[efiop]

@byaka Good point, the only thing it might break is if some users commit subsets of files, as dvc will have all of those git-added automatically. Would that break your setup?

We have heard some requests about git-adding automatically previously: #1576, #1558 .

CC @iterative/engineering what do you think, guys?

[casperdcl]

Still think --git flag would be the first thing to add (which would in this case git add). Making it automatic/default would be a discussion for later.

[jorgeorpinel]

CC @iterative/engineering what do you think, guys?

Maybe let's discuss that in #1576 (comment)? 🙂

---

As for this issue, I think we should definitely wrap file names that have spaces in them (or maybe just all file names) with quotes in the git add hint after dvc add. It's not even just a Windows problem, it can happen anywhere. It's basically a bug.

POSIX terminal repro script:

$ echo foo > data\ with\ spaces
$ ls -l
total 16
-rw-r--r--  1 jop  staff    4 Feb 24 21:05 data with spaces
$ dvc add data\ with\ spaces
$ 100% Add|█████████████████████████████████████████████████|1/1 [00:01,  1.03s/file]

To track the changes with git, run:

	git add data with spaces.dvc .gitignore
$ git add data with spaces.dvc .gitignore
fatal: pathspec 'data' did not match any files

p.s. if I had an unstaged file/dir named data this could lead to errors.

[efiop]

Still think --git flag would be the first thing to add (which would in this case git add). Making it automatic/default would be a discussion for later.

That flag doesn't make much sense, as we already know that we are running in git, hence the note about git add after dvc adding anything.

[efiop]

@jorgeorpinel we do have a PR that handles that for posix through shlex, but for windows we need mslex which doesn't even have a conda package, plus it is not really compatible with gitbash, and we are currently not distinguishing between windows cmd/powershell and gitbash in our code. So if we go through so much trouble for mslex, it might be more reasonable to discuss auto git-adding instead.