[ 
https://issues.apache.org/jira/browse/TWILL-208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15836976#comment-15836976
 ] 

ASF GitHub Bot commented on TWILL-208:
--------------------------------------

Github user anew commented on a diff in the pull request:

    https://github.com/apache/twill/pull/26#discussion_r97690501
  
    --- Diff: 
twill-common/src/main/java/org/apache/twill/filesystem/LocalLocation.java ---
    @@ -251,6 +251,34 @@ public boolean mkdirs() throws IOException {
         return file.mkdirs();
       }
     
    +  @Override
    +  public boolean mkdirs(String permission) throws IOException {
    --- End diff --
    
    hmm that does not seem to apply the permissions correctly, it applies the 
umask to the permissions. What we want is the exact permissions as passed in.


> Location should have a way to set permissions when creating directories
> -----------------------------------------------------------------------
>
>                 Key: TWILL-208
>                 URL: https://issues.apache.org/jira/browse/TWILL-208
>             Project: Apache Twill
>          Issue Type: Bug
>            Reporter: Andreas Neumann
>            Assignee: Andreas Neumann
>             Fix For: 0.10.0
>
>
> That is, we need to introduce a method
> {code}
> boolean mkdirs(String permissions);
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to