[Rpm-maint] [rpm-software-management/rpm] Rip out the atexit handler (#1667)
Panu Matilainen
notifications at github.com
Mon Jun 14 08:08:25 UTC 2021
It should be safe from the db POV to remove this now, yes. However to me this is only a part of a bigger change to remove the associated and now unnecessary signal queue from rpm API and ABI, and there's not a whole lot benefit in doing this part without doing the others (the whole rpm*Rock tracking in rpmdb.c is thread-unsafe) as well. It may also have unexpected side-effects that we'd rather deal with all at once when the signal handler gets purged.
In other words, the change is not wrong as such and will happen sooner or later as a part of a bigger change, but not yet. Thanks for the patch though.
--
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/1667#issuecomment-860479467
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210614/48c0c496/attachment.html>
More information about the Rpm-maint
mailing list