You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by js...@apache.org on 2007/11/19 12:04:06 UTC

svn commit: r596266 - /activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java

Author: jstrachan
Date: Mon Nov 19 03:04:04 2007
New Revision: 596266

URL: http://svn.apache.org/viewvc?rev=596266&view=rev
Log:
applied patch for https://issues.apache.org/activemq/browse/CAMEL-232 with thanks!

Modified:
    activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java

Modified: activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java?rev=596266&r1=596265&r2=596266&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java (original)
+++ activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java Mon Nov 19 03:04:04 2007
@@ -91,7 +91,11 @@
         for (int i = 0; i < previousBundles.length; i++) {
             int state = previousBundles[i].getState();
             if (state == Bundle.RESOLVED || state == Bundle.ACTIVE) {
-                mayBeAddComponentFor(previousBundles[i]);
+            	try {
+            		mayBeAddComponentFor(previousBundles[i]);
+            	} catch (Exception e) {
+            		LOG.error("Component " + previousBundles[i] + " not added due to " + e.toString(), e);
+            	}
             }
         }
     }



Re: svn commit: r596266 - /activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java

Posted by James Strachan <js...@apache.org>.
Cut n paste mistake - this patch was for

https://issues.apache.org/activemq/browse/CAMEL-233

On 19/11/2007, jstrachan@apache.org <js...@apache.org> wrote:
> Author: jstrachan
> Date: Mon Nov 19 03:04:04 2007
> New Revision: 596266
>
> URL: http://svn.apache.org/viewvc?rev=596266&view=rev
> Log:
> applied patch for https://issues.apache.org/activemq/browse/CAMEL-232 with thanks!
>
> Modified:
>     activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java
>
> Modified: activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java
> URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java?rev=596266&r1=596265&r2=596266&view=diff
> ==============================================================================
> --- activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java (original)
> +++ activemq/camel/trunk/components/camel-osgi/src/main/java/org/apache/camel/osgi/OsgiComponentResolver.java Mon Nov 19 03:04:04 2007
> @@ -91,7 +91,11 @@
>          for (int i = 0; i < previousBundles.length; i++) {
>              int state = previousBundles[i].getState();
>              if (state == Bundle.RESOLVED || state == Bundle.ACTIVE) {
> -                mayBeAddComponentFor(previousBundles[i]);
> +               try {
> +                       mayBeAddComponentFor(previousBundles[i]);
> +               } catch (Exception e) {
> +                       LOG.error("Component " + previousBundles[i] + " not added due to " + e.toString(), e);
> +               }
>              }
>          }
>      }
>
>
>


-- 
James
-------
http://macstrac.blogspot.com/

Open Source Integration
http://open.iona.com