[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