From efec767fb9c4ba15ae76fb9872c9f254be91d562 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Mon, 26 Nov 2007 01:16:16 +0000 Subject: Bug #124041 - Make emerge show an informative warning message when one or more eclasses override eclasses from PORTDIR. The warning can be permanently disabled by setting PORTAGE_ECLASS_WARNING_ENABLE="0" in /etc/make.conf. (trunk r8668) svn path=/main/branches/2.1.2/; revision=8669 --- pym/eclass_cache.py | 10 +++++++++- pym/portage.py | 9 ++++++++- 2 files changed, 17 insertions(+), 2 deletions(-) (limited to 'pym') diff --git a/pym/eclass_cache.py b/pym/eclass_cache.py index ede081d9c..b8d849368 100644 --- a/pym/eclass_cache.py +++ b/pym/eclass_cache.py @@ -23,6 +23,7 @@ class cache: self.porttrees = [self.porttree_root]+overlays self.porttrees = tuple(map(normalize_path, self.porttrees)) self._master_eclass_root = os.path.join(self.porttrees[0],"eclass") + self._master_eclasses_overridden = {} self.update_eclasses() def close_caches(self): @@ -41,6 +42,7 @@ class cache: def update_eclasses(self): self.eclasses = {} self._eclass_locations = {} + master_eclasses = {} eclass_len = len(".eclass") ignored_listdir_errnos = (errno.ENOENT, errno.ENOTDIR) for x in [normalize_path(os.path.join(y,"eclass")) for y in self.porttrees]: @@ -63,7 +65,13 @@ class cache: ys=y[:-eclass_len] self.eclasses[ys] = (x, long(mtime)) self._eclass_locations[ys] = x - + if x == self._master_eclass_root: + master_eclasses[ys] = mtime + else: + master_mtime = master_eclasses.get(ys) + if master_mtime and master_mtime != mtime: + self._master_eclasses_overridden[ys] = x + def is_eclass_data_valid(self, ec_dict): if not isinstance(ec_dict, dict): return False diff --git a/pym/portage.py b/pym/portage.py index f82074f74..bbaee8027 100644 --- a/pym/portage.py +++ b/pym/portage.py @@ -961,7 +961,11 @@ class config: Generally if you need data like USE flags, FEATURES, environment variables, virtuals ...etc you look in here. """ - + + # Filter selected variables in the config.environ() method so that + # they don't needlessly propagate down into the ebuild environment. + _environ_filter = frozenset(["PORTAGE_ECLASS_WARNING_ENABLE"]) + def __init__(self, clone=None, mycpv=None, config_profile_path=None, config_incrementals=None, config_root=None, target_root=None, local_config=True): @@ -2429,7 +2433,10 @@ class config: def environ(self): "return our locally-maintained environment" mydict={} + environ_filter = self._environ_filter for x in self: + if x in environ_filter: + continue myvalue = self[x] if not isinstance(myvalue, basestring): writemsg("!!! Non-string value in config: %s=%s\n" % \ -- cgit v1.2.3-1-g7c22