aries-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jwr...@apache.org
Subject svn commit: r1420357 - /aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java
Date Tue, 11 Dec 2012 19:39:05 GMT
Author: jwross
Date: Tue Dec 11 19:39:04 2012
New Revision: 1420357

URL: http://svn.apache.org/viewvc?rev=1420357&view=rev
Log:
Explicitly uninstall bundles on failure rather than remove from region.

If a subsystem fails to install, content and dependency bundles should be explicitly uninstalled
rather than simply
removing them from the region. Removal from a region does not result in the bundle being uninstalled;
however, uninstalling
a bundle does result in its removal from the region.

Modified:
    aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java

Modified: aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java
URL: http://svn.apache.org/viewvc/aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java?rev=1420357&r1=1420356&r2=1420357&view=diff
==============================================================================
--- aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java
(original)
+++ aries/trunk/subsystem/subsystem-core/src/main/java/org/apache/aries/subsystem/core/internal/BundleResourceInstaller.java
Tue Dec 11 19:39:04 2012
@@ -204,7 +204,7 @@ public class BundleResourceInstaller ext
 			}
 
 			public void failed(Coordination coordination) throws Exception {
-				provisionTo.getRegion().removeBundle(bundle);
+				bundle.uninstall();
 			}
 		});
 		// Set the start level of all bundles managed (i.e. installed) by the



Mime
View raw message