Re: [Yum-devel] [PATCH] Drop self._pkgtup2pkgs cache, leak checker to the rescue again

2009-10-13 Thread James Antill
On Tue, 2009-10-13 at 12:44 -0400, Seth Vidal wrote: > > On Tue, 13 Oct 2009, James Antill wrote: > > > --- > > yum/sqlitesack.py |1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/yum/sqlitesack.py b/yum/sqlitesack.py > > index ce2988f..bc4b0ab 100644 > > --- a/yu

Re: [Yum-devel] [PATCH] Remove yum-metadata-parser from the default run_with_package_names list

2009-10-13 Thread Seth Vidal
On Tue, 13 Oct 2009, James Antill wrote: self.plugins.run('pretrans') +# We may want to put this other places, eventually, but for now it's +# good as long as we get it right for history. +for repo in self.repos.listEnabled(): +if repo._xml2sqlite_

Re: [Yum-devel] [PATCH] Drop self._pkgtup2pkgs cache, leak checker to the rescue again

2009-10-13 Thread Seth Vidal
On Tue, 13 Oct 2009, James Antill wrote: --- yum/sqlitesack.py |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/yum/sqlitesack.py b/yum/sqlitesack.py index ce2988f..bc4b0ab 100644 --- a/yum/sqlitesack.py +++ b/yum/sqlitesack.py @@ -478,6 +478,7 @@ class YumSqlitePackageS

[Yum-devel] [PATCH] Remove yum-metadata-parser from the default run_with_package_names list

2009-10-13 Thread James Antill
We now add it conditionally if we are using any repos. with .xml only MD. This means it isn't logged every time in history (apart from if you have the adobe repo. enabled :). --- yum/__init__.py |7 +++ yum/config.py |2 +- yum/repos.py|1 + yum/yumRepo.py |1 + 4 fil

[Yum-devel] [PATCH] Remove yum-metadata-parser from the default run_with_package_names list

2009-10-13 Thread James Antill
We now add it conditionally if we are using any repos. with .xml only MD. This means it isn't logged every time in history (apart from if you have the adobe repo. enabled :). Also adding it is a slight layer violation ... but I can't think of a better way of doing it. --- yum/config.py |

[Yum-devel] [PATCH] Drop self._pkgtup2pkgs cache, leak checker to the rescue again

2009-10-13 Thread James Antill
--- yum/sqlitesack.py |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/yum/sqlitesack.py b/yum/sqlitesack.py index ce2988f..bc4b0ab 100644 --- a/yum/sqlitesack.py +++ b/yum/sqlitesack.py @@ -478,6 +478,7 @@ class YumSqlitePackageSack(yumRepo.YumPackageSack): self._

Re: [Yum-devel] [PATCH] fix pylint error (misc. used in line #111)

2009-10-13 Thread Seth Vidal
On Tue, 13 Oct 2009, Tim Lauridsen wrote: --- yum/history.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/yum/history.py b/yum/history.py index 97571c5..c689582 100644 --- a/yum/history.py +++ b/yum/history.py @@ -23,12 +23,13 @@ import glob from weakref import pro