At the moment, the only drush command that uses chmod is _drush_recursive_copy (although the unit tests change permissions to set up some tests). At the moment, the file system permissions step is just skipped on Windows, which can leave the file permissions in an incorrect state (see #990812: Add a "permissions" subcommand to fix/set all file permissions).

We should create a wrapper function, drush_set_permissions (or something similar) that used chmod on Linux and cacls on Windows. It might also be useful to have wrapper functions to read and preserve permissions.

Comments

carole’s picture

Subscribe

greg.1.anderson’s picture

Priority: Normal » Minor

php chmod seems to work fine on Windows. Any motivation to switch to something different? I suspect this can be closed.

greg.1.anderson’s picture

Status: Active » Postponed (maintainer needs more info)
greg.1.anderson’s picture

Version: » 8.x-6.x-dev
Status: Postponed (maintainer needs more info) » Closed (won't fix)
Issue tags: +Needs migration

This issue was marked closed (won't fix) because Drush has moved to Github.

If desired, you may copy this bug to our Github project and then post a link here to the new issue. Please also change the status of this issue to closed (duplicate).

Please ask support questions on Drupal Answers.