You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2007/09/07 21:59:56 UTC
svn commit: r573685 - in /geronimo/sandbox/gshell/trunk:
gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java
gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java
Author: jdillon
Date: Fri Sep 7 12:59:55 2007
New Revision: 573685
URL: http://svn.apache.org/viewvc?rev=573685&view=rev
Log:
More minor logging muck, notes about moving error handling to shell
Modified:
geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java
geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java
Modified: geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java?rev=573685&r1=573684&r2=573685&view=diff
==============================================================================
--- geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java (original)
+++ geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandSupport.java Fri Sep 7 12:59:55 2007
@@ -61,6 +61,10 @@
Object result = null;
+ //
+ // TODO: Move this error handling/logging muck up to the shell....
+ //
+
try {
CommandLineProcessor clp = new CommandLineProcessor(this);
clp.process(Arguments.toStringArray(args));
Modified: geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java?rev=573685&r1=573684&r2=573685&view=diff
==============================================================================
--- geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java (original)
+++ geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/ShellImpl.java Fri Sep 7 12:59:55 2007
@@ -132,7 +132,7 @@
try {
result = command.execute(args);
- log.debug("Command completed in: {}", watch);
+ log.debug("Command completed with result: {}, after: {}", result, watch);
}
finally {
// Make sure that the commands output has been flushed