Also need do cherry-pick to 5.1 branch You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1316
-- Commit Summary --
* pkg/kamailio/obs: Added missed directories to packaging (non systemd dists) * pkg/kamailio/obs: Ported ca39ea810c01f6947db70870cc1d786e95cb4bc3 to OBS SPEC * pkg/kamailio/obs: Ported 32c5d9dd9f7a5e02ab428e5c16bf7406e4ff7856 ot OBS spec file * pkg/kamailio/obs: Added sipdump module * pkg/kamailio/obs: Added acc_diameter module files
-- File Changes --
M pkg/kamailio/obs/kamailio.spec (137)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1316.patch https://github.com/kamailio/kamailio/pull/1316.diff
oyvindkolbu commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
This reverts 612e96d8a2cc2a9ac31a7fb05508084fb82dbcf5. Unintentional?
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
This section is applicable to RHEL7 dist only, not CentOS7. RHEL7 is not have libunistring package and build will fail.
Same is related to sctp. RHEL not have [lksctp-tools-devel ](https://build.opensuse.org/package/show/home:sergey-safarov:v5.0.x-rpms/kama...)
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
What repo you used to build `sctp` and `websocket` modules on `RHEL`?
@sergey-safarov pushed 1 commit.
afbcb96 pkg/kamailio/obs: Fixed build of lua module
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
Created PR https://github.com/kamailio/kamailio/pull/1317
oyvindkolbu commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
Both libunistring-devel and lksctp-tools-devel are in the rhel7-server-optional repo, which my machine has enabled by default.
When bulding packages in EPEL they enable the same repo, ref http://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#Policy
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
Could you help find way how to attache this optional repo on OpenSUSE build server. I see OBS not provide this optional repo. Without this optional repo we cannot build this packages on available infrastructure.
miconda commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
With a quick search I found some details at:
* https://access.redhat.com/documentation/en-us/red_hat_subscription_managemen...
But not sure if that command can be used in OBS.
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
yes Daniel This is requires active RedHat support contact.
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
I think add instruction how to build this modules to README file for RHEL dist.
@sergey-safarov pushed 1 commit.
35f28d7 pkg/kamailio/rhel: Updated build instructions
sergey-safarov commented on this pull request.
%bcond_without redis
-%bcond_without sctp -%bcond_without websocket +%bcond_with sctp +%bcond_with websocket %bcond_without xmlrpc
@oyvindkolbu Please test instructions on commit https://github.com/kamailio/kamailio/pull/1316/commits/35f28d741f485a8b6b6ff... And let me know, please.
@sergey-safarov, @oyvindkolbu -- was all sorted out here? Should I merge this one?
yes, Daniel. This may be merged. Readme instruction i can check when get access to RHEL with optional repo.
Merged #1316.
-
Daniel-Constantin Mierla -
oyvindkolbu
-
sergey-safarov