On 02/15/2011 05:07 PM, James Antill wrote:
On Tue, 2011-02-15 at 13:06 +0200, Panu Matilainen wrote:
Avoids unnecessary import dependency from rpmsack to the higher
level depsolve class, allowing rpmsack to be used earlier.
---
  yum/depsolve.py |    7 +------
  yum/rpmsack.py  |   11 ++++++++---
  2 files changed, 9 insertions(+), 9 deletions(-)

  ACK, I guess. Although this is just for the config. usage?

Yup, that was the only immediate goal with this - to remove the loop that prevents rpmsack use from config. So whether I care at all about this patch depends on what the status for the rpmsack-from-config ends up being :)

  It esp. feels kind of weird as there are a couple of other uses of
rpm.blah in depsolve still.
  ts_flags_to_rpm seems like it'd be easy to move to rpmsack, the bit in
processConflict is much more blah though.

  Technically this is an API change, but I really hope nobody is relying
on it.

API users shouldn't really see a difference as it's depsolve.py imports the moved flags with the original name.

        - Panu -
_______________________________________________
Yum-devel mailing list
[email protected]
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to