From: Robert Varga <rovarga@cisco.com>
Date: Sun, 31 Jul 2016 20:42:37 +0000 (+0200)
Subject: Move generated sources to target/ directory
X-Git-Tag: release/boron~34
X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=commitdiff_plain;h=1a306f228ccbdbb6c5d302a1f8e24e5f4de4f0e0

Move generated sources to target/ directory

Generating bindings into src/ is a very bad idea, as that:
- does not conform the expectations
- does not get cleaned by maven
- makes the life for bindings hard (checkstyle, cpd, etc.)

Change-Id: I0d594aa849934c4ac88f8ebb0d4dcc8ea5a4e3e6
Signed-off-by: Robert Varga <rovarga@cisco.com>
---

diff --git a/opendaylight/commons/opendaylight/pom.xml b/opendaylight/commons/opendaylight/pom.xml
index 98e957e38d..16eb5cbb9e 100644
--- a/opendaylight/commons/opendaylight/pom.xml
+++ b/opendaylight/commons/opendaylight/pom.xml
@@ -85,7 +85,10 @@
     <ietf-topology.version>2013.10.21.9-SNAPSHOT</ietf-topology.version>
     <ietf-yang-types.version>2013.07.15.9-SNAPSHOT</ietf-yang-types.version>
     <jdepend.maven.plugin.version>2.0-beta-2</jdepend.maven.plugin.version>
+
+    <!-- Path prefix where we generate configuration subsystem bindings -->
     <jmxGeneratorPath>target/generated-sources/config-binding</jmxGeneratorPath>
+
     <jolokia-bridge.version>0.3.0-SNAPSHOT</jolokia-bridge.version>
     <karaf.branding.version>1.3.0-SNAPSHOT</karaf.branding.version>
     <karaf.shell.version>${karaf.version}</karaf.shell.version>
@@ -110,7 +113,11 @@
     <sal.implementation.version>0.7.0-SNAPSHOT</sal.implementation.version>
     <sal.networkconfiguration.version>0.3.0-SNAPSHOT</sal.networkconfiguration.version>
     <sal.version>0.11.0-SNAPSHOT</sal.version>
-    <salGeneratorPath>src/main/yang-gen-sal</salGeneratorPath>
+
+    <!-- Path prefix where we generate YANG Java Bindings. This really should be inherited
+         from mdsal's binding-parent -->
+    <salGeneratorPath>target/generated-sources/mdsal-binding</salGeneratorPath>
+
     <samples.loadbalancer>0.8.0-SNAPSHOT</samples.loadbalancer>
     <samples.loadbalancer.northbound.version>0.7.0-SNAPSHOT</samples.loadbalancer.northbound.version>
     <samples.simpleforwarding.version>0.7.0-SNAPSHOT</samples.simpleforwarding.version>