Perl-Critic / PPI

53 stars 44 forks source link

PPI::Cache - default permissions for directories 0777 (plus umask) #172

Open zhurs opened 9 years ago

zhurs commented 9 years ago

In current version, cache files creates with default (umask) permissions, but directories with 0755. We want to use one shared PPI cache for all team, so this behaviour is not comfortable for us.

With patch, permissions of created files and directories will correspond umask (so behaviour will not changed for users with default umask=022)

wchristian commented 8 years ago

Can you please instead provide a patch which adds an attribute to the PPI::Cache object for the path perms? That way we can leave the default where it is and you can configure your perms as you like them.