+1
On 6/28/06, Matt Hogstrom <[EMAIL PROTECTED]> wrote:
I had the same issue with Jencks noted and a few patches got rejected.
However, after looking and
the rejects I'm not sure why as I simply made the changes in the files and they
were fine.
+1 from me.
David Jencks wrote:
> I have studied the patch, built the branch, and tested it a bit. Aside
> from one small issue about where openejb gets checked out that I fixed
> and that will not be in the final merge, I see no problems.
>
> +1 to completing this merge now.
>
> We will need to add a pom.xml for the new interceptor module. I suggest
> we do this as soon as the merge is complete.
>
> thanks
> david jencks
>
> On Jun 26, 2006, at 6:46 PM, Dain Sundstrom wrote:
>
>> I have finished the work to update the openejb dead-2.2 branch to run
>> with Geronimo 1.2. The dead-2.2 branch contains a major
>> rearchitecture of the OpenEJB container system which split the
>> deployment units from reusable container and this rearchitecture has
>> already been adopted in OpenEJB 3. This work was done in the
>> https://svn.apache.org/repos/asf/geronimo/branches/dain/openejb-2.2-merge
>> branch, and is largely composed of merges from the Geronimo dead-1.2
>> branch.
>>
>> A patch for this change can be found in GERONIMO-12345, but the
>> attached patch is not expected work perfectly. Any mismatches or
>> problems will be worked out during the final merge with trunk. The
>> best way to test this change is to checkout and build the branch
>> directly. The maven m:co goal should work to check out the
>> https://svn.codehaus.org/openejb/branches/dain-2.2-merge/merged branch
>> from openejb. For those with tck access, there is a
>> dain-openejb-2.2-merge branch available for testing the tck.
>>
>> Please, review this quickly so the branch doesn't drift too much.
>>
>> Thanks in advance,
>>
>> -dain
>>
>>
>>
>> A quick history of this branch follows:
>>
>> 417321 Fixed openejb version number in project.properties
>> ------------------------------------------------------------------------
>> 416533 Applied GERONIMO-1960 which verifies GBean references when the
>> final configuration data is created in a deployment context
>> 416483 Fixed deployment plans to match newest openejb changes
>> ------------------------------------------------------------------------
>> 416428 Fixed more plans
>> ------------------------------------------------------------------------
>> 415721 Biggest change which updates the j2ee deployment context to
>> contain the transactionManager name amongst some smaller changes
>> ------------------------------------------------------------------------
>> 415720 Update maven.xml to use new openejb module names
>> ------------------------------------------------------------------------
>> 415225 Set svn:ignore on new interceptor module
>> ------------------------------------------------------------------------
>> 415224 svn merge -r 378403:378404
>> https://svn.apache.org/repos/asf/geronimo/[EMAIL PROTECTED]
>> ------------------------------------------------------------------------
>> 415222 svn merge -r 378358:378359
>> https://svn.apache.org/repos/asf/geronimo/[EMAIL PROTECTED]
>> ------------------------------------------------------------------------
>> 415220 svn merge -r 378346:378347
>> https://svn.apache.org/repos/asf/geronimo/[EMAIL PROTECTED]
>> ------------------------------------------------------------------------
>> 415218 svn merge -r 378182:378183
>> https://svn.apache.org/repos/asf/geronimo/[EMAIL PROTECTED]
>> ------------------------------------------------------------------------
>> 415217 svn merge -r 378161:378162
>> https://svn.apache.org/repos/asf/geronimo/[EMAIL PROTECTED]
>>
>>
>
>
>
>
