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
_______________________________________________
Rpm-maint mailing list
[email protected]
http://lists.rpm.org/mailman/listinfo/rpm-maint
