Two fixes in the commands under "script to set up a computer with many users"

rsnk96 / Ubuntu-Setup-Scripts

Scripts to help you set up your Ubuntu quickly, especially if you're in any subfield of Data Science or AI!

Apache License 2.0

128 stars 76 forks source link

Two fixes in the commands under "script to set up a computer with many users" #58

Closed Samjith888 closed 4 years ago

Samjith888 commented 4 years ago

Found error while using the following commands sudo cp /opt/.zsh/zim/ /home/$NEW_USER/.zim sudo cp ~/zshrc /home/$NEW_USER/.zshrc

Replace the above lines with following : sudo cp -r /opt/.zsh/zim/ /home/$NEW_USER/.zim sudo cp ~/.zshrc /home/$NEW_USER/.zshrc

rajat2004 commented 4 years ago

Hi @Samjith888 Thanks for the PR! Though I haven't setup a system for multiple users, the changes you've mentioned seem correct. @rsnk96 would have experience with it The PR base branch is incorrect, right now it's for a commit which is already in master, so you'll need to open a new PR It is interesting however that we can merge a commit which is already present in the target branch

rajat2004 commented 4 years ago

Closing, new PR should be made for this