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

_______________________________________________
Yum-devel mailing list
Yum-devel@lists.baseurl.org
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to