Thanks Ivan and Louis,

Apologize. I made a mistake here.   Could you please help correct it for me
?

On Fri, Jan 14, 2011 at 4:56 PM, Jean-Louis MONTEIRO <
[email protected]> wrote:

>
> OK apologize.
> In the future, i will try to also work in Geronimo to avoid that kind of
> regression.
> Thanks again for your patches.
>
> I'm running again a full compilation+tests+itests
> I'd really like to push a new snapshot because, the one available does not
> work due to OWB changes.
>
> Just one question regarding this line in DeploymentResolver
>
> DeploymentLoader deploymentLoader = (ddDir == null) ? new
> DeploymentLoader(ddDir) : new DeploymentLoader(ddDir);
>
> Shouldn't it be
> DeploymentLoader deploymentLoader = (ddDir == null) ? new
> DeploymentLoader()
> : new DeploymentLoader(ddDir);
>
> In order to set ddDir to the default "META-INF/" dir?
> Otherwise the test is not relevant.
>
> Jean-Louis
>
>
> --
> View this message in context:
> http://openejb.979440.n4.nabble.com/New-openejb-3-2-SNAPSHOT-tp3214747p3217307.html
> Sent from the OpenEJB Dev mailing list archive at Nabble.com.
>



-- 
Shawn

Reply via email to