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 recommended command to make directories writable #13630

Closed
wants to merge 1 commit into from
Closed

Change recommended command to make directories writable #13630

wants to merge 1 commit into from

Conversation

taylankasap
Copy link

Fixes #13628

Should the same command for Windows be also updated? See line 183

@Findus23 Findus23 added Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. Needs Review PRs that need a code review labels Oct 20, 2018
@Findus23
Copy link
Member

Hi,

I think this would break the feature in case a file needs to be writable (e.g. config/config.ini.php) as I think only directories are changed.

@taylankasap
Copy link
Author

I see. What about something like this?

chmod -R a+rw /path ; find /path -type d -exec chmod a+x {} \;

chmod -R a+rw /path to give all files and directories read and write permission for all users.
find /path -type d -exec chmod a+x {} \; to give all directories execute permission for all users (since folders need execute to traverse).

This way the command won't alter any execute permission for files. Does Matomo require any files with execute permission?

@tsteur
Copy link
Member

tsteur commented Apr 12, 2019

Files still need to be writable too or not (so they can be deleted during update etc)?

@tsteur
Copy link
Member

tsteur commented May 3, 2019

@taylankasap are you still working on this PR please let us know. This will help us to know whether we can otherwise close the issue and whether we should work on a solution or not.

@taylankasap
Copy link
Author

@tsteur I'm not working on this PR

@tsteur
Copy link
Member

tsteur commented May 5, 2019

cheers for letting us know 👍

@tsteur tsteur closed this May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help wanted Beginner friendly issues or issues where we'd highly appreciate community's help and involvement. Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't tell people to run chmod -R 755 on their install if update fails
4 participants