In the event that you're using a networked file system for storing site files, it may not be necessary to try to chmod the different subdirectories in sites/sitename/files. In my case, it's actually causing warnings on site verifications (the ACL for my file system is stored elsewhere - apache can write files to it already, and it doesn't need to be changed).

So I propose that we have some way to skip chmod'ing the directories. I have a way to do this, too. Patch is coming up.

CommentFileSizeAuthor
#2 2099057-skip-chmod-chgrp-02.patch1.4 KBdanquah
#1 2099057_01.patch996 bytescweagans

Comments

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
commented
Status: Active » Needs review
StatusFileSize
new 2099057_01.patch996 bytes

In this case, you can implement hook_provision_drupal_create_directories_alter(), and change any permission you want to FALSE. Easy :)

danquah’s picture

danquah commented
StatusFileSize
new 2099057-skip-chmod-chgrp-02.patch1.4 KB

Looks good.

After applying the patch and doing a quick implementation of hook_provision_drupal_create_directories_alter the path whose permission I set to FALSE no longer got chmod'ed.

For the sake of completness I've updated the patch to also allow chgrp to be skipped via hook_provision_drupal_chgrp_directories_alter.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
commented

Ah, yeah, that makes sense. +1 from me.

anarcat’s picture

anarcat commented
Status: Needs review » Needs work

Could we update api.php with that too please?

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented
Status: Needs work » Fixed

Fixed in f78effa. Thanks all!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit f78effa on 6.x-2.x, 7.x-3.x, 6.x-2.x-backports, dev-helmo-3.x by ergonlogic: 
    Issue #2099057 by cweagans, danquah, ergonlogic: Allow skipping chmod...