[kamailio/kamailio] pkg/kamailio/centos: Updated RPM spec file to support build fedora, RHEL, openSUSE (#1227)
Updated RPM spec file to support build fedora, RHEL, openSUSE via openSUSE build service. Removed unnecessary RPM spec files
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1227
-- Commit Summary --
* pkg/kamailio/centos. Updated RPM spec file to support build fedora, RHEL, openSUSE via openSUSE build service * pkg/kamailio: Removed unnecessary RPM spec files
-- File Changes --
A pkg/kamailio/centos/6 (1) D pkg/kamailio/centos/6/README (29) D pkg/kamailio/centos/6/kamailio.init (134) D pkg/kamailio/centos/6/kamailio.spec (1279) D pkg/kamailio/centos/6/kamailio.sysconfig (30) M pkg/kamailio/centos/7/kamailio.spec (558) D pkg/kamailio/fedora/17/README (67) D pkg/kamailio/fedora/17/kamailio-build.appl (49) D pkg/kamailio/fedora/17/kamailio.appl (42) D pkg/kamailio/fedora/17/kamailio.service (14) D pkg/kamailio/fedora/17/kamailio.spec (1325) D pkg/kamailio/fedora/17/kamailio.sysconfig (12) D pkg/kamailio/fedora/18/README (1) D pkg/kamailio/fedora/18/kamailio-build.appl (49) D pkg/kamailio/fedora/18/kamailio.appl (42) D pkg/kamailio/fedora/18/kamailio.service (1) D pkg/kamailio/fedora/18/kamailio.spec (1) D pkg/kamailio/fedora/18/kamailio.sysconfig (1) A pkg/kamailio/oracle (1) D pkg/kamailio/oracle/el6/kamailio.default (30) D pkg/kamailio/oracle/el6/kamailio.init (128) D pkg/kamailio/oracle/el6/kamailio.spec (852) D pkg/kamailio/oracle/el7/kamailio.service (17) D pkg/kamailio/oracle/el7/kamailio.spec (851) D pkg/kamailio/oracle/el7/kamailio.sysconfig (36) D pkg/kamailio/oracle/kamailio.logrotate (7) D pkg/kamailio/oracle/kamailio.rsyslog (2) D pkg/kamailio/rpm/kamailio.default (30) D pkg/kamailio/rpm/kamailio.init (128) D pkg/kamailio/rpm/kamailio.init.SuSE (133) D pkg/kamailio/rpm/kamailio.spec-4.1 (409) D pkg/kamailio/rpm/kamailio.spec.CenOS (745) D pkg/kamailio/rpm/kamailio.spec.SuSE (389) A pkg/kamailio/suse/1315 (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1227.patch https://github.com/kamailio/kamailio/pull/1227.diff
Using this RPM spec file created packages for Fedora, CentOS, RHEL, OpenSUSE distrs [more info](https://build.opensuse.org/package/show/home:sergey-safarov:v5.0.x-rpms/kama...)
Is this PR removing old specs and adds the one for OpenSuse Build Service (OBS)?
Yes. Now one SPEC file for all RPM based dists. Old SPEC is removed.
As far as I understood, at least it was the case in the past, those `%if` in OBS were extensions of OBS, not working on usual rpm build. That's the reason I did OBS specs only in the OBS repo, not in Kamailio repo.
So I would rather keep the old specs as they are for those that want to build locally and have obs folder to host the specs for the suse build service. Later, if proves to be ok to use the obs spec for building locally on the rpm distros (centos, fedora, redhat, oracle, ...), then we can either make symlinks or remove the specific folder.
Therefore, you should make a PR just for inclusion of the OBS spec files. Removal of the others will be subject of further discussions in the community.
Btw, there has to be a PR for master branch first. Then one to backport to 5.0.
Created PR https://github.com/kamailio/kamailio/pull/1228
Closed #1227.
-
Daniel-Constantin Mierla -
sergey-safarov