[OpenSER-Devel] svn status

[Daniel-Constantin Mierla]

On 06/24/08 00:07, Dan Pascu wrote:
> On Monday 23 June 2008, Daniel-Constantin Mierla wrote:
>   
>>  From my perspective is unfrozen in regard to local_route -- 
>>     
>
> To be honest I'm a bit put off by this. The svn was frozen on 6-7 June and 
> the consensus was that the release should be made on 7 July if no 
> extension was required. Then the request was made to un-froze it for 4-5 
> days to fix issues with the local_route.
to unfroze and let the other developers to review their code and 
integrate with local_route.That didn't happen because there were claims 
that local_route do not break architecture, has no impact on other code 
and functions in modules were enabled for local_route without developer 
agreement. The discussion moved on technical aspects of local_route impact.

>  Yet we are almost 3 weeks later, 
> with the original release date less than 2 weeks away, and the svn is 
> still unfrozen, nothing was done to set things straight and discussion 
> switched from fixing the issues to removing the new feature.
>   
Proper fixing will take too long, removing was not my case, but 
disabling it by default with defines. Alternative is to let each 
developer to fix it how it considers for its code -- but this last 
solution will create a rather patched release, bringing more inconsistency.

I guess it is time to decide:
- disable local_route with defines (have the local_route functionality 
as it is now for 1.4 and resume development and integration after unfreeze)
- let developers apply fixes how they consider to be best for their code 
in 1.4 (after unfreeze, when integration and design is properly done, 
remove these patches)

Cheers,
Daniel

>   
>>  things are 
>> very inconsistent there, internal design affected and too short time to
>> make it right if we stick close to original release plan. Therefore,
>> the steps I see:
>> - enclose local_route in defines and disable it for 1.4
>> - resume the testing and review of other stuff to ensure that proper
>> functionality is preserved and new features are ok
>> - release in one moth if everything goes fine
>>     
>
>   

-- 
http://www.asipto.com