Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geronimo.apache.org
Received-SPF: pass (asf.osuosl.org: domain of jrsisson@gmail.com designates
 64.233.184.233 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:user-agent:mime-version:to:subject:references:in-reply-to:content-type:content-transfer-encoding;
        b=dNM1G9R62MYC6PhRfyBHf97AfPvF6Ak/X8urVa27nXF1KHs9eEzbomTMy9CyyP+4kH1MFsTJHVSuHUUiUxx7AvrhMRm8k3U7crGfS10pP+VdndTR+cVJLNeHJT3xKyM41EKNoo2WCLB3H+v+vcsLHJPtD0qsWXR/+pXpWJ+F3Do=
Message-ID: <44810527.9020902@gmail.com>
Date: Sat, 03 Jun 2006 13:42:31 +1000
From: John Sisson <jrsisson@gmail.com>
User-Agent: Thunderbird 1.5.0.2 (Windows/20060308)
MIME-Version: 1.0
To: dev@geronimo.apache.org
Subject: Re: [Vote: RTC] needed for eclipse plugin
References: <EE5D5295-32AC-4FCE-A496-6D53FE40C8DE@gmail.com>
 <536DCD00-EC72-49F0-A31D-9FF42012C6EA@gmail.com>
In-Reply-To: <536DCD00-EC72-49F0-A31D-9FF42012C6EA@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

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.
>
I will retract my +1, I mistakenly thought that the testing requirement 
was relaxed, but after reviewing the mailing lists I didn't see a 
response to Greg's comments on this topic:

http://marc.theaimsgroup.com/?l=geronimo-dev&m=114850042818334&w=2

Agree that we need to discuss getting the process relaxed officially 
with Ken's approval.

John
> --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
>>
>>
>
>