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 default config directory permission to 0755 #9

Merged

Conversation

nikukyugamer
Copy link
Contributor

Creating config directory initially, config file isn't created because permission denied as below.

$ deepl-translate-cli
2022/03/12 18:12:28 open /home/USERNAME/.config/deepl-translate-cli/setting.json: permission denied

So I fixed it.

After fixed, config file was created correctly.

$ deepl-translate-cli
2022/03/12 18:25:06 Not exists such file. /home/USERNAME/.config/deepl-translate-cli/setting.json
        auto make it, please write it.
exit status 1

Note:

~/.config/deepl-translate-cli directory permission is always 0777 but ~/.config/deepl-translate-cli/setting.json permission depends on user's umask.

@Omochice Omochice merged commit 05af429 into Omochice:main Mar 15, 2022
@Omochice
Copy link
Owner

I think it's no problem to merge this PR because it works fine in my pc too.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants