[Rpm-maint] [rpm-software-management/rpm] Cleanups for running external programs (#1047)
Florian Festi
notifications at github.com
Wed Feb 5 10:34:03 UTC 2020
While not using rpmlog in the forked process is this right thing I am not sure if the replacement is good enough. For one it omits the N_("error: ") previously printed as rpmlogMsgPrefix for RPMLOG_ERR, second it assumes that the message should be printed at all. This is a complicated problem as the rpmlogCallback can do whatever it wants with the log entry. We could check whether the default handler is installed or just output the message in any case (as the patch does right now). Not 100% sure yet what's the best thing to do.
--
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/1047#issuecomment-582343995
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20200205/126680c4/attachment.html>
More information about the Rpm-maint
mailing list