Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Jeremy Katz
On Wednesday, May 13 2009, Seth Vidal said: > On Wed, 13 May 2009, James Antill wrote: >> On Wed, 2009-05-13 at 14:16 -0400, Jeremy Katz wrote: >>> Rather than requiring higher level callers to catch the error, >>> we should catch the error at the low-level when opening the >>> repo file and turn i

Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Seth Vidal
On Wed, 13 May 2009, James Antill wrote: On Wed, 2009-05-13 at 14:16 -0400, Jeremy Katz wrote: Rather than requiring higher level callers to catch the error, we should catch the error at the low-level when opening the repo file and turn it into a RepoMDError. Related to rh bz#500526 Not th

Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread James Antill
On Wed, 2009-05-13 at 14:16 -0400, Jeremy Katz wrote: > Rather than requiring higher level callers to catch the error, > we should catch the error at the low-level when opening the > repo file and turn it into a RepoMDError. Related to > rh bz#500526 Not that I think this change is bad, but I do

Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Tim Lauridsen
On 05/13/2009 08:19 PM, Seth Vidal wrote: On Wed, 13 May 2009, Jeremy Katz wrote: Rather than requiring higher level callers to catch the error, we should catch the error at the low-level when opening the repo file and turn it into a RepoMDError. Related to rh bz#500526 yah - that looks ko

Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Jeremy Katz
On Wednesday, May 13 2009, Seth Vidal said: > On Wed, 13 May 2009, Jeremy Katz wrote: >> Rather than requiring higher level callers to catch the error, >> we should catch the error at the low-level when opening the >> repo file and turn it into a RepoMDError. Related to >> rh bz#500526 > > yah - t

Re: [Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Seth Vidal
On Wed, 13 May 2009, Jeremy Katz wrote: Rather than requiring higher level callers to catch the error, we should catch the error at the low-level when opening the repo file and turn it into a RepoMDError. Related to rh bz#500526 yah - that looks kosher - though it might technically be an AP

[Yum-devel] [PATCH] Catch error opening repo files

2009-05-13 Thread Jeremy Katz
Rather than requiring higher level callers to catch the error, we should catch the error at the low-level when opening the repo file and turn it into a RepoMDError. Related to rh bz#500526 --- yum/repoMDObject.py |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/yum/rep