ryanmcgrath / wrench-js

Recursive file operations in Node.js
MIT License
435 stars 71 forks source link

chmodSyncRecursive should not unset directory execute permission #93

Open erhhung opened 10 years ago

erhhung commented 10 years ago

If I want to call chmodSyncRecursive(dir, 0644), then that directory and all subdirectories become unlistable because directories must have execute permissions. chmodSyncRecursive (and maybe other similar functions) should take a third optional argument for dirmode (second arg being filemode) that is used specifically for directories (and defaults to filemode).