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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cweagans’s picture

Status: Active » Needs review
FileSize
996 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

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

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

anarcat’s picture

Status: Needs review » Needs work

Could we update api.php with that too please?

ergonlogic’s picture

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...