MakeDirTree: use umask 774 rathern than 744, so group members doing

development have access, also.  More importantly, zero the umask
before actually creating the dir (and restore it afterwards), so the
permissions are exactly as specified,

Also wrapped a long line.

1 files changed, 8 insertions, 3 deletions

diff --git a/modules/mm_utils.py b/modules/mm_utils.py
index 5869cc91d..12e46c99a 100644
--- a/modules/mm_utils.py
+++ b/modules/mm_utils.py
@@ -67,16 +67,21 @@ def GetPathPieces(path):
 	pass
     return l
 
-def MakeDirTree(path, perms=0744, verbose=0):
+def MakeDirTree(path, perms=0774, verbose=0):
     made_part = '/'
     path_parts = GetPathPieces(path)
     for item in path_parts:
 	made_part = os.path.join(made_part, item)
 	if os.path.exists(made_part):
 	    if not os.path.isdir(made_part):
-		raise "RuntimeError", "Couldn't make dir tree for %s.  (%s already exists)" % (path, made_part)
+		raise "RuntimeError", ("Couldn't make dir tree for %s.  (%s"
+				       " already exists)" % (path, made_part))
 	else:
-	    os.mkdir(made_part, perms)
+	    ou = os.umask(0)
+	    try:
+		os.mkdir(made_part, perms)
+	    finally:
+		os.umask(ou)
 	    if verbose:
 		print 'made directory: ', madepart