Push needs to store the previous vsflags value, not the new one,
so that pop can restore. Also pop was trying to access already removed
value from the "stack", without much success.
---
 rpmUtils/transaction.py |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/rpmUtils/transaction.py b/rpmUtils/transaction.py
index 897c30f..569c8a1 100644
--- a/rpmUtils/transaction.py
+++ b/rpmUtils/transaction.py
@@ -79,12 +79,10 @@ class TransactionWrapper:
     # set value, and we can potentiall debug a bit
     # easier
     def pushVSFlags(self, flags):
-        self.tsflags.append(flags)
-        self.ts.setVSFlags(self.tsflags[-1])
+        self.tsflags.append(self.ts.setVSFlags(flags))
 
     def popVSFlags(self):
-        del self.tsflags[-1]
-        self.ts.setVSFlags(self.tsflags[-1])
+        self.ts.setVSFlags(self.tsflags.pop())
 
     def addTsFlag(self, flag):
         curflags = self.ts.setFlags(0)
-- 
1.7.4

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

Reply via email to