commit 2c066769bf9153a855580679e56e471234e30dbe
author Yu-Ju Hong <yjhong@chromium.org> Mon Oct 28 14:05:08 2013 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Oct 30 01:05:57 2013 +0000
tree 8b3bb1e42ab758f24998a107d5b28458c6646a9c
parent b71765197d75236923131ea79de5fb833615d8f1

Allow root to create cache directories as a non-root user

This change ensures that cache directories are owned by a non-root
user even if the process is running as root.

BUG=chromium:312420
TEST=unittests + tryjobs

Change-Id: I1336c547c7a8580e26fab7c24cf2382e9115d5a3
Reviewed-on: https://chromium-review.googlesource.com/174893
Reviewed-by: David James <davidjames@chromium.org>
Commit-Queue: Yu-Ju Hong <yjhong@chromium.org>
Tested-by: Yu-Ju Hong <yjhong@chromium.org>

scripts/cbuildbot.py

diff --git a/scripts/cbuildbot.py b/scripts/cbuildbot.py
index 6bb322a..92ec3c2 100644
--- a/scripts/cbuildbot.py
+++ b/scripts/cbuildbot.py
@@ -1308,7 +1308,7 @@
     options.cache_dir = os.path.abspath(options.cache_dir)
     parser.ConfigureCacheDir(options.cache_dir)
 
-  osutils.SafeMakedirs(options.cache_dir)
+  osutils.SafeMakedirsNonRoot(options.cache_dir)
 
   if options.local_patches:
     options.local_patches = _CheckLocalPatches(