From: Zac Medico <zmedico@gentoo.org>
Date: Mon, 16 Jun 2008 08:10:49 +0000 (-0000)
Subject: Add some write access checks in post_merge() in order to avoid triggering
X-Git-Tag: v2.2_rc1~35
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=46d84b1283e73cb9280aae735a155a7c5a6da3d3;p=portage.git

Add some write access checks in post_merge() in order to avoid triggering
permission related exceptions.

svn path=/main/trunk/; revision=10658
---

diff --git a/pym/_emerge/__init__.py b/pym/_emerge/__init__.py
index fef25c3f0..208635a02 100644
--- a/pym/_emerge/__init__.py
+++ b/pym/_emerge/__init__.py
@@ -6766,7 +6766,8 @@ def chk_updated_info_files(root, infodirs, prev_mtimes, retval):
 				if inforoot=='':
 					continue
 
-				if not os.path.isdir(inforoot):
+				if not os.path.isdir(inforoot) or \
+					not os.access(inforoot, os.W_OK):
 					continue
 				errmsg = ""
 				file_list = os.listdir(inforoot)
@@ -6929,13 +6930,16 @@ def post_emerge(trees, mtimedb, retval):
 
 	vdb_path = os.path.join(target_root, portage.VDB_PATH)
 	portage.util.ensure_dirs(vdb_path)
-	vdb_lock = portage.locks.lockdir(vdb_path)
+	vdb_lock = None
+	if os.access(vdb_path, os.W_OK):
+		vdb_lock = portage.locks.lockdir(vdb_path)
 	try:
 		if "noinfo" not in settings.features:
 			chk_updated_info_files(target_root, infodirs, info_mtimes, retval)
 		mtimedb.commit()
 	finally:
-		portage.locks.unlockdir(vdb_lock)
+		if vdb_lock:
+			portage.locks.unlockdir(vdb_lock)
 
 	chk_updated_cfg_files(target_root, config_protect)