Cool. I've updated the Geronimo side as follows:
Index: src/main/java/org/apache/geronimo/openejb/OpenEjbSystemGBean.java
===================================================================
--- src/main/java/org/apache/geronimo/openejb/
OpenEjbSystemGBean.java (revision 570242)
+++ src/main/java/org/apache/geronimo/openejb/
OpenEjbSystemGBean.java (working copy)
@@ -104,7 +104,8 @@
}
System.setProperty("duct tape","");
System.setProperty("admin.disabled", "true");
-
+ System.setProperty("openejb.logger.external", "true");
+
setDefaultProperty("openejb.deploymentId.format",
"{moduleId}/{ejbName}");
setDefaultProperty("openejb.jndiname.strategy.class",
"org.apache.openejb.assembler.classic.JndiBuilder$TemplatedStrategy");
setDefaultProperty("openejb.jndiname.format",
"{deploymentId}/{interfaceClass}");
On Sep 6, 2007, at 1:52 PM, Karan Malhi wrote:
Yes it should work.
On 9/6/07, David Blevins <[EMAIL PROTECTED]> wrote:
On Sep 6, 2007, at 9:04 AM, Karan Malhi wrote:
This is because Geronimo also uses the same classes as we use in
embedded mode. So it will not be possible to set the SystemProperty
there. For example, OpenEjbSystemGBean also uses SystemInstance, but
our code also uses SystemInstance heavily. So setting it in
SystemInstance will not lead to the desired behaviour.
We can however try the following options:
1. Ask Geronimo team to set this property in
org.apache.geronimo.openejb.OpenEjbSystemGBean
Not sure I followed the explanation above. Ideally we'd be able to
use option 1 here as that's what we do for other properties.
Setting openejb.logging.external=true as a plain system property in
OpenEjbSystemGBean should work right?
-David
--
Karan Singh Malhi
