From b3222041e0eeca52708b36da8462b09160f48789 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Thu, 28 Jan 2010 05:37:57 +0000 Subject: 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 -- cgit v1.2.3-1-g7c22