diff options
author | Zac Medico <zmedico@gentoo.org> | 2011-01-31 02:01:48 -0800 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2011-01-31 02:01:48 -0800 |
commit | 2a2b9e6485950757c59a1d5aa026d7f875311def (patch) | |
tree | 05eb53c80c9885802ac1b9eceb7f8cb6c7d55464 /pym/portage/package/ebuild/_config/KeywordsManager.py | |
parent | aa2c6ac3fa1890957f6c6cb85795cfc05023c56d (diff) | |
download | portage-2a2b9e6485950757c59a1d5aa026d7f875311def.tar.gz portage-2a2b9e6485950757c59a1d5aa026d7f875311def.tar.bz2 portage-2a2b9e6485950757c59a1d5aa026d7f875311def.zip |
Display: Simplify mask code for bug #58416.
Hopefully this makes the logic more understandable, though it may
support fewer cases. If we want to handle more cases then hopefully
we can do it without making the code too complex.
Diffstat (limited to 'pym/portage/package/ebuild/_config/KeywordsManager.py')
-rw-r--r-- | pym/portage/package/ebuild/_config/KeywordsManager.py | 23 |
1 files changed, 4 insertions, 19 deletions
diff --git a/pym/portage/package/ebuild/_config/KeywordsManager.py b/pym/portage/package/ebuild/_config/KeywordsManager.py index 0b593f6e5..51270f2d3 100644 --- a/pym/portage/package/ebuild/_config/KeywordsManager.py +++ b/pym/portage/package/ebuild/_config/KeywordsManager.py @@ -142,14 +142,13 @@ class KeywordsManager(object): slot, keywords, repo, - global_accept_keywords, - backuped_accept_keywords): + global_accept_keywords): """ Take a package and return a list of any KEYWORDS that the user may need to accept for the given package. If the KEYWORDS are empty, the returned list will contain ** alone (in order to distinguish from the case of "none missing"). This DOES NOT apply any user config - keywording acceptance. + package.accept_keywords acceptance. @param cpv: The package name (for package.keywords support) @type cpv: String @@ -159,29 +158,15 @@ class KeywordsManager(object): @type keywords: String @param global_accept_keywords: The current value of ACCEPT_KEYWORDS @type global_accept_keywords: String - @param backuped_accept_keywords: ACCEPT_KEYWORDS from the backup env - @type backuped_accept_keywords: String @rtype: List @return: lists of KEYWORDS that have not been accepted and the keywords it looked for. """ mygroups = self.getKeywords(cpv, slot, keywords, repo) - # Repoman may modify this attribute as necessary. pgroups = global_accept_keywords.split() - - # Hack: Need to check the env directly here as otherwise stacking - # doesn't work properly as negative values are lost in the config - # object (bug #139600) - if backuped_accept_keywords: - pgroups = self._getEgroups(backuped_accept_keywords.split(), - pgroups) - else: - pgroups = set(pgroups) - - missing = self._getMissingKeywords(cpv, pgroups, mygroups) - - return missing, list(pgroups) + pgroups = set(pgroups) + return self._getMissingKeywords(cpv, pgroups, mygroups) @staticmethod |