You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by va...@apache.org on 2008/01/26 00:06:44 UTC
svn commit: r615389 - in /geronimo/server:
branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/
trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/
Author: vamsic007
Date: Fri Jan 25 15:06:43 2008
New Revision: 615389
URL: http://svn.apache.org/viewvc?rev=615389&view=rev
Log:
GERONIMO-3764 Deployer fails to cleanup the temp files created during deployment process
o NestedJarFile should close the baseJar irrespective of whether it is packed or not. In case of inPlace deployment of an ear file, not closing baseJar is resulting in a lock on DUMMY_JAR_FILE.
Modified:
geronimo/server/branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java
geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java
Modified: geronimo/server/branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java?rev=615389&r1=615388&r2=615389&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java Fri Jan 25 15:06:43 2008
@@ -231,7 +231,7 @@
super.close();
} catch(IOException ignored) {
}
- if (baseJar != null && isPacked()) {
+ if (baseJar != null) {
baseJar.close();
}
} finally {
Modified: geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java?rev=615389&r1=615388&r2=615389&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java (original)
+++ geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/util/NestedJarFile.java Fri Jan 25 15:06:43 2008
@@ -231,7 +231,7 @@
super.close();
} catch(IOException ignored) {
}
- if (baseJar != null && isPacked()) {
+ if (baseJar != null) {
baseJar.close();
}
} finally {