search

xforty / drush-deploy

Drupal deploy tool based on Capistrano and Drush
https://github.com/xforty/drush-deploy/wiki
Other
35 stars 4 forks source link

Set u+w on the sites/default directory for cleanup. #14

Open pearcec opened 11 years ago

pearcec commented 11 years ago

10 - issuecomment-11558213

Drupal sets the sites/default to r-x for the user. This has the adverse side effect of making the cleanup job fail. As David noted if you blanket set the entire release directory with u+w you can remove the current release. This works, but I think we need to be a bit more surgical. If there is something else that is set wrong, I would want to see it and have drush-deploy fail so we can inspect what happened.

pearcec commented 11 years ago

The cleanup code is a bit more involved. There isn't a specific variable that is referenced. It puts together a list of directories to whack. This is done using local variables. So we need to do the same. Ugly but it will work.

pearcec commented 11 years ago

I take back my original comment about being surgical. See my commit comment for a further explanation.