BUG-4219: clean up features.xml 31/49431/4
authorStephen Kitt <skitt@redhat.com>
Thu, 15 Dec 2016 13:40:45 +0000 (14:40 +0100)
committerRobert Varga <nite@hq.sk>
Fri, 16 Dec 2016 09:47:22 +0000 (09:47 +0000)
In models' features.xml, odl-mdsal-binding-base includes yang-ext so
it's not strictly necessary in odl-mdsal-models. (This is potentially
controversial of course since it should perhaps be construed as a
declared part of the odl-mdsal-models API.)

In mdsal's features.xml:
* use odl-lmax-3 instead of odl-lmax;
* drop mdsal-dom-{api,spi} from odl-mdsal-dom-broker (they're part of
  odl-mdsal-dom-api);
* odl-mdsal-binding-api needs odl-yangtools-yang-parser, and
  mdsal-binding-util;
* odl-mdsal-singleton-dom needs odl-mdsal-common.

Change-Id: I4f18d697065f4e9df2b0e621fd32d4b18b8068fd
Signed-off-by: Stephen Kitt <skitt@redhat.com>
common/features/pom.xml
common/features/src/main/features/features.xml
model/features/src/main/features/features.xml

index 31bf8556f9b6185cf68deab087918bb5644f7083..a8c8c0c378835c936d10529bb3d2390c72a7948f 100644 (file)
@@ -22,6 +22,7 @@
     <packaging>jar</packaging>
 
     <properties>
+        <odlparent.version>1.8.0-SNAPSHOT</odlparent.version>
         <yangtools.version>1.1.0-SNAPSHOT</yangtools.version>
     </properties>
 
@@ -30,7 +31,7 @@
             <dependency>
                 <groupId>org.opendaylight.odlparent</groupId>
                 <artifactId>odlparent-artifacts</artifactId>
-                <version>1.8.0-SNAPSHOT</version>
+                <version>${odlparent.version}</version>
                 <scope>import</scope>
                 <type>pom</type>
             </dependency>
index 0e6b88cb32e24e4c2a16299cbe2982bd02203cb8..50b29417b5053cecf5d1c9b36bf936b0d7597560 100644 (file)
 
     <feature name='odl-mdsal-dom-broker' version='${project.version}' description='OpenDaylight :: MD-SAL :: DOM Broker'>
         <feature version='${yangtools.version}' dependency="true">odl-yangtools-yang-parser</feature>
-        <feature version='[3.3.0,4.0.0)' dependency="true">odl-lmax</feature>
+        <feature version="${odlparent.version}" dependency="true">odl-lmax-3</feature>
         <feature version='${project.version}' dependency="true">odl-mdsal-dom-api</feature>
-        <bundle>mvn:org.opendaylight.mdsal/mdsal-dom-api/{{VERSION}}</bundle>
-        <bundle>mvn:org.opendaylight.mdsal/mdsal-dom-spi/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal/mdsal-dom-broker/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal/mdsal-dom-inmemory-datastore/{{VERSION}}</bundle>
     </feature>
     <feature name='odl-mdsal-binding-api' version='${project.version}' description='OpenDaylight :: MD-SAL :: Binding Base Concepts'>
         <feature version='${project.version}' dependency="true">odl-mdsal-common</feature>
         <feature version='${project.version}' dependency="true">odl-mdsal-binding-base</feature>
+        <feature version="${yangtools.version}" dependency="true">odl-yangtools-yang-parser</feature>
         <bundle>mvn:org.opendaylight.mdsal.model/general-entity/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal/mdsal-binding-api/{{VERSION}}</bundle>
+        <bundle>mvn:org.opendaylight.mdsal/mdsal-binding-util/{{VERSION}}</bundle>
     </feature>
 
     <feature name='odl-mdsal-binding-dom-adapter' version='${project.version}' description='OpenDaylight :: MD-SAL :: Binding Generator'>
     <feature name='odl-mdsal-singleton-dom' version='${project.version}' description='OpenDaylight :: MD-SAL :: Singleton :: DOM'>
         <feature version='${project.version}' dependency="true">odl-mdsal-singleton-common</feature>
         <feature version='${project.version}' dependency="true">odl-mdsal-eos-dom</feature>
+        <feature version="${project.version}" dependency="true">odl-mdsal-common</feature>
         <bundle>mvn:org.opendaylight.mdsal/mdsal-singleton-dom-impl/{{VERSION}}</bundle>
     </feature>
 
index 5f7812851fa129b575828ced6bd132abfb2b9a25..3cbc37a58b31de69c250925fb3bb64e072ad7ea1 100644 (file)
@@ -22,7 +22,6 @@
         <bundle>mvn:org.opendaylight.mdsal.model/ietf-yang-types/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal.model/ietf-yang-types-20130715/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal.model/ietf-restconf/{{VERSION}}</bundle>
-        <bundle>mvn:org.opendaylight.mdsal.model/yang-ext/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal.model/opendaylight-l2-types/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal.model/ietf-ted/{{VERSION}}</bundle>
         <bundle>mvn:org.opendaylight.mdsal.model/ietf-topology/{{VERSION}}</bundle>