felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject svn commit: r391633 - /incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java
Date Wed, 05 Apr 2006 14:57:51 GMT
Author: marrs
Date: Wed Apr  5 07:57:49 2006
New Revision: 391633

URL: http://svn.apache.org/viewcvs?rev=391633&view=rev
Log:
FELIX-53 Added a check to make sure the dependency that is stopped is actually a required
dependency.

Modified:
    incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java

Modified: incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java
URL: http://svn.apache.org/viewcvs/incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java?rev=391633&r1=391632&r2=391633&view=diff
==============================================================================
--- incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java
(original)
+++ incubator/felix/trunk/org.apache.felix.dependencymanager/src/main/java/org/apache/felix/dependencymanager/ServiceImpl.java
Wed Apr  5 07:57:49 2006
@@ -112,9 +112,11 @@
             // if we're waiting for required dependencies, then
             // we only need to stop tracking the dependency if it
             // too is required; this might trigger a state change
-            dependency.stop(this);
-            if (allRequiredDependenciesAvailable()) {
-                activateService();
+            if (dependency.isRequired()) {
+                dependency.stop(this);
+                if (allRequiredDependenciesAvailable()) {
+                    activateService();
+                }
             }
         }
         return this;



Mime
View raw message