geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r1227940 - /geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java
Date Fri, 06 Jan 2012 02:39:01 GMT
Author: djencks
Date: Fri Jan  6 02:39:00 2012
New Revision: 1227940

URL: http://svn.apache.org/viewvc?rev=1227940&view=rev
Log:
GERONIMO-6249 make sure existing bundle is at least starting before trying to activate it

Modified:
    geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java

Modified: geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java?rev=1227940&r1=1227939&r2=1227940&view=diff
==============================================================================
--- geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java
(original)
+++ geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java
Fri Jan  6 02:39:00 2012
@@ -192,7 +192,10 @@ public class DependencyManager {
                 if (bundleEvent == null) {
                     // existing bundles first added to the tracker with no event change
                     installed(bundle);
-                    start(bundle);
+                    //make sure existing bundle is at least starting
+                    if (bundle.getBundleContext() != null) {
+                        start(bundle);
+                    }
                 } else {
                     bundleChanged(bundleEvent);
                 }



Mime
View raw message