Re: [Rpm-maint] [rpm-software-management/rpm] 3 cleanup fixes (#359)

2018-06-04 Thread Alan Jenkins
Agreed, fab7348 removes the need for my commit. I notice exit() does not seem to be on the list of POSIX signal-safe functions (in contrast with `_exit()`). I guess in general, if rpm is really trying to clean up inside a signal handler, I'd expect a lot of really hairy edge cases. E.g.

Re: [Rpm-maint] [rpm-software-management/rpm] 3 cleanup fixes (#359)

2018-06-04 Thread Panu Matilainen
Finally got around to investigate the rpmdb leak thing properly... @ffesti pointed out that perhaps a better fix is to ensure the new handle is always added to the list. And indeed, this actually fixes another problem too: there was a window where a newly opened database would not be properly

Re: [Rpm-maint] [rpm-software-management/rpm] 3 cleanup fixes (#359)

2018-06-04 Thread Panu Matilainen
Closed #359. -- 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/359#event-1661156271___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] lua: add rpm.execute() (#390)

2018-06-04 Thread Panu Matilainen
Missed the part about moving to rpmio/rpmlua.c... anyway, been long enough - I adjusted that part myself and pushed as commit 18e792340e33eade01967d01d5801f1ae9e0ad33 (and cb2ae4bdf2f60876fdc68e3f84938e9c37182fab). Thanks for the patches! -- You are receiving this because you are subscribed

Re: [Rpm-maint] [rpm-software-management/rpm] lua: add rpm.execute() (#390)

2018-06-04 Thread Panu Matilainen
Closed #390. -- 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/390#event-1661014629___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] RFC: rpmlib efi provides (#438)

2018-06-04 Thread Florian Festi
I am closing this PR as the proposed code is clearly not going in. Further discussion can be done in the newly created #446 -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] RFC: rpmlib efi provides (#438)

2018-06-04 Thread Florian Festi
Closed #438. -- 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/438#event-1660932368___ Rpm-maint mailing list

[Rpm-maint] [rpm-software-management/rpm] RFE: Depenencies matching to system properties out side of packages (#446)

2018-06-04 Thread Florian Festi
This has come up in #438 For now there is no way to use the dependency system to configure the system depending on hardware or other properties. There is something like this in libzypp (used by openSUSE) which injects system Provides into the transaction. Quoting from #438: Main use of such a