yeoman / update-notifier

Update notifications for your CLI app
BSD 2-Clause "Simplified" License
1.76k stars 132 forks source link

chown arguments are wrong #168

Closed baleevskiy closed 4 years ago

baleevskiy commented 5 years ago

My users' group contains spaces and backslashes, that's why the following command won't work. image

Potential fix: add quotes to the command

sudo chown -R "$USER":"$(id -gn $USER)" ./path
sindresorhus commented 5 years ago

Usernames and user group names are not meant to have spaces. I don't think we should do anything here. If you decide to go against the convention, it's up to you to handle the pitfalls.

https://unix.stackexchange.com/questions/401752/how-to-add-to-group-when-name-has-a-space