Re: [sr-dev] [kamailio/kamailio] Add kjansson_basic group (#990)

2017-02-14 Thread Daniel-Constantin Mierla
Merging and we will try to find a proper solution in the future. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Add kjansson_basic group (#990)

2017-02-14 Thread Daniel-Constantin Mierla
Merged #990. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/990#event-962573773___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] Add kjansson_basic group (#990)

2017-02-13 Thread Fabrice Fontaine
You're completely right, having that much groups is not the best approach. 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

Re: [sr-dev] [kamailio/kamailio] Add kjansson_basic group (#990)

2017-02-13 Thread Daniel-Constantin Mierla
I wonder if makes sense to split more into groups of a single module, because it is possible to install a single module at once, which I guess it is what you are looking for. Overall, I am not against this approach, but it may be better to understand the needs you have for packaging and other

[sr-dev] [kamailio/kamailio] Add kjansson_basic group (#990)

2017-02-13 Thread Fabrice Fontaine
This group could be used to compile kjansson module when libevent is not available Signed-off-by: Fabrice Fontaine You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/990 -- Commit Summary -- * Add