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

[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 "Your rpmdb is screwed" exception would it make 
> sense to raise the same exception we raise in rpmsack.py:
> 
> raise Errors.PackageSackError, 'Rpmdb changed underneath us'
> 
> more or less.
> 
> Thoughts?
> 
> If you think we should have one of these per major command method in yum, 
> that's fine, too - just a thought.

 What do you think about adding the raise to
getInstalledPackageObject(), and removing the deprecation warning there?
Then just having a new RpmDBError exception?