This is an automated email from the ASF dual-hosted git repository.

heneveld pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/brooklyn-server.git

commit c3b1de1139e17ee99a2e74cc8e9d8d31d6f02a0f
Author: Alex Heneveld <[email protected]>
AuthorDate: Tue Nov 9 14:25:09 2021 +0000

    remove vcac jclouds dependency, and bump its yaml version
    
    seems all we need to get a dist that builds
---
 karaf/features/src/main/feature/feature.xml | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/karaf/features/src/main/feature/feature.xml 
b/karaf/features/src/main/feature/feature.xml
index ae14721..65d0468 100644
--- a/karaf/features/src/main/feature/feature.xml
+++ b/karaf/features/src/main/feature/feature.xml
@@ -308,7 +308,6 @@
         <bundle>mvn:org.apache.jclouds/jclouds-core/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds/jclouds-loadbalancer/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds/jclouds-scriptbuilder/${jclouds.version}</bundle>
-        <bundle>mvn:org.apache.jclouds.labs/vcac/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds.provider/aws-ec2/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds.provider/aws-s3/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds.provider/azureblob/${jclouds.version}</bundle>
@@ -333,16 +332,11 @@
         
<bundle>mvn:org.apache.jclouds.provider/skalicloud-sdg-my/${jclouds.version}</bundle>
         
<bundle>mvn:org.apache.jclouds.provider/softlayer/${jclouds.version}</bundle>
 
-        <!-- As at jclouds 2.1.0 this pulls in snakeyaml 1.17 via 
jclouds-api-byon;
-             I cannot see any way to prevent this, short of not using the 
jclouds feature
-             definitions. Things we have tried include redeclaring that 
feature here
-             with our preferred bundle and it works most places but at least 
one runtime
-             (an RPM build of a downstream project installed to Docker) won't 
start as
-             it seems to read the other feature definition.  We also tried the 
etc/overrides.properties
-             but that did not prevent it from pulling in 1.17 at build, or if 
we built correctly,
-             it did not prevent it from needing the 1.17 at runtime. Luckily 
having the two
-             snakeyaml versions doesn't seem to be a problem. -->
-        <bundle dependency='true'>mvn:org.yaml/snakeyaml/1.17</bundle>
+        <bundle dependency='true'>mvn:org.yaml/snakeyaml/1.26</bundle> <!-- 
jclouds uses this version now; we might not need to include it, but trying for 
good measure -->
+
+<!--    exclude vcac; not available in 2.4.0, and not widely used or tested.
+        <bundle>mvn:org.apache.jclouds.labs/vcac/${jclouds.version}</bundle>
+-->
 
         <bundle 
start-level="85">mvn:org.apache.brooklyn/brooklyn-locations-jclouds/${project.version}</bundle>
         <feature>brooklyn-software-winrm</feature>

Reply via email to