You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by pd...@apache.org on 2016/12/04 23:16:38 UTC

svn commit: r1772583 - /felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java

Author: pderop
Date: Sun Dec  4 23:16:38 2016
New Revision: 1772583

URL: http://svn.apache.org/viewvc?rev=1772583&view=rev
Log:
FELIX-5432: Message when dependency is already added to a component is incomplete. Applied the fix suggested by Jeroen (thanks !).

Modified:
    felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java

Modified: felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java?rev=1772583&r1=1772582&r2=1772583&view=diff
==============================================================================
--- felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java (original)
+++ felix/trunk/dependencymanager/org.apache.felix.dependencymanager/src/org/apache/felix/dm/impl/ComponentImpl.java Sun Dec  4 23:16:38 2016
@@ -371,7 +371,7 @@ public class ComponentImpl implements Co
             for (Dependency d : dependencies) {
                 DependencyContext dc = (DependencyContext) d;
                 if (dc.getComponentContext() != null) {
-                    m_logger.err("%s can't be added to %s (dependency already added to another component).", dc, ComponentImpl.this);
+                    m_logger.err("%s can't be added to %s (dependency already added to component %s).", dc, ComponentImpl.this, dc.getComponentContext());
                     continue;
                 }
                 m_dependencyEvents.put(dc, new ConcurrentSkipListSet<Event>());