Make all the python code respect USE conditionals in RESTRICT. It's already handled on the bash side since RESTRICT is assigned the value of PORTAGE_RESTRICT. (branches/2.1.2 r6763:6765)

svn path=/main/branches/2.1.2.9/; revision=7452

2 files changed, 30 insertions, 17 deletions

diff --git a/bin/emerge b/bin/emerge
index e1209640e..31868d2fb 100755
--- a/bin/emerge
+++ b/bin/emerge
@@ -2418,6 +2418,8 @@ class depgraph:
 				depth >= mylist[i+1][1]:
 					del mylist[i]
 
+		from portage import flatten
+		from portage_dep import use_reduce, paren_reduce
 		display_overlays=False
 		# files to fetch list - avoids counting a same file twice
 		# in size display (verbose mode)
@@ -2456,6 +2458,7 @@ class depgraph:
 				blockers.append(addl)
 			else:
 				mydbapi = self.trees[myroot][self.pkg_tree_map[pkg_type]].dbapi
+				pkg_status = x[3]
 				binary_package = True
 				if "ebuild" == pkg_type:
 					if "merge" == x[3] or \
@@ -2476,9 +2479,19 @@ class depgraph:
 					self.useFlags[myroot][pkg_key] = mydbapi.aux_get(
 						pkg_key, ["USE"])[0].split()
 
+				try:
+					restrict = flatten(use_reduce(paren_reduce(
+						mydbapi.aux_get(pkg_key, ["RESTRICT"])[0]),
+						uselist=self.useFlags[myroot][pkg_key]))
+				except portage_exception.InvalidDependString, e:
+					if pkg_status != "nomerge":
+						restrict = mydbapi.aux_get(pkg_key, ["RESTRICT"])[0]
+						show_invalid_depstring_notice(x, restrict, str(e))
+						del e
+						sys.exit(1)
+					restrict = []
 				if "ebuild" == pkg_type and x[3] != "nomerge" and \
-					"fetch" in portdb.aux_get(
-					x[2], ["RESTRICT"])[0].split():
+					"fetch" in restrict:
 					fetch = red("F")
 					if ordered:
 						counters.restrict_fetch += 1
diff --git a/pym/portage.py b/pym/portage.py
index e4df2ddd6..051b4fa58 100644
--- a/pym/portage.py
+++ b/pym/portage.py
@@ -2230,11 +2230,9 @@ def spawn(mystring, mysettings, debug=0, free=0, droppriv=0, sesandbox=0, **keyw
 		keywords["fd_pipes"] = fd_pipes
 
 	features = mysettings.features
-	# XXX: Negative RESTRICT word
-	droppriv=(droppriv and ("userpriv" in features) and not \
-		(("nouserpriv" in mysettings["RESTRICT"].split()) or \
-		 ("userpriv" in mysettings["RESTRICT"].split())))
-
+	restrict = mysettings.get("PORTAGE_RESTRICT","").split()
+	droppriv=(droppriv and "userpriv" in features and not \
+		("nouserpriv" in restrict or "userpriv" in restrict))
 	if droppriv and not uid and portage_gid and portage_uid:
 		keywords.update({"uid":portage_uid,"gid":portage_gid,"groups":userpriv_groups,"umask":002})
 
@@ -2287,9 +2285,10 @@ def fetch(myuris, mysettings, listonly=0, fetchonly=0, locks_in_subdir=".locks",
 	"fetch files.  Will use digest file if available."
 
 	features = mysettings.features
+	restrict = mysettings.get("PORTAGE_RESTRICT","").split()
 	# 'nomirror' is bad/negative logic. You Restrict mirroring, not no-mirroring.
-	if ("mirror" in mysettings["RESTRICT"].split()) or \
-	   ("nomirror" in mysettings["RESTRICT"].split()):
+	if "mirror" in restrict or \
+	   "nomirror" in restrict:
 		if ("mirror" in features) and ("lmirror" not in features):
 			# lmirror should allow you to bypass mirror restrictions.
 			# XXX: This is not a good thing, and is temporary at best.
@@ -2324,8 +2323,8 @@ def fetch(myuris, mysettings, listonly=0, fetchonly=0, locks_in_subdir=".locks",
 	if custommirrors.has_key("local"):
 		mymirrors += custommirrors["local"]
 
-	if ("nomirror" in mysettings["RESTRICT"].split()) or \
-	   ("mirror"   in mysettings["RESTRICT"].split()):
+	if "nomirror" in restrict or \
+	   "mirror" in restrict:
 		# We don't add any mirrors.
 		pass
 	else:
@@ -2346,7 +2345,7 @@ def fetch(myuris, mysettings, listonly=0, fetchonly=0, locks_in_subdir=".locks",
 			fsmirrors += [mymirrors[x]]
 			del mymirrors[x]
 
-	restrict_fetch = "fetch" in mysettings["RESTRICT"].split()
+	restrict_fetch = "fetch" in restrict
 	custom_local_mirrors = custommirrors.get("local", [])
 	if restrict_fetch:
 		# With fetch restriction, a normal uri may only be fetched from
@@ -2393,7 +2392,7 @@ def fetch(myuris, mysettings, listonly=0, fetchonly=0, locks_in_subdir=".locks",
 			if restrict_fetch:
 				# Only fetch from specific mirrors is allowed.
 				continue
-			if "primaryuri" in mysettings["RESTRICT"].split():
+			if "primaryuri" in restrict:
 				# Use the source site first.
 				if primaryuri_indexes.has_key(myfile):
 					primaryuri_indexes[myfile] += 1
@@ -3711,13 +3710,14 @@ def doebuild(myebuild, mydo, myroot, mysettings, debug=0, listonly=0,
 
 		#initial dep checks complete; time to process main commands
 
+		restrict = mysettings["PORTAGE_RESTRICT"].split()
 		nosandbox = (("userpriv" in features) and \
 			("usersandbox" not in features) and \
-			("userpriv" not in mysettings["RESTRICT"]) and \
-			("nouserpriv" not in mysettings["RESTRICT"]))
+			"userpriv" not in restrict and \
+			"nouserpriv" not in restrict)
 		if nosandbox and ("userpriv" not in features or \
-			"userpriv" in mysettings["RESTRICT"] or \
-			"nouserpriv" in mysettings["RESTRICT"]):
+			"userpriv" in restrict or \
+			"nouserpriv" in restrict):
 			nosandbox = ("sandbox" not in features and \
 				"usersandbox" not in features)