You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2008/11/14 14:56:05 UTC

svn commit: r714005 - /incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java

Author: bdelacretaz
Date: Fri Nov 14 05:56:05 2008
New Revision: 714005

URL: http://svn.apache.org/viewvc?rev=714005&view=rev
Log:
SLING-731 - reverting change of revision 713758 (except logging which is good) - refreshing packages is not needed in this case, restarting the bundles will happen anyway when processResourceQueue() is called

Modified:
    incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java

Modified: incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java?rev=714005&r1=714004&r2=714005&view=diff
==============================================================================
--- incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java (original)
+++ incubator/sling/trunk/extensions/jcrinstall/src/main/java/org/apache/sling/jcr/jcrinstall/osgi/impl/BundleResourceProcessor.java Fri Nov 14 05:56:05 2008
@@ -394,13 +394,9 @@
                         + ", rescheduling for start"
                         ,be);
 
-                    // add the failed bundle to the activeBundles list
-                    // to start, and trigger a refresh - the exception might
-                    // be caused by a missing dependency that is in the queue,
-                    // so we want to refresh even if no new bundles were added
+                    // re-add the failed bundle to the activeBundles list
                     synchronized (activeBundles) {
                         activeBundles.add(bundleId);
-                        needsRefresh = true;
                     }
                 }
             }