[Yum-devel] [PATCH 2/2] Make ts.check() unconditional + adjust related messages

James Antill james at fedoraproject.org
Wed Feb 9 15:39:23 UTC 2011


On Wed, 2011-02-09 at 17:11 +0200, Panu Matilainen wrote:
> On 02/09/2011 04:55 PM, James Antill wrote:
> > On Wed, 2011-02-09 at 16:44 +0200, Panu Matilainen wrote:
> >> On 02/09/2011 04:40 PM, James Antill wrote:
> >>>    Pretty sure we can't remove this, due to API compat. Otherwise, ACK.
> >>
> >> So instead of removing it, add a comment it's unused?
> >
> >   Yeh, that's fine.
> 
> Okay. What about the 1/2 patch to move the check_debug stuff into the 
> transaction wrapper ts.check(), which this patch needs?

 Yeh, ACK was recursive :).

>  That is an API 
> change, at least sort of...

 The old API was a private one (and has no callers I know about), so
it's fine to rm it.



More information about the Yum-devel mailing list