You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dw...@apache.org on 2008/06/04 19:03:50 UTC

svn commit: r663305 - in /geronimo/server/branches/2.1: assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java

Author: dwoods
Date: Wed Jun  4 10:03:50 2008
New Revision: 663305

URL: http://svn.apache.org/viewvc?rev=663305&view=rev
Log:
GERONIMO-4016 The exception of failing to start client is not recorded in client.log.  Thanks Yun Feng for the patch.

Modified:
    geronimo/server/branches/2.1/assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties
    geronimo/server/branches/2.1/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java

Modified: geronimo/server/branches/2.1/assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties?rev=663305&r1=663304&r2=663305&view=diff
==============================================================================
--- geronimo/server/branches/2.1/assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties (original)
+++ geronimo/server/branches/2.1/assemblies/geronimo-boilerplate-minimal/src/main/underlay/var/log/client-log4j.properties Wed Jun  4 10:03:50 2008
@@ -20,7 +20,7 @@
 log4j.rootLogger=INFO, CONSOLE, FILE
 
 log4j.appender.CONSOLE=org.apache.log4j.ConsoleAppender
-log4j.appender.CONSOLE.Threshold=INFO
+log4j.appender.CONSOLE.Threshold=${org.apache.geronimo.log.ConsoleLogLevel}
 log4j.appender.CONSOLE.Target=System.out
 log4j.appender.CONSOLE.layout=org.apache.log4j.PatternLayout
 log4j.appender.CONSOLE.layout.ConversionPattern=%d{ABSOLUTE} %-5p [%c{1}] %m%n

Modified: geronimo/server/branches/2.1/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java?rev=663305&r1=663304&r2=663305&view=diff
==============================================================================
--- geronimo/server/branches/2.1/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java (original)
+++ geronimo/server/branches/2.1/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/main/ClientCommandLine.java Wed Jun  4 10:03:50 2008
@@ -63,7 +63,7 @@
             return startClient(configuration, parser.getApplicationClientArgs());
         } catch (Exception e) {
             ExceptionUtil.trimStackTrace(e);
-            e.printStackTrace();
+            log.error("Client failed with exception: ", e);
             return 2;
         }
     }