Re: [sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Daniel-Constantin Mierla
For reference, there is an ini-like config parser in core, see cfg_parser.h -- it is used by tls module, afaik. Cheers, Daniel On 10/21/13 4:14 AM, Ovidiu Sas wrote: I had a discussion with Henning about this a while ago and he insisted on keeping the libconfuse (more exactly the libconfuse st

Re: [sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Ovidiu Sas
I had a discussion with Henning about this a while ago and he insisted on keeping the libconfuse (more exactly the libconfuse style text config), don't remember exactly the details. I even raised this issue on the openser tracker: http://sourceforge.net/p/openser/feature-requests/145/ It would be

[sr-dev] git:master: pkg/kamailio/centos: updated rel in .spec

2013-10-20 Thread Peter Dunkley
Module: sip-router Branch: master Commit: 9f08b58f58585559646b470ae9f4b9801617ba73 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9f08b58f58585559646b470ae9f4b9801617ba73 Author: Peter Dunkley Committer: Peter Dunkley Date: Sun Oct 20 23:52:23 2013 +0100 pkg/kama

[sr-dev] git:master: pkg/kamailio/centos: removed out-of-date BoxGrinder appliance files

2013-10-20 Thread Peter Dunkley
Module: sip-router Branch: master Commit: ce9f46a3778cd6c79f2411b5934e3ad320cf6de4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ce9f46a3778cd6c79f2411b5934e3ad320cf6de4 Author: Peter Dunkley Committer: Peter Dunkley Date: Sun Oct 20 23:46:35 2013 +0100 pkg/kama

Re: [sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Peter Dunkley
Is this something that could be done for 4.1.0? Would it count as a new feature, or simply a "fix" to make the module available on a wider range of platforms? Regards, Peter On 20 October 2013 23:31, Ovidiu Sas wrote: > Why bother writing a parser for the file when there is db_text? > This w

Re: [sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Ovidiu Sas
Why bother writing a parser for the file when there is db_text? This will make the module simpler and easier to maintain. Regards, Ovidiu Sas On Sun, Oct 20, 2013 at 4:51 PM, Marius Zbihlei wrote: > Hi all, > > This has been in the back of my head from some time now. Libconfuse is a > small proj

[sr-dev] git:master: pkg/kamailio/centos: updated documentation

2013-10-20 Thread Peter Dunkley
Module: sip-router Branch: master Commit: 0de2ae72410f0672f174a8cae34d4ac12798a188 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0de2ae72410f0672f174a8cae34d4ac12798a188 Author: Peter Dunkley Committer: Peter Dunkley Date: Sun Oct 20 23:27:58 2013 +0100 pkg/kama

[sr-dev] Does iptrtpproxy still work? Should there be consideration into marking it obsolete in Kamailio 4.1.0

2013-10-20 Thread Peter Dunkley
Hello, I was looking through the module dependencies for this and noticed that it requires the netfilter-rtpproxy which has not had a new release in over two years. Further, there is a recent bug report (http://www.2p.cz/en/node/1419 ) which indicates that netfilter-rtpproxy won't even compile wi

[sr-dev] What version of memcached/libmemcached is needed for the memcached module in master?

2013-10-20 Thread Peter Dunkley
Hello, The memcached module cannot be built for "stock" Enterprise Linux 6 because the versions of libmemcached/memcached that come with EL6 are the wrong ones. Does anyone know what the right versions are, and is it worth updating the module documentation to specify them? That way if someone us

Re: [sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Peter Dunkley
Sounds like a good idea to me. I don't build carrierroute in Enterprise Linux RPMs (and don't even consider it as an option on my deployments) for exactly this reason. Regards, Peter On 20 October 2013 21:51, Marius Zbihlei wrote: > Hi all, > > This has been in the back of my head from some

[sr-dev] Removing carrierroute dependency on libconfuse

2013-10-20 Thread Marius Zbihlei
Hi all, This has been in the back of my head from some time now. Libconfuse is a small project not distributed with major distros and unmaintained for 3.5 years now. As it is only used for reading of a simple cfg file, I want to write a simple parser that does the job(I do not want external deps .

Re: [sr-dev] Patch: Fix SRV lookups when dns failover is enabled

2013-10-20 Thread Daniel-Constantin Mierla
I applied the two patches related to this matter. I only renamed the structure you defined, stripping '_t' from its name as this suffix is commonly used across the code as a typedef'ed struct name. Cheers, Daniel On 10/18/13 10:01 AM, Øyvind Kolbu wrote: On 17.10.2013 19:09, Daniel-Constantin

Re: [sr-dev] Patch: Respect Order field in NAPTR, as required by RFC 2915

2013-10-20 Thread Daniel-Constantin Mierla
Hello, thanks for the patch, I will look over it soon. Cheers, Daniel On 10/18/13 3:14 PM, Øyvind Kolbu wrote: Hi, Kamailio's NAPTR behavior by default ignores the Order field, which it is not allowed to do. Excerpt from RFC 2915: Order A 16-bit unsigned integer specifying the ord

[sr-dev] git:master: fix dns srv failover when no UDP record is present

2013-10-20 Thread Daniel-Constantin Mierla
Module: sip-router Branch: master Commit: 61ad39509ca1d739d00d0386ab5a991cf54477d8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=61ad39509ca1d739d00d0386ab5a991cf54477d8 Author: Øyvind Kolbu Committer: Daniel-Constantin Mierla Date: Sun Oct 20 22:04:16 2013 +0200