Fixup feature packaging

We have a number of upstream references which are not expressed
properly:
- the dependencies should be pulled through artifacts
- the features should use version ranges

This patch corrects all features to behave.

Change-Id: I08c264e09241069d5680e65bc4f49af069e3b177
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/features/odl-lispflowmapping-mappingservice/pom.xml b/features/odl-lispflowmapping-mappingservice/pom.xml
index 6ed7593d6fff8bbf12d7948c675282d2003666ae..33db972da863dc1908d8d62cd72f644d36faadd7 100644 (file)
--- a/features/odl-lispflowmapping-mappingservice/pom.xml
+++ b/features/odl-lispflowmapping-mappingservice/pom.xml
@@ -24,11 +24,22 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <dependencyManagement>
+    <dependencies>
+      <dependency>
+        <groupId>org.opendaylight.netconf</groupId>
+        <artifactId>netconf-artifacts</artifactId>
+        <version>6.0.2</version>
+        <scope>import</scope>
+        <type>pom</type>
+      </dependency>
+    </dependencies>
+  </dependencyManagement>
+
   <dependencies>
     <dependency>
       <groupId>org.opendaylight.netconf</groupId>
       <artifactId>odl-restconf</artifactId>
-      <version>6.0.2</version>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>

diff --git a/features/odl-lispflowmapping-mappingservice/src/main/feature/feature.xml b/features/odl-lispflowmapping-mappingservice/src/main/feature/feature.xml
new file mode 100644 (file)
index 0000000..a323182
--- /dev/null
+++ b/features/odl-lispflowmapping-mappingservice/src/main/feature/feature.xml
@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<!--
+Copyright © 2023 PANTHEON.tech, s.r.o. and others.
+
+This program and the accompanying materials are made available under the
+terms of the Eclipse Public License v1.0 which accompanies this distribution,
+and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
+-->
+<features xmlns="http://karaf.apache.org/xmlns/features/v1.4.0" name="odl-lispflowmapping-mappingservice">
+    <feature name="odl-lispflowmapping-mappingservice">
+        <feature version="[6,7)">odl-restconf</feature>
+    </feature>
+</features>

diff --git a/features/odl-lispflowmapping-southbound/pom.xml b/features/odl-lispflowmapping-southbound/pom.xml
index d0fcb1464c42cda453744442a17f3bb0860813a6..5ff3f9d88e74eb10bd6789670eff93a7b53b4f9b 100644 (file)
--- a/features/odl-lispflowmapping-southbound/pom.xml
+++ b/features/odl-lispflowmapping-southbound/pom.xml
@@ -24,18 +24,35 @@ and is available at http://www.eclipse.org/legal/epl-v10.html
        build failure. Please do not modify this unless you have a good reason. -->
   <name>ODL :: lispflowmapping :: ${project.artifactId}</name>
 
+  <dependencyManagement>
+    <dependencies>
+      <dependency>
+        <groupId>org.opendaylight.controller</groupId>
+        <artifactId>controller-artifacts</artifactId>
+        <version>8.0.1</version>
+        <scope>import</scope>
+        <type>pom</type>
+      </dependency>
+      <dependency>
+        <groupId>org.opendaylight.netconf</groupId>
+        <artifactId>netconf-artifacts</artifactId>
+        <version>6.0.2</version>
+        <scope>import</scope>
+        <type>pom</type>
+      </dependency>
+    </dependencies>
+  </dependencyManagement>
+
   <dependencies>
     <dependency>
-      <groupId>org.opendaylight.netconf</groupId>
-      <artifactId>odl-restconf</artifactId>
-      <version>6.0.2</version>
+      <groupId>org.opendaylight.controller</groupId>
+      <artifactId>odl-mdsal-broker</artifactId>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>
     <dependency>
-      <groupId>org.opendaylight.controller</groupId>
-      <artifactId>odl-mdsal-broker</artifactId>
-      <version>8.0.1</version>
+      <groupId>org.opendaylight.netconf</groupId>
+      <artifactId>odl-restconf</artifactId>
       <type>xml</type>
       <classifier>features</classifier>
     </dependency>

diff --git a/features/odl-lispflowmapping-southbound/src/main/feature/feature.xml b/features/odl-lispflowmapping-southbound/src/main/feature/feature.xml
index 1c00de49a8ddccd960a5d5ec64171ad2832ae3de..c436ceccdaf3f3230a15318ba595017e627301a8 100644 (file)
--- a/features/odl-lispflowmapping-southbound/src/main/feature/feature.xml
+++ b/features/odl-lispflowmapping-southbound/src/main/feature/feature.xml
@@ -10,5 +10,6 @@ and is available at http://www.eclipse.org/legal/epl-v10.html INTERNAL
     <feature name="odl-lispflowmapping-southbound">
         <feature version="[12,14)">odl-netty-4</feature>
         <feature version="[8,9)">odl-mdsal-broker</feature>
+        <feature version="[6,7)">odl-restconf</feature>
     </feature>
 </features>