You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2011/11/14 14:23:18 UTC
svn commit: r1201694 -
/sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java
Author: cziegeler
Date: Mon Nov 14 13:23:18 2011
New Revision: 1201694
URL: http://svn.apache.org/viewvc?rev=1201694&view=rev
Log:
SLING-2043 : Upon the installation of a fragment bundle, a refreshPackages call should be made on the host bundle
Modified:
sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java
Modified: sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java
URL: http://svn.apache.org/viewvc/sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java?rev=1201694&r1=1201693&r2=1201694&view=diff
==============================================================================
--- sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java (original)
+++ sling/trunk/installer/core/src/main/java/org/apache/sling/installer/core/impl/tasks/SynchronousRefreshPackagesTask.java Mon Nov 14 13:23:18 2011
@@ -82,9 +82,10 @@ public class SynchronousRefreshPackagesT
// Refreshing packages might cause some bundles to be stopped,
// make sure all currently active ones are restarted after
- // this task executes
+ // this task has executed
+ // we don't check for fragments!
for(final Bundle b : this.bundleTaskCreator.getBundleContext().getBundles()) {
- if ( BundleStartTask.isBundleActive(b) ) {
+ if ( BundleStartTask.isBundleActive(b) && BundleStartTask.getFragmentHostHeader(b) == null ) {
final InstallTask t = new BundleStartTask(null, b.getBundleId(), this.bundleTaskCreator);
ctx.addTaskToCurrentCycle(t);
this.getLogger().debug("Added {} to restart bundle if needed after refreshing packages", t);