summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorZac Medico <zmedico@gentoo.org>2010-12-15 18:55:53 -0800
committerZac Medico <zmedico@gentoo.org>2010-12-15 18:55:53 -0800
commit9a193d42032005396800eb30e550691513529c79 (patch)
treea038e6b1613f1cd681a4419611713c02b4911ec2
parent7be2f4b8db7c4194df6541f27118a57f1e006f0c (diff)
downloadportage-9a193d42032005396800eb30e550691513529c79.tar.gz
portage-9a193d42032005396800eb30e550691513529c79.tar.bz2
portage-9a193d42032005396800eb30e550691513529c79.zip
slot_conflict_handler: fix for bug #348389
This fixes a broken assumption about USE dep matching behavior, since commit 766988aa56bb1bbd2796f96946e62ed78cdf259 introduced stricter IUSE matching by using Atom.unevaluated_atom for IUSE checks (thus changing behavior in cases where conditional USE dependencies disappear upon evaluation).
-rw-r--r--pym/_emerge/resolver/slot_collision.py25
1 files changed, 17 insertions, 8 deletions
diff --git a/pym/_emerge/resolver/slot_collision.py b/pym/_emerge/resolver/slot_collision.py
index c51b9f734..0f4c6a53f 100644
--- a/pym/_emerge/resolver/slot_collision.py
+++ b/pym/_emerge/resolver/slot_collision.py
@@ -231,14 +231,23 @@ class slot_conflict_handler(object):
collision_reasons[("version", sub_type)] = atoms
elif not atom_set.findAtomForPackage(other_pkg, \
modified_use=_pkg_use_enabled(other_pkg)):
- #Use conditionals not met.
- violated_atom = atom.violated_conditionals(_pkg_use_enabled(other_pkg), \
- other_pkg.iuse.is_valid_flag)
- for flag in violated_atom.use.enabled.union(violated_atom.use.disabled):
- atoms = collision_reasons.get(("use", flag), set())
- atoms.add((ppkg, atom, other_pkg))
- collision_reasons[("use", flag)] = atoms
- num_all_specific_atoms += 1
+ missing_iuse = other_pkg.iuse.get_missing_iuse(
+ atom.unevaluated_atom.use.required)
+ if missing_iuse:
+ for flag in missing_iuse:
+ atoms = collision_reasons.get(("use", flag), set())
+ atoms.add((ppkg, atom, other_pkg))
+ collision_reasons[("use", flag)] = atoms
+ num_all_specific_atoms += 1
+ else:
+ #Use conditionals not met.
+ violated_atom = atom.violated_conditionals(_pkg_use_enabled(other_pkg), \
+ other_pkg.iuse.is_valid_flag)
+ for flag in violated_atom.use.enabled.union(violated_atom.use.disabled):
+ atoms = collision_reasons.get(("use", flag), set())
+ atoms.add((ppkg, atom, other_pkg))
+ collision_reasons[("use", flag)] = atoms
+ num_all_specific_atoms += 1
msg.append(" pulled in by\n")