Skip to content

Conversation

robbat2
Copy link

@robbat2 robbat2 commented Nov 23, 2014

Signed-off-by: Robin H. Johnson robin.johnson@bc.libraries.coop

…blems on large deploys.

Signed-off-by: Robin H. Johnson <robin.johnson@bc.libraries.coop>
Signed-off-by: Robin H. Johnson <robin.johnson@bc.libraries.coop>
Copy link
Owner

Choose a reason for hiding this comment

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

for what reason is the parameter included?

@fnerdwq
Copy link
Owner

fnerdwq commented Nov 23, 2014

Thanks for the pull request.
I'm aware of the problem with the recursive handling of directories... actually I postponed the implementation of a better solution - apperently too long ;-)

I was thinking to handle this in a different way: What do you think about initially - when installing dokuwiki - notifying an exec resource doing the chmod directly?
Initial install is actally the most important usecase for the recursive handling, I think.

I'd try to implent it the next days.

@fnerdwq
Copy link
Owner

fnerdwq commented Dec 19, 2014

Hi @robbat2,
the latest commit in master 80928ad handles the directory ownerships by an explicit chown statement. That should fix your performance issues.

@fnerdwq fnerdwq closed this Dec 19, 2014
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