sas added a comment.

In https://reviews.llvm.org/D31086#705159, @labath wrote:

> In https://reviews.llvm.org/D31086#704518, @sas wrote:
>
> > The second behavioral change seems good but the first thing you described 
> > is a bit odd. Creating folders with 770 does not seem like a common 
> > behavior, and 700 or 755 is usually more standard.
>
>
> Don't forget that the real permissions will also reflect the user's 
> `umask(2)`. The completely standard way would be to use 777 or 666 for the 
> permissions (except perhaps for some temporary/sensitive files), and then let 
> the user choose what are the real permissions he wants by setting umask in 
> his bashrc. I guess we are being a bit paranoid here and disallowing the 
> "others" permissions just in case...


Didn't know that. If these respect the user's `umask(2)` then this is good.


Repository:
  rL LLVM

https://reviews.llvm.org/D31086



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to