geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sachin Patel <sppat...@gmail.com>
Subject Re: [Vote: RTC] needed for eclipse plugin
Date Fri, 02 Jun 2006 14:40:11 GMT
One option is if we could allow non-committers to vote if they  
declare that they have applied AND tested the patch.

On Jun 2, 2006, at 10:23 AM, Kevan Miller wrote:

> It seems that there are now +1's from 3 committers for this change.  
> Although I also support the change (once copyrights are corected),  
> I cannot offer a +1. Apologies for being a pedant, but to my  
> knowledge the current RTC "rules" we are living under are:
>
>   'I have applied this patch and tested it and found it good'
>
> I'm pretty sure that there's only one person who has done this and  
> his name is Sachin. Although it's likely that I could apply the  
> patch and build devtools, I wouldn't know how to integrate and test  
> the change.
>
> I think many of us feel that the requirement to apply and test a  
> patch is too restrictive and cumbersome -- especially in smaller  
> subprojects such as devtools and daytrader. Rather than ignoring  
> this directive, let's get the process changed or at least start a  
> discussion... I'll start a thread.
>
> --kevan
>
> On Jun 1, 2006, at 9:42 AM, Sachin Patel wrote:
>
>> The following big change is part 1 of 2 to correctly handle  
>> loading of the deployment plan editors.  To briefly summarize, the  
>> change introduces a new extension point which requires an  
>> implementation of IGeronimoFormContentLoader.  Previously the  
>> editor extension point loaded the editor for 1.0 plans.  Now the  
>> editor must handle multiple versions of the deployment plans so  
>> now there is a single editor extension def which loads a single  
>> editor impl, SharedDeploymentEditor which loads all the  
>> IGeronimoFormContent loaders (one impl for 1.0 and another for  
>> 1.1) and delegates to the appropriate emf models to generate the  
>> UI for the plans.  With this commit, the 1.0 plans load correctly  
>> and there is some work still need to be done for 1.1 to be done.   
>> There is currently some duplicate code which will need to be  
>> refactored, part 2 will address this, but since the patch is about  
>> 3000 lines I wanted to go ahead and commit.
>>
>> Please vote...
>>
>> Thanks.
>> <patch2.txt>
>>
>> -sachin
>>
>>
>


-sachin



Mime
View raw message