The quickstart fix permissions command

drush qf

doesn't switch the group bit to writeable on files (does for dirs), but it has to be since the web server running as group www-data won't be able to delete or overwrite existing files without it. I've attached a simple patch.

CommentFileSizeAuthor
#3 quickstart.perms_.again_.patch1.44 KBchrisschaub
#2 quickstart.fix_.patch1.44 KBchrisschaub
#1 quickstart.drush_.inc_.fileperms.patch1.21 KBchrisschaub
quickstart.drush_.inc_.fileperms.patch1.31 KBchrisschaub
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chrisschaub’s picture

chrisschaub CreditAttribution: chrisschaub commented
FileSize
quickstart.drush_.inc_.fileperms.patch1.21 KB

Sorry, that last patch was not quite right. Here's an updated version.

chrisschaub’s picture

chrisschaub CreditAttribution: chrisschaub commented
FileSize
quickstart.fix_.patch1.44 KB

Man, working too fast! Ignore those last two patches.

chrisschaub’s picture

chrisschaub CreditAttribution: chrisschaub commented
FileSize
quickstart.perms_.again_.patch1.44 KB

Sigh, found a typo. Here it is again.

mike stewart’s picture

mike stewart CreditAttribution: mike stewart commented
Status: Needs review » Closed (works as designed)

@cschaub -- hmm, at first glance this patch doesn't do anything that these existing lines aren't trying to do... 

sudo find $codepath/sites -type d -name files -exec chmod -R a=,ug=rwX '{}' \;
sudo find $codepath/sites -type d -name files_private -exec chmod -R a=,ug=rwX '{}' \;

In the first line, the statement searches for any child folder under sites named files (in the scond it searches for files_private). For each instance, it executes a chmod recursively on all files and folders setting user & group permissions to RW (and folders get X -- which allows listing of files).

Maybe you can provide more details and I can help debug whats going on? Please feel free to hit me up in IRC #drupalpro

mike stewart’s picture

mike stewart CreditAttribution: mike stewart commented

Note, I REALLY appreciate your effort and the patch!!!