10 Mar
2009
10 Mar
'09
8:08 p.m.
Perfect. All the kamailio modules are now listed under:
http://sip-router.org/wiki/devel/kamailio-integration#module_integration_st…
I marked the snmpstats module as being in "OK" status.
We should update all the other modules that are compiling with the
"OK" status and fill in the blocking items for the modules that are
not compiling.
As soon as I have new patches, I will submit them to you for integration.
Thank you for your work,
Ovidiu Sas
On Tue, Mar 10, 2009 at 7:58 PM, Jan Janak <jan(a)iptel.org> wrote:
Ovidiu,
Thanks a lot, it is in the repository:
http://git.sip-router.org/cgi-bin/gitweb.cgi?p=kamailio-3.0/.git;a=shortlog…
I also added the original kamailio statistics interface into libkcore
and snmpstats module now compiles.
Jan.
On 10-03 13:03, Ovidiu Sas wrote:
Hello Jan,
Here's a small patch that will fix the OPENSER_OID issue.
Regards,
Ovidiu Sas
On Tue, Mar 10, 2009 at 9:32 AM, Ovidiu Sas <osas(a)voipembedded.com> wrote:
> Having the OPENSER_OID defined inside the snmstats module should work.
>
> I think that all k module (except tm) should be part of a single
> branch. Like this, one fix in a module can be visible for other
> modules that relay on it. It will make compile fixes to be solved
> faster.
>
>
> Regards,
> Ovidiu Sas
>
> On Tue, Mar 10, 2009 at 8:43 AM, Jan Janak <jan(a)iptel.org> wrote:
>> On 10-03 00:34, Ovidiu Sas wrote:
>>> The snmpstats branch should be updated with the latest updates for the
>>> usrloc module. This will get rid of some db related compile issues.
>>
>> Oh yes, this is something I forgot to mention in my initial email. Modules
>> that include headers from other modules won't compile because every module
>> branch only contains only changes related to that particular module.
>>
>> Modules that other depend on are: alias_db, usrloc, and presence. I haven't
>> put the changes in the git repository because I had a script that did the
>> updates, but now I realized that others might have troubles with that so
I'll
>> fix it in the public repository.
>>
>> Thanks for pointing this out.
>>
>>> Also, OPENSER_OID needs to be defined inside config.h:
>>
>> Given that there is only one module (snmpstats) using OPENSER_OID,
shouldn't
>> we rather put the define in the module? What do you think?
>>
>> Jan.
>>
>