[Devel] seas module

[Daniel-Constantin Mierla]

Hello,

On 03/08/07 13:05, Elias Baixas wrote:
> Hi,
>
> just did a
> find modules/seas/ -name "*.c" -o -name "*.h" |while read i; do sed 
> "s/SLOG(/LOG(/g" $i > $i.2 && mv $i.2 $i; done
>
> and it compiled without complains... so it should also in SunOS and 
> freeBSD V.whatever
great, thanks.
>
> I'm fixing the warnings also (just casting to (int))... in what 
> arch/OS where they seen ?

was Debian etch amd64.

Cheers,
Daniel

>
> regards,
>
> Elias
>
> Daniel-Constantin Mierla wrote:
>> Hello,
>>
>> apart (or related to) reported issues, there are warnings:
>>
>> ha.c: In function 'init_pingtable':
>> ha.c:310: warning: format '%d' expects type 'int', but argument 5 has 
>> type 'long unsigned int'
>> ...
>>
>> statistics.c: In function 'init_seas_stats_table':
>> statistics.c:48: warning: format '%d' expects type 'int', but 
>> argument 5 has type 'long unsigned int'
>> ...
>>
>> If there will be no feedback by tomorrow, I will revert your changes 
>> to custom logging system to a previous version. Not sure if along 
>> with moving to this new logging mechanism there were other fixes, so 
>> the alternative is to remove seas module from release 1.2.0.
>>
>> Cheers,
>> Daniel
>>
>>
>> On 03/06/07 14:44, Daniel-Constantin Mierla wrote:
>>> Hello Elias,
>>>
>>> any update on this?
>>>
>>> Cheers,
>>> Daniel
>>>
>>>
>>> On 03/01/07 19:49, Daniel-Constantin Mierla wrote:
>>>> Hello,
>>>>
>>>> as far as I see, the changes you introduced are not portable on 
>>>> SunOS and other architecture ... the SVN is frozen for the next 
>>>> release which is in about 1 week. Only fixes should be applied. We 
>>>> have no time to port to all OSes right now.
>>>>
>>>> Was it so necessary to add these changes right now? For the next 
>>>> future we have in mind to introduce options to get same kind of log 
>>>> messages. I face right now same problem compiling on gg 
>>>> 2.95/FreeBSD the SST module  which has as well a custom logging 
>>>> system. Would be better to stick to the default on in openser, and 
>>>> if it is limited, then we should improve that one, rather than 
>>>> adding specific solutions for each module.
>>>>
>>>> Cheers,
>>>> Daniel
>>>>
>>>>
>>>> On 03/01/07 19:25, elias.baixas at voztele.com wrote:
>>>>> Revision: 1725
>>>>>           
>>>>> http://openser.svn.sourceforge.net/openser/?rev=1725&view=rev
>>>>> Author:   eliasbaixas
>>>>> Date:     2007-03-01 09:25:55 -0800 (Thu, 01 Mar 2007)
>>>>>
>>>>> Log Message:
>>>>> -----------
>>>>> Changed the logging system
>>>>>
>>>>> Modified Paths:
>>>>> --------------
>>>>>     trunk/modules/seas/cluster.c
>>>>>     trunk/modules/seas/encode_allow.c
>>>>>     trunk/modules/seas/encode_contact.c
>>>>>     trunk/modules/seas/encode_cseq.c
>>>>>     trunk/modules/seas/encode_digest.c
>>>>>     trunk/modules/seas/encode_header.c
>>>>>     trunk/modules/seas/encode_msg.c
>>>>>     trunk/modules/seas/encode_route.c
>>>>>     trunk/modules/seas/encode_to_body.c
>>>>>     trunk/modules/seas/encode_uri.c
>>>>>     trunk/modules/seas/encode_via.c
>>>>>     trunk/modules/seas/event_dispatcher.c
>>>>>     trunk/modules/seas/ha.c
>>>>>     trunk/modules/seas/seas.c
>>>>>     trunk/modules/seas/seas.h
>>>>>     trunk/modules/seas/seas_action.c
>>>>>     trunk/modules/seas/seas_action.h
>>>>>     trunk/modules/seas/statistics.c
>>>>>     trunk/modules/seas/xaddress.h
>>>>>
>>>>>
>>>>> This was sent by the SourceForge.net collaborative development 
>>>>> platform, the world's largest Open Source development site.
>>>>>
>>>>> _______________________________________________
>>>>> Devel mailing list
>>>>> Devel at openser.org
>>>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>>>
>>>>>   
>>>>
>>>> _______________________________________________
>>>> Devel mailing list
>>>> Devel at openser.org
>>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>>
>>>
>>> _______________________________________________
>>> Devel mailing list
>>> Devel at openser.org
>>> http://openser.org/cgi-bin/mailman/listinfo/devel
>>>
>>
>
>