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

Change file perm #796

Merged
merged 1 commit into from
Jun 8, 2021
Merged

Change file perm #796

merged 1 commit into from
Jun 8, 2021

Conversation

ersonp
Copy link
Contributor

@ersonp ersonp commented Jun 7, 2021

Did you run make format && make check?
Yes

Fixes #

Changes:

  • Changed file perm of local folder from 0750 to 0707
  • Changed file perm of transport_logs folder from 0700 to 0707

How to test this PR:

  1. Run sudo rm -rf transport_logs/; sudo rm -rf local/
  2. Run make build; make install
  3. Run sudo ./skywire-visor skywire-config
  4. Press Ctrl+C
  5. Run go mod vendor
  6. Should not get permission denied error for local and transport_logs

@jdknives
Copy link
Member

jdknives commented Jun 7, 2021

@ersonp need to get rid of dmsgpty folder creation if possible.

@jdknives jdknives merged commit 1b0c38d into skycoin:develop Jun 8, 2021
@ersonp ersonp deleted the change-perm branch April 11, 2022 15:37
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