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

Seth Vidal skvidal at fedoraproject.org
Fri Sep 18 22:12:26 UTC 2009



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 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?

Ack
-sv



More information about the Yum-devel mailing list