<p>Thanks for the PR and your significant contributions to rpm packaging.</p>
<p>But again I have to say that I am not confident the (sym)linking to obs folder is the right solution for other distributions. obs itself is by design an packing system for many distros. Just storing in the repo symlinks is creating some relations that should not be placed there.</p>
<p>As an idea that popped up right now in my mind, maybe it is better to have the folder rhel with a README.md and a Makefile inside. The makefile will have some rules to create subfolders for rhel versions (e.g., 6, 7...) that can be symlinks to obs if that works for packaging. The readme will explain that. Then the symlinks are created on the local copy of the person in need to do the packaging.</p>
<p>I know that there is a symlink for one of the packaging specs, but I find it confusing and not necessary to store it in the repo, being something needed to be created temporary.</p>
<p>Maybe we can discuss more during the irc devel meeting:</p>
<ul>
<li><a href="https://www.kamailio.org/wiki/devel/irc-meetings/2017b">https://www.kamailio.org/wiki/devel/irc-meetings/2017b</a></li>
</ul>
<p>To conclude: I am fine to merge your updates, but remove those symlinking, let's discuss in the community how to do that properly.</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/1241#issuecomment-331373764">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZdsKddKjwZBl6mHoeCCt2RA-0yQzks5sk2MfgaJpZM4PgROZ">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZaRLEmpJ-SrSeTNHgCleS7fF-pS9ks5sk2MfgaJpZM4PgROZ.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/1241#issuecomment-331373764"></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 #1241: Thanks for the PR and your significant contributions to rpm packaging.\r\n\r\nBut again I have to say that I am not confident the (sym)linking to obs folder is the right solution for other distributions. obs itself is by design an packing system for many distros. Just storing in the repo symlinks is creating some relations that should not be placed there.\r\n\r\nAs an idea that popped up right now in my mind, maybe it is better to have the folder rhel with a README.md and a Makefile inside. The makefile will have some rules to create subfolders for rhel versions (e.g., 6, 7...) that can be symlinks to obs if that works for packaging. The readme will explain that. Then the symlinks are created on the local copy of the person in need to do the packaging.\r\n\r\nI know that there is a symlink for one of the packaging specs, but I find it confusing and not necessary to store it in the repo, being something needed to be created temporary.\r\n\r\nMaybe we can discuss more during the irc devel meeting:\r\n\r\n  * https://www.kamailio.org/wiki/devel/irc-meetings/2017b\r\n\r\nTo conclude: I am fine to merge your updates, but remove those symlinking, let's discuss in the community how to do that properly."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1241#issuecomment-331373764"}}}</script>