<p>You're completely right, having that much groups is not the best approach.</p>
<p>However, group_include and include_modules options can not be mixed. So I have to stick with one of these variables. I could use include_modules instead of group_include but then I would have to recopy most of what has been done in Makefile.groups that is to say trying to guess what modules should be build depending on what packages are installed on his (embedded) system.</p>
<p>A better approach would be to add a configure script (or Makefile variables) that would allow the user to enable/disable modules based on third-party dependencies. For example, we could add --enable-libevent, --enable-curl, --enable-ssl and so on. In case the user did not specify an enable/disable for a specific option, kamailio could use pkgconfig to try to check if the external dependency is installed on the system. Most of the open source components have this approach but it would be a bigger addition to kamailio.</p>
<p>For information, except kjanssson_basic, my last need would be to split kldap group as db2_ldap module depends on ldap and sasl.</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/990#issuecomment-279532834">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZcjUEK7I4PikZ041jraf4Hf3qjz8ks5rcM8ngaJpZM4L_fDw">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZWATrj4zGlUfl0RZ7PB4kIc8OV0Uks5rcM8ngaJpZM4L_fDw.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/990#issuecomment-279532834"></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":"@ffontaine in #990: You're completely right, having that much groups is not the best approach.\r\n\r\nHowever, group_include and include_modules options can not be mixed. So I have to stick with one of these variables. I could use include_modules instead of group_include but then I would have to recopy most of what has been done in Makefile.groups that is to say trying to guess what modules should be build depending on what packages are installed on his (embedded) system.\r\n\r\nA better approach would be to add a configure script (or Makefile variables) that would allow the user to enable/disable modules based on third-party dependencies. For example, we could add --enable-libevent, --enable-curl, --enable-ssl and so on. In case the user did not specify an enable/disable for a specific option, kamailio could use pkgconfig to try to check if the external dependency is installed on the system. Most of the open source components have this approach but it would be a bigger addition to kamailio.\r\n\r\nFor information, except kjanssson_basic, my last need would be to split kldap group as db2_ldap module depends on ldap and sasl."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/990#issuecomment-279532834"}}}</script>