#### Pre-Submission Checklist - [X] Commit message has the format required by CONTRIBUTING guide - [X] Commits are split per component (core, individual modules, libs, utils, ...) - [X] Each component has a single commit (if not, squash them into one commit) - [X] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [X] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist: <!-- Go over all points below, and after creating the PR, tick the checkboxes that apply --> - [ ] PR should be backported to stable branches - [X] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description EPEL now includes a openssl11 package for CentOS 7. This feature is for users who want to build kamailio-tls with OpenSSL 1.1.1 instead of OpenSSL 1.0.2.
It is implemented as a `--with=openssl11` conditional (the default os off), so should not affect/change the regular CentOS 7 builds as distributed by kamailio.org.
This is for adventurous users on CentOS 7 willing to maintain their own OpenSSL 1.1.1-enabled TLS package. EPEL dependencies( openssl11-libs, openssl11-devel) are required at runtime and for build.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/2779
-- Commit Summary --
* pkg: enable CentOS 7 to build with OpenSSL 1.1.1
-- File Changes --
M pkg/kamailio/obs/kamailio.spec (32)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/2779.patch https://github.com/kamailio/kamailio/pull/2779.diff
@sergey-safarov pushed 1 commit.
11ea00e7d3028db754e498690d5f6862e895b2dc moved openssl11 outside of modules build definition [skip ci]
Hello @space88man Could build with my commit. I prefer dist specific options to move outside of module definition.
Hello @space88man Could build with my commit. I prefer dist specific options to move outside of module definition.
Thanks - with your change I was able to build inside my mock. Force pushed.
Hi @space88man Could you also replace `%{?centos_ver}` macro to `%{?rhel}`. *Example* ``` %if 0%{?centos_ver} == 7 && %{with openssl11} ``` To ``` %if 0%{?rhel} == 7 && %{with openssl11} ```
This will allow using the same approach for RHEL dist too.
Could you also replace `%{?centos_ver}` macro to `%{?rhel}`.
Done — rebased and force pushed.
Merged #2779 into master.
Will this be pushed to 5.4 also or only to 5.5?
Hello Daniel @miconda What is a policy in this case?
Will this be pushed to 5.4 also or only to 5.5?
Hi Daniel @miconda , is there a plan to push this to 5.4?
cherry-piked at 106997d1d1924c987f7e6d2ff32c4d40d7ee0e10 will be compiled with next release.
Hello,
Do you have an estimation on when this will be compiled in the next 5.4 release?
Hi @sergey-safarov ,
Do you have an estimation on when this will be compiled in the next 5.4 release?
cherry-piked at [106997d](https://github.com/kamailio/kamailio/commit/106997d1d1924c987f7e6d2ff32c4d40...) will be compiled with next release.
Hi @vhatzist I do not manage release dates. Let's wait for other members feedback.