kwin commented on a change in pull request #6: SLING-9172 - adding support for property sling.installer.multiversion… URL: https://github.com/apache/sling-org-apache-sling-installer-core/pull/6#discussion_r388294116
########## File path: src/main/java/org/apache/sling/installer/core/impl/tasks/BundleTaskCreator.java ########## @@ -71,14 +71,17 @@ private BundleBlackList bundleBlacklist; + private boolean isMultiVersion; + /** * @see org.apache.sling.installer.core.impl.InternalService#init(org.osgi.framework.BundleContext, org.apache.sling.installer.api.ResourceChangeListener, RetryHandler) */ @Override public void init(final BundleContext bc, final ResourceChangeListener listener, final RetryHandler retryHandler) { this.bundleContext = bc; this.retryHandler = retryHandler; - + + this.isMultiVersion = Boolean.valueOf(bc.getProperty("sling.installer.multiversion")) ? true : false; Review comment: > the statehandling falls appart and you eventually end up with ignored resources where you can't recover the installer state without starting over from scratch And this PR fixes that? Can we have a test for that? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services