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

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Mar 20 03:57:46 PDT 2017

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



More information about the lldb-commits mailing list