stataru8 commented on code in PR #297:
URL: https://github.com/apache/camel-karaf/pull/297#discussion_r1616263380


##########
features/src/main/feature/camel-features.xml:
##########
@@ -2332,8 +2332,8 @@
     </feature>
     <feature name='camel-swift' version='${project.version}' start-level='50'>
         <feature version='${camel.osgi.version.range}'>camel-core</feature>
-        <bundle 
dependency='true'>wrap:mvn:com.prowidesoftware/pw-swift-core/SRU2023-10.1.13</bundle>
-        <bundle 
dependency='true'>wrap:mvn:com.prowidesoftware/pw-iso20022/${prowide-version}</bundle>
+        <bundle 
dependency='true'>wrap:mvn:com.prowidesoftware/pw-swift-core/SRU2023-10.1.13$Bundle-Version=0.0.0.SRU2023-10_1_13</bundle>
+        <bundle 
dependency='true'>wrap:mvn:com.prowidesoftware/pw-iso20022/${prowide-version}$Bundle-Version=0.0.0.SRU2023-10_1_4</bundle>

Review Comment:
   The version coming from the version placeholder doesn't work
   `<prowide-version>SRU2023-10.1.4</prowide-version>`
   ```
   karaf@root()> feature:install camel-swift
   org.osgi.framework.BundleException: Could not create bundle object.
           at org.apache.felix.framework.Felix.installBundle(Felix.java:3339)
           at 
org.apache.felix.framework.BundleContextImpl.installBundle(BundleContextImpl.java:147)
           at 
org.eclipse.equinox.internal.region.BundleIdBasedRegion.installBundle0(BundleIdBasedRegion.java:120)
           at 
org.eclipse.equinox.internal.region.BundleIdBasedRegion.installBundleAtLocation(BundleIdBasedRegion.java:100)
           at 
org.apache.karaf.features.internal.service.BundleInstallSupportImpl.installBundle(BundleInstallSupportImpl.java:135)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.installBundle(FeaturesServiceImpl.java:1145)
           at 
org.apache.karaf.features.internal.service.Deployer.deploy(Deployer.java:906)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.doProvision(FeaturesServiceImpl.java:1069)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.lambda$doProvisionInThread$13(FeaturesServiceImpl.java:1004)
           at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
           at 
java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
           at 
java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
           at java.base/java.lang.Thread.run(Thread.java:833)
   Caused by: java.lang.IllegalArgumentException: invalid version 
"SRU2023-10.1.4": non-numeric "SRU2023-10"
           at org.osgi.framework.Version.parseInt(Version.java:169)
           at org.osgi.framework.Version.<init>(Version.java:126)
           at org.osgi.framework.Version.valueOf(Version.java:255)
           at org.osgi.framework.Version.parseVersion(Version.java:226)
           at 
org.apache.felix.framework.util.manifestparser.ManifestParser.<init>(ManifestParser.java:129)
           at 
org.apache.felix.framework.BundleRevisionImpl.<init>(BundleRevisionImpl.java:117)
           at 
org.apache.felix.framework.BundleImpl.createRevision(BundleImpl.java:1290)
           at org.apache.felix.framework.BundleImpl.<init>(BundleImpl.java:114)
           at org.apache.felix.framework.Felix.installBundle(Felix.java:3287)
           ... 12 more
   Caused by: java.lang.NumberFormatException: For input string: "SRU2023-10"
           at 
java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67)
           at java.base/java.lang.Integer.parseInt(Integer.java:668)
           at java.base/java.lang.Integer.parseInt(Integer.java:786)
           at org.osgi.framework.Version.parseInt(Version.java:167)
           ... 20 more
   ```



##########
features/src/main/feature/camel-features.xml:
##########
@@ -2539,13 +2539,13 @@
         <bundle dependency='true'>mvn:org.jxmpp/jxmpp-jid/0.6.4</bundle>
         <bundle dependency='true'>mvn:org.jxmpp/jxmpp-core/0.6.4</bundle>
         <bundle dependency='true'>mvn:org.jxmpp/jxmpp-util-cache/0.6.4</bundle>
-        <bundle 
dependency='true'>wrap:mvn:org.minidns/minidns-core/${minidns-version}</bundle>
-        <bundle dependency='true'>wrap:mvn:xpp3/xpp3/${xpp3-version}</bundle>
+        <bundle 
dependency='true'>wrap:mvn:org.minidns/minidns-core/${minidns-version}$Bundle-Version=${minidns-version}</bundle>
+        <bundle 
dependency='true'>wrap:mvn:xpp3/xpp3/${xpp3-version}$Bundle-Version=1.1.0.4c</bundle>

Review Comment:
   The version coming from the version placeholder doesn't work
   `<xpp3-version>1.1.4c</xpp3-version>`
   ```
   karaf@root()> feature:install camel-xmpp
   org.osgi.framework.BundleException: Could not create bundle object.
           at org.apache.felix.framework.Felix.installBundle(Felix.java:3339)
           at 
org.apache.felix.framework.BundleContextImpl.installBundle(BundleContextImpl.java:147)
           at 
org.eclipse.equinox.internal.region.BundleIdBasedRegion.installBundle0(BundleIdBasedRegion.java:120)
           at 
org.eclipse.equinox.internal.region.BundleIdBasedRegion.installBundleAtLocation(BundleIdBasedRegion.java:100)
           at 
org.apache.karaf.features.internal.service.BundleInstallSupportImpl.installBundle(BundleInstallSupportImpl.java:135)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.installBundle(FeaturesServiceImpl.java:1145)
           at 
org.apache.karaf.features.internal.service.Deployer.deploy(Deployer.java:906)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.doProvision(FeaturesServiceImpl.java:1069)
           at 
org.apache.karaf.features.internal.service.FeaturesServiceImpl.lambda$doProvisionInThread$13(FeaturesServiceImpl.java:1004)
           at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
           at 
java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
           at 
java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
           at java.base/java.lang.Thread.run(Thread.java:833)
   Caused by: java.lang.IllegalArgumentException: invalid version "1.1.4c": 
non-numeric "4c"
           at org.osgi.framework.Version.parseInt(Version.java:169)
           at org.osgi.framework.Version.<init>(Version.java:134)
           at org.osgi.framework.Version.valueOf(Version.java:255)
           at org.osgi.framework.Version.parseVersion(Version.java:226)
           at 
org.apache.felix.framework.util.manifestparser.ManifestParser.<init>(ManifestParser.java:129)
           at 
org.apache.felix.framework.BundleRevisionImpl.<init>(BundleRevisionImpl.java:117)
           at 
org.apache.felix.framework.BundleImpl.createRevision(BundleImpl.java:1290)
           at org.apache.felix.framework.BundleImpl.<init>(BundleImpl.java:114)
           at org.apache.felix.framework.Felix.installBundle(Felix.java:3287)
           ... 12 more
   Caused by: java.lang.NumberFormatException: For input string: "4c"
           at 
java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67)
           at java.base/java.lang.Integer.parseInt(Integer.java:668)
           at java.base/java.lang.Integer.parseInt(Integer.java:786)
           at org.osgi.framework.Version.parseInt(Version.java:167)
           ... 20 more
   ```
   



-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to