From b3222041e0eeca52708b36da8462b09160f48789 Mon Sep 17 00:00:00 2001
From: Zac Medico <zmedico@gentoo.org>
Date: Thu, 28 Jan 2010 05:37:57 +0000
Subject: [PATCH] Add an Atom.evaluate_conditionals() method and use where
 appropriate.

svn path=/main/trunk/; revision=15212
---
 bin/portageq            |  6 +-----
 pym/portage/__init__.py |  6 +-----
 pym/portage/dep.py      | 16 ++++++++++++++++
 3 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/bin/portageq b/bin/portageq
index 342e19fd2..e02a9ddbb 100755
--- a/bin/portageq
+++ b/bin/portageq
@@ -49,11 +49,7 @@ from portage.util import writemsg, writemsg_stdout
 def eval_atom_use(atom):
 	if atom.use.conditional and 'USE' in os.environ:
 		use = os.environ['USE'].split()
-		evaluated_atom = portage.dep.remove_slot(atom)
-		if atom.slot:
-			evaluated_atom += ":%s" % atom.slot
-		evaluated_atom += str(atom.use.evaluate_conditionals(use))
-		atom = portage.dep.Atom(evaluated_atom)
+		atom = atom.evaluate_conditionals(use)
 	return atom
 
 #-----------------------------------------------------------------------------
diff --git a/pym/portage/__init__.py b/pym/portage/__init__.py
index 23e69cabf..5f62fd7ae 100644
--- a/pym/portage/__init__.py
+++ b/pym/portage/__init__.py
@@ -7815,11 +7815,7 @@ def _expand_new_virtuals(mysplit, edebug, mydbapi, mysettings, myroot="/",
 		if not repoman and \
 			myuse is not None and isinstance(x, portage.dep.Atom) and x.use:
 			if x.use.conditional:
-				evaluated_atom = portage.dep.remove_slot(x)
-				if x.slot:
-					evaluated_atom += ":%s" % x.slot
-				evaluated_atom += str(x.use.evaluate_conditionals(myuse))
-				x = portage.dep.Atom(evaluated_atom)
+				x = x.evaluate_conditionals(myuse)
 
 		mykey = x.cp
 		if not mykey.startswith("virtual/"):
diff --git a/pym/portage/dep.py b/pym/portage/dep.py
index 70d95e484..409bc93de 100644
--- a/pym/portage/dep.py
+++ b/pym/portage/dep.py
@@ -612,6 +612,22 @@ class Atom(_atom_base):
 
 		return False
 
+	def evaluate_conditionals(self, use):
+		"""
+		Create an atom instance with any USE conditionals evaluated.
+		@param use: The set of enabled USE flags
+		@type other: set
+		@rtype: Atom
+		@return: an atom instance with any USE conditionals evaluated
+		"""
+		if not self.use.conditional:
+			return self
+		atom = remove_slot(self)
+		if self.slot:
+			atom += ":%s" % self.slot
+		atom += str(self.use.evaluate_conditionals(use))
+		return Atom(atom)
+
 	def __copy__(self):
 		"""Immutable, so returns self."""
 		return self
-- 
2.26.2