<p>As far as I understood, at least it was the case in the past, those <code>%if</code> 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.</p>
<p>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.</p>
<p>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.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/1227#issuecomment-326911243">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZaMmwwjFJO6ImDUc6kRO-RccllE5ks5se8AygaJpZM4PLRqe">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZSy0NGv4y4IGYJdFp0aEsIH_d-bvks5se8AygaJpZM4PLRqe.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/kamailio/kamailio/pull/1227#issuecomment-326911243"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@miconda in #1227: 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.\r\n\r\nSo 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.\r\n\r\nTherefore, 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."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1227#issuecomment-326911243"}}}</script>