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 ammulder@gmail.com designates
 72.14.214.201 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:sender:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references:x-google-sender-auth;
        b=RxTs88xRNVH2Na+AJvH/XuR5FKyuH1DKiFwUmxWWNAbIIlub8OMoi0FWVLbYGzClEXOa3o2VYQNVM+YsYi4hw8E7KApa8UkdIJKZcP0YK9sh0wpL8zqOKT+tGAw3qCbWDMub4vaTBkNDc4004bkqqsCviH7UXlPjyc48Odf61RQ=
Message-ID: <74e15baa0606280028gdd59cd2had41adb8105a796c@mail.gmail.com>
Date: Wed, 28 Jun 2006 03:28:12 -0400
From: "Aaron Mulder" <ammulder@alumni.princeton.edu>
Sender: ammulder@gmail.com
To: dev@geronimo.apache.org
Subject: Re: [RTC] Update for new OpenEJB 2.2
In-Reply-To: <44A22042.1060008@hogstrom.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <14F7517A-68F0-468E-A30F-6FCEF1D5691B@iq80.com>
	 <D384BD6C-C6E0-445F-9585-AD81B1858C1E@yahoo.com>
	 <44A22042.1060008@hogstrom.org>

+1

On 6/28/06, Matt Hogstrom <matt@hogstrom.org> 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/trunk@378404
> >> ------------------------------------------------------------------------
> >> 415222 svn merge -r 378358:378359
> >> https://svn.apache.org/repos/asf/geronimo/trunk@378359
> >> ------------------------------------------------------------------------
> >> 415220 svn merge -r 378346:378347
> >> https://svn.apache.org/repos/asf/geronimo/trunk@378347
> >> ------------------------------------------------------------------------
> >> 415218 svn merge -r 378182:378183
> >> https://svn.apache.org/repos/asf/geronimo/trunk@378183
> >> ------------------------------------------------------------------------
> >> 415217 svn merge -r 378161:378162
> >> https://svn.apache.org/repos/asf/geronimo/trunk@378162
> >>
> >>
> >
> >
> >
> >
>