From 61af83b0855f9ea2b5dc5f1a1342eef7faa4953b Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Mon, 5 May 2008 18:42:13 +0000 Subject: [PATCH] * In vardbapi.cpv_all() use catpkgsplit() for validation since that's what with cp_all() uses. * Use the listdir() dirsonly=1 parameter to avoid unnecessary stat calls via os.path.isdir() (trunk r10204) svn path=/main/branches/2.1.2/; revision=10205 --- pym/portage.py | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/pym/portage.py b/pym/portage.py index 176a21dbe..0f8a393cd 100644 --- a/pym/portage.py +++ b/pym/portage.py @@ -6726,13 +6726,19 @@ class vardbapi(dbapi): continue if not self._category_re.match(x): continue - for y in listdir(basepath+x,EmptyOnError=1): + for y in listdir(basepath + x, EmptyOnError=1, dirsonly=1): if self._excluded_dirs.match(y) is not None: continue - subpath = x+"/"+y + subpath = x + "/" + y # -MERGING- should never be a cpv, nor should files. - if os.path.isdir(basepath+subpath) and (pkgsplit(y) is not None): - returnme += [subpath] + try: + if catpkgsplit(subpath) is None: + self.invalidentry(os.path.join(self.root, subpath)) + continue + except portage_exception.InvalidData: + self.invalidentry(os.path.join(self.root, subpath)) + continue + returnme.append(subpath) return returnme def cp_all(self,use_cache=1): -- 2.26.2