You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2007/09/20 12:57:26 UTC

svn commit: r577680 - /felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java

Author: fmeschbe
Date: Thu Sep 20 03:57:25 2007
New Revision: 577680

URL: http://svn.apache.org/viewvc?rev=577680&view=rev
Log:
FELIX-373 Log the unsatisfied dependencies of a component which prevent activation of the component

Modified:
    felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java

Modified: felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java?rev=577680&r1=577679&r2=577680&view=diff
==============================================================================
--- felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java (original)
+++ felix/trunk/scr/src/main/java/org/apache/felix/scr/AbstractComponentManager.java Thu Sep 20 03:57:25 2007
@@ -330,9 +330,15 @@
             if ( !dm.isSatisfied() )
             {
                 // at least one dependency is not satisfied
-                getActivator().log( LogService.LOG_DEBUG, "Dependency not satisfied: " + dm.getName(),
+                getActivator().log( LogService.LOG_INFO, "Dependency not satisfied: " + dm.getName(),
                     m_componentMetadata, null );
                 setState( STATE_UNSATISFIED );
+            }
+            
+            // if at least one dependency is missing, we cannot continue and
+            // have to return
+            if (getState() == STATE_UNSATISFIED)
+            {
                 return;
             }
         }