You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ga...@apache.org on 2008/01/25 01:28:14 UTC
svn commit: r615086 - in
/geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local:
CommandSupport.java DistributeCommand.java
Author: gawor
Date: Thu Jan 24 16:28:12 2008
New Revision: 615086
URL: http://svn.apache.org/viewvc?rev=615086&view=rev
Log:
make sure to fail on any exception or error (GERONIMO-3767)
Modified:
geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java
geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/DistributeCommand.java
Modified: geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java?rev=615086&r1=615085&r2=615086&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/CommandSupport.java Thu Jan 24 16:28:12 2008
@@ -130,7 +130,7 @@
sendEvent(message, state);
}
- public void doFail(Exception e) {
+ public void doFail(Throwable e) {
if (e instanceof InternalKernelException) {
Exception test = (Exception)e.getCause();
if(test != null) {
Modified: geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/DistributeCommand.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/DistributeCommand.java?rev=615086&r1=615085&r2=615086&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/DistributeCommand.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-deploy-jsr88/src/main/java/org/apache/geronimo/deployment/plugin/local/DistributeCommand.java Thu Jan 24 16:28:12 2008
@@ -60,7 +60,7 @@
for(int i = 0; i < targetList.length; i++) {
doDeploy(targetList[i], true);
}
- } catch (Exception e) {
+ } catch (Throwable e) {
doFail(e);
} finally {
if (spool) {