SLING-6484 remove workaround as FileVault 3.1.40 fixed the bug with JcrPackage.isInst...
authorKonrad Windszus <kwin@apache.org>
Thu, 29 Jun 2017 13:57:05 +0000 (13:57 +0000)
committerKonrad Windszus <kwin@apache.org>
Thu, 29 Jun 2017 13:57:05 +0000 (13:57 +0000)
git-svn-id: https://svn.apache.org/repos/asf/sling/trunk@1800283 13f79535-47bb-0310-9956-ffa450edef68

pom.xml
src/main/java/org/apache/sling/installer/factory/packages/impl/PackageTransformer.java

diff --git a/pom.xml b/pom.xml
index 8936434..699121c 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -79,7 +79,7 @@
         <dependency>
             <groupId>org.apache.jackrabbit.vault</groupId>
             <artifactId>org.apache.jackrabbit.vault</artifactId>
-            <version>3.1.30</version>
+            <version>3.1.40</version>
             <scope>provided</scope>
         </dependency>
     </dependencies>
index ee49568..b3fabbf 100644 (file)
@@ -215,16 +215,12 @@ public class PackageTransformer implements ResourceTransformer, InstallTaskFacto
                     return;
                 }
 
-                // if this is a SNAPSHOT version we always trigger a reinstall 
-                // (this workaround can be removed once https://issues.apache.org/jira/browse/JCRVLT-155 is implemented)
-                if (!pkgId.getVersionString().endsWith("-SNAPSHOT")) {
-                    // check if package was installed previously by some other means (or even by a previous run of the installer)
-                    if (pkg.isInstalled()) {
-                        String message = MessageFormat.format("Package {0} was installed externally. Marking as installed.", pkgId);
-                        logger.info(message);
-                        this.setFinishedState(ResourceState.INSTALLED, null, message);
-                        return;
-                    }
+                // check if package was installed previously by some other means (or even by a previous run of the installer)
+                if (pkg.isInstalled()) {
+                    String message = MessageFormat.format("Package {0} was installed externally. Marking as installed.", pkgId);
+                    logger.info(message);
+                    this.setFinishedState(ResourceState.INSTALLED, null, message);
+                    return;
                 }
 
                 // check if dependencies are installed