diff options
author | Sebastian Luther <SebastianLuther@gmx.de> | 2010-09-13 15:25:57 +0200 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2010-09-13 07:05:59 -0700 |
commit | 2380617b66056bbdeee5c219e018acccbb9183e3 (patch) | |
tree | 4e28c91fdec68738001f97bcb3c40682ef8498f6 /pym/_emerge | |
parent | 901d18381c277784d243e33cbad58480c9f1f8d5 (diff) | |
download | portage-2380617b66056bbdeee5c219e018acccbb9183e3.tar.gz portage-2380617b66056bbdeee5c219e018acccbb9183e3.tar.bz2 portage-2380617b66056bbdeee5c219e018acccbb9183e3.zip |
Don't miss a "missing IUSE" because of [foo?] -> [] evaluations.
This patch also drops flags with EAPI 4 use dep defaults from use.required.
Diffstat (limited to 'pym/_emerge')
-rw-r--r-- | pym/_emerge/depgraph.py | 29 |
1 files changed, 14 insertions, 15 deletions
diff --git a/pym/_emerge/depgraph.py b/pym/_emerge/depgraph.py index dce1041e9..252975418 100644 --- a/pym/_emerge/depgraph.py +++ b/pym/_emerge/depgraph.py @@ -2136,12 +2136,10 @@ class depgraph(object): for pkg in missing_use: use = self._pkg_use_enabled(pkg) missing_iuse = [] - for x in pkg.iuse.get_missing_iuse(atom.use.required): - #FIXME: If a use flag occures more then it might be possible that - #one has a default one doesn't. - if x not in atom.use.missing_enabled and \ - x not in atom.use.missing_disabled: - missing_iuse.append(x) + #Use the unevaluated atom here, because some flags might have gone + #lost during evaluation. + required_flags = atom.unevaluated_atom.use.required + missing_iuse = pkg.iuse.get_missing_iuse(required_flags) mreasons = [] if missing_iuse: @@ -2264,7 +2262,9 @@ class depgraph(object): elif unmasked_iuse_reasons: masked_with_iuse = False for pkg in masked_pkg_instances: - if not pkg.iuse.get_missing_iuse(atom.use.required): + #Use atom.unevaluated here, because some flags might have gone + #lost during evaluation. + if not pkg.iuse.get_missing_iuse(atom.unevaluated_atom.use.required): # Package(s) with required IUSE are masked, # so display a normal masking message. masked_with_iuse = True @@ -2747,14 +2747,7 @@ class depgraph(object): found_available_arg = True if atom.use: - missing_iuse = [] - for x in pkg.iuse.get_missing_iuse(atom.use.required): - #FIXME: If a use flag occures more then it might be possible that - #one has a default one doesn't. - if x not in atom.use.missing_enabled and \ - x not in atom.use.missing_disabled: - missing_iuse.append(x) - if missing_iuse: + if pkg.iuse.get_missing_iuse(atom.use.required): # Don't add this to packages_with_invalid_use_config # since IUSE cannot be adjusted by the user. continue @@ -2780,6 +2773,12 @@ class depgraph(object): if not pkg.built: packages_with_invalid_use_config.append(pkg) continue + elif atom.unevaluated_atom.use: + #Make sure we don't miss a 'missing IUSE'. + if pkg.iuse.get_missing_iuse(atom.unevaluated_atom.use.required): + # Don't add this to packages_with_invalid_use_config + # since IUSE cannot be adjusted by the user. + continue #check REQUIRED_USE constraints if not pkg.built and pkg.metadata["REQUIRED_USE"] and \ |