Re: [Rpm-maint] [rpm-software-management/rpm] lib/transaction: Don't SEGV in sync if a txn was failed (#319)

2017-09-11 Thread Panu Matilainen
Closed #319. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/319#event-1242722957___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] lib/transaction: Don't SEGV in sync if a txn was failed (#319)

2017-09-11 Thread Panu Matilainen
Fixed differently (cause instead of symptom) in commits 9e65d6ee9ab851db5b5d4f0b390330abb6d38029 and 7481f0ff15e4d639301fffbbe402a0d27ed02b93: number of transaction elements was the wrong condition for syncing, sync should only be done if transaction actually ran. The second commit ensures

Re: [Rpm-maint] [rpm-software-management/rpm] lib/transaction: Don't SEGV in sync if a txn was failed (#319)

2017-09-08 Thread Panu Matilainen
Right, it shouldn't crash. But disabled disk-space checking silently disabling sync is wrong too. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Rpm-maint] [rpm-software-management/rpm] lib/transaction: Don't SEGV in sync if a txn was failed (#319)

2017-09-07 Thread Colin Walters
I was actually testing a change to better handle `rpm -e atomic` on Fedora Atomic Host, wondering why my patch was crashing, but in fact it was the recently added sync code in master. You can view, comment on, or merge this pull request online at: