[Lldb-commits] [PATCH] D31086: Remove FileSystem::MakeDirectory

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

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...


Repository:
  rL LLVM

https://reviews.llvm.org/D31086