You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ri...@apache.org on 2010/03/23 06:56:00 UTC
svn commit: r926467 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java
Author: rickhall
Date: Tue Mar 23 05:56:00 2010
New Revision: 926467
URL: http://svn.apache.org/viewvc?rev=926467&view=rev
Log:
If we are unable to reload a cached bundle, then we should delete the
entire cached bundle directory, not just the revision. (FELIX-2222)
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java
Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java?rev=926467&r1=926466&r2=926467&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/cache/BundleCache.java Tue Mar 23 05:56:00 2010
@@ -142,9 +142,10 @@ public class BundleCache
}
catch (Exception ex)
{
- // Log and ignore.
+ // Log exception and remove bundle archive directory.
m_logger.log(Logger.LOG_ERROR,
- "Error creating archive.", ex);
+ "Error reloading cached bundle, removing it: " + children[i], ex);
+ deleteDirectoryTree(children[i]);
}
}
}