geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vamsic...@apache.org
Subject 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/
Date Fri, 25 Jan 2008 23:06:44 GMT
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 {



Mime
View raw message