You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1826
-- Commit Summary --
* pkg/docker: Updated submodule * pkg/kamailio/alpine: Update APKBUILD * pkg/docker: Alpine switched to use openssl. Updated rules for alpine docker container * kamdbctl: Merged patch from alpine packaging (#1786)
-- File Changes --
M pkg/docker (2) D pkg/kamailio/alpine/0004-src_core_tcp_read_c.patch (20) M pkg/kamailio/alpine/APKBUILD (12) M pkg/kamailio/alpine/kamailio.initd (2) D pkg/kamailio/alpine/kamctl_build.patch (236)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1826.patch https://github.com/kamailio/kamailio/pull/1826.diff
The patch for 78e948d does not seem to touch kamdbctl at all, so the commit message is not matching what the patch does.
At https://github.com/kamailio/kamailio/pull/1786 commit partially (but not all) is back ported to 5.1 branch. Remaining part is important for `kamailio:kamailio-ci` docker image and here is cherry-picked remaining strings.
For the future, a patch that touches different components must be split in commits per component. Right now that commit has nothing to do with kamdbctl. Overall, it is not something critical, but is good to keep the commit message coherent with what it does, because the commits history is used quite often to try to track back changes. I am going to merge it, thanks.
Merged #1826 into 5.1.