Mailing-List: contact issues-help@karaf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@karaf.apache.org
Date: Wed, 12 Mar 2014 12:34:43 +0000 (UTC)
From: "Igor Lazebny (JIRA)" <jira@apache.org>
To: issues@karaf.apache.org
Message-ID: <JIRA.12700950.1394627579080.50940.1394627683648@arcas>
In-Reply-To: <JIRA.12700950.1394627579080@arcas>
References: <JIRA.12700950.1394627579080@arcas>
Subject: [jira] [Created] (KARAF-2820)
 features-maven-plugin:add-features-to-repo adds wrong transitive feature
 version
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Igor Lazebny created KARAF-2820:
-----------------------------------

             Summary: features-maven-plugin:add-features-to-repo adds wrong transitive feature version
                 Key: KARAF-2820
                 URL: https://issues.apache.org/jira/browse/KARAF-2820
             Project: Karaf
          Issue Type: Bug
          Components: karaf-tooling
    Affects Versions: 2.3.4
            Reporter: Igor Lazebny


features-maven-plugin:add-features-to-repo tries to find greatest version of transitive feature to add. But the problem is it compares versions as string. So, for example, "2.7.7" > "2.7.10".
Here is that part in tooling/features-maven-plugin/src/main/java/org/apache/karaf/tooling/features/AddFeaturesToRepoMojo.java

 313                 // looking for the feature name (with the greatest version)
314                 for (String key : featuresMap.keySet()) {
315                     String[] nameVersion = key.split("/");
316                     if (feature.equals(nameVersion[0])) {
317                         if (f == null || f.getVersion().compareTo(featuresMap.get(key).getVersion()) < 0) {
318                             f = featuresMap.get(key);
319                         }
320                     }
321                 }

I have an example, which adds 2 features: cxf (2.7.10) and camel (2.12.2). Expexted result: 2.7.10 CXF bundles in target/features-repo. Actual result: 2.7.7 CXF bundles - as transitive through Camel features descriptor.


--
This message was sent by Atlassian JIRA
(v6.2#6252)