Re: [Yum-devel] Patch: Hard-link packages that have already been downloaded.

2009-09-18 Thread Ryan Lynch
Hi, James, Thanks for the feedback. I understood most of your comments, and I agree with everything I understood. I need a couple of clarifications, for the parts I didn't grasp. I also had a chance to reflect on the design, and I made some changes. I started out with a "quick and dirty" appro

Re: [Yum-devel] [PATCH] Be nice and don't traceback when people screw their rpmdb, BZ 522259

2009-09-18 Thread Seth Vidal
On Fri, 18 Sep 2009, James Antill wrote: On Fri, 2009-09-18 at 14:27 -0400, Seth Vidal wrote: On Fri, 18 Sep 2009, James Antill wrote: At first glance this looks great but a number of places where we do the same thing we'd want more or less the same function but a different exception raise

Re: [Yum-devel] [PATCH] Be nice and don't traceback when people screw their rpmdb, BZ 522259

2009-09-18 Thread James Antill
On Fri, 2009-09-18 at 14:27 -0400, Seth Vidal wrote: > > On Fri, 18 Sep 2009, James Antill wrote: > > > At first glance this looks great but a number of places where we do the > same thing we'd want more or less the same function but a different > exception raised. > > Given it is a generic "

Re: [Yum-devel] First "usable" yum history patches

2009-09-18 Thread James Antill
On Tue, 2009-09-15 at 23:10 -0400, James Antill wrote: > On Wed, 2009-09-02 at 14:51 +0200, Tim Lauridsen wrote: > > > Did some testing with history-BIG.patch it is working great :) > > Cool. I've updated the big patch¹ with a couple of minor tweaks I've > done (obsoletes at least pretends to w

Re: [Yum-devel] [PATCH] Ignore old repomd.xml files if they fail to parse, BZ 520614.

2009-09-18 Thread Seth Vidal
On Fri, 18 Sep 2009, James Antill wrote: --- yum/yumRepo.py |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/yum/yumRepo.py b/yum/yumRepo.py index fac218a..367a68e 100644 --- a/yum/yumRepo.py +++ b/yum/yumRepo.py @@ -1034,6 +1034,8 @@ class YumRepository(Repository, con

Re: [Yum-devel] [PATCH] Be nice and don't traceback when people screw their rpmdb, BZ 522259

2009-09-18 Thread Seth Vidal
On Fri, 18 Sep 2009, James Antill wrote: At first glance this looks great but a number of places where we do the same thing we'd want more or less the same function but a different exception raised. Given it is a generic "Your rpmdb is screwed" exception would it make sense to raise the s

[Yum-devel] [PATCH] Ignore old repomd.xml files if they fail to parse, BZ 520614.

2009-09-18 Thread James Antill
--- yum/yumRepo.py |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/yum/yumRepo.py b/yum/yumRepo.py index fac218a..367a68e 100644 --- a/yum/yumRepo.py +++ b/yum/yumRepo.py @@ -1034,6 +1034,8 @@ class YumRepository(Repository, config.RepoConf): old_local = loca

[Yum-devel] [PATCH] Be nice and don't traceback when people screw their rpmdb, BZ 522259

2009-09-18 Thread James Antill
--- yum/__init__.py | 15 --- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/yum/__init__.py b/yum/__init__.py index d13f7a9..c5ca780 100644 --- a/yum/__init__.py +++ b/yum/__init__.py @@ -2592,12 +2592,21 @@ class YumBase(depsolve.Depsolve): pkgnames.up