You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ja...@apache.org on 2008/05/30 16:46:42 UTC

svn commit: r661741 - /geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java

Author: jawarner
Date: Fri May 30 07:46:42 2008
New Revision: 661741

URL: http://svn.apache.org/viewvc?rev=661741&view=rev
Log:
GERONIMO-4087: Improve usability of gshell commands deploy/* when failing to connect to server

Modified:
    geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java

Modified: geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java?rev=661741&r1=661740&r2=661741&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/factories/BaseDeploymentFactory.java Fri May 30 07:46:42 2008
@@ -180,16 +180,12 @@
         } catch (IOException e) {
             DeploymentManagerCreationException deploymentManagerCreationException = 
                     (DeploymentManagerCreationException) new DeploymentManagerCreationException(e.getMessage()).initCause(e);
-            if (log.isDebugEnabled()) {
-                log.debug("throwing ", deploymentManagerCreationException);
-            }
+            log.debug("throwing ", deploymentManagerCreationException);
             throw deploymentManagerCreationException;
         } catch (SecurityException e) {
             AuthenticationFailedException authenticationFailedException = 
                     (AuthenticationFailedException) new AuthenticationFailedException("Invalid login.").initCause(e);
-            if (log.isDebugEnabled()) {
-                log.debug("throwing ", authenticationFailedException);
-            }
+            log.debug("throwing ", authenticationFailedException);
             throw authenticationFailedException;
         }
     }