22 Dec
2021
22 Dec
'21
12:49 p.m.
Fine to merge it from my point of view, only one comment as I noticed in the makefile it needs both libuv and libevent. I thought they are alternative for each other, not expecting to be needed at the same time, but maybe they are required in this manner because of dependencies.
@miconda thank you for the feedback. I agree with you, we should probably not use two different eventing libraries. I will work on switching to libuv for the publish worker and submit a new PR.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/2978#issuecomment-999552463
You are receiving this because you are subscribed to this thread.
Message ID: kamailio/kamailio/pull/2978/c999552463@github.com