Skip to content
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 chown in dev_setup when GROUP != USER #2094

Merged
merged 1 commit into from
Apr 17, 2019

Conversation

forslund
Copy link
Collaborator

Description

This finds the default group of the user and uses that as group when doing chown in dev_setup.sh the same way as it's done in the prepare-msm.sh script

Resolves #2092

How to test

Remove / rename your skills directory and remove the .dev_opts file rerun the dev_setup.sh and check that the skills directory is recreated correctly.

Contributor license agreement signed?

CLA [ Yes ]

This finds the default group of the user and uses that as group when doing chown in dev_setup.sh the same way as it's done in the prepare-msm.sh script
@forslund forslund added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Apr 14, 2019
@forslund forslund requested review from penrods and Ruthvicp and removed request for penrods April 14, 2019 06:59
Copy link
Contributor

@Ruthvicp Ruthvicp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Ruthvicp Ruthvicp merged commit f4da3e7 into MycroftAI:dev Apr 17, 2019
@forslund forslund deleted the bugfix/dev_setup-chown branch April 22, 2019 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error in dev_setup.sh: chown at line 226
2 participants