<p>Thanks for the contribution!</p>
<p>The option to enable features in a module with compile time defines is not preferred at all. We had in the past the RADIUS (and DIAMETER) accounting as part of acc and it was very inconvenient to maintain or package.</p>
<p>The solution is to implement this as a new module on top of acc module, like <code>acc_json</code>, similar to <code>acc_radius</code>. In this way the module doesn't have compile time defines and can be packaged separately as part of jansson-modules pkg.</p>
<p>It should be very easy to build the new module by reusing the code you have in this patch. I did the split of acc_radius module.</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/1437#issuecomment-365178761">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZfBrE0leWW7Hd9pvw0aAJQ6OLTq3ks5tUT_IgaJpZM4SCznD">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AF36ZXEZoGIxm61SrELUIzfsaJuqGd41ks5tUT_IgaJpZM4SCznD.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/1437#issuecomment-365178761"></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 #1437: Thanks for the contribution!\r\n\r\nThe option to enable features in a module with compile time defines is not preferred at all. We had in the past the RADIUS (and DIAMETER) accounting as part of acc and it was very inconvenient to maintain or package.\r\n\r\nThe solution is to implement this as a new module on top of acc module, like `acc_json`, similar to `acc_radius`. In this way the module doesn't have compile time defines and can be packaged separately as part of jansson-modules pkg.\r\n\r\nIt should be very easy to build the new module by reusing the code you have in this patch. I did the split of acc_radius module."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1437#issuecomment-365178761"}}}</script>