Re: [Yum-devel] [PATCH] Fix no script output from transaction, BZ 554462

2010-01-20 Thread Christoph Thiel
On Wed, Jan 20, 2010 at 10:35:44AM -0500, Seth Vidal wrote: > On Wed, 20 Jan 2010, Panu Matilainen wrote: > > >The patch cannot have been correct, really. Either it would've > >required other changes to TransactionWrapper class, or (perhaps > >more likely) the change just masked away the real prob

Re: [Yum-devel] [PATCH] Fix no script output from transaction, BZ 554462

2010-01-20 Thread Seth Vidal
On Wed, 20 Jan 2010, Christoph Thiel wrote: On Wed, Jan 20, 2010 at 10:35:44AM -0500, Seth Vidal wrote: On Wed, 20 Jan 2010, Panu Matilainen wrote: The patch cannot have been correct, really. Either it would've required other changes to TransactionWrapper class, or (perhaps more likely) the

Re: [Yum-devel] [PATCH] Fix no script output from transaction, BZ 554462

2010-01-20 Thread Seth Vidal
On Wed, 20 Jan 2010, Panu Matilainen wrote: The patch cannot have been correct, really. Either it would've required other changes to TransactionWrapper class, or (perhaps more likely) the change just masked away the real problem. I don't disagree with you - I just wanted an explanation of

Re: [Yum-devel] [PATCH] Fix no script output from transaction, BZ 554462

2010-01-20 Thread Panu Matilainen
On Tue, 19 Jan 2010, Seth Vidal wrote: On Tue, 19 Jan 2010, James Antill wrote: --- yum/rpmtrans.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/yum/rpmtrans.py b/yum/rpmtrans.py index f33f048..0340153 100644 --- a/yum/rpmtrans.py +++ b/yum/rpmtrans.py @@ -209,7 +2