You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2007/01/15 12:16:24 UTC

svn commit: r496275 - in /incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi: GenerateServiceUnitDescriptorMojo.java JbiServiceUnitDescriptorWriter.java

Author: gnodet
Date: Mon Jan 15 03:16:21 2007
New Revision: 496275

URL: http://svn.apache.org/viewvc?view=rev&rev=496275
Log:
SM-821: missing mandatory binding-component attribute in jbi.xml descriptor for service units
The attribte is currently set to false to be compliant with the xsd, but should be found
by looking at the dependencies ...

Modified:
    incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceUnitDescriptorMojo.java
    incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/JbiServiceUnitDescriptorWriter.java

Modified: incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceUnitDescriptorMojo.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceUnitDescriptorMojo.java?view=diff&rev=496275&r1=496274&r2=496275
==============================================================================
--- incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceUnitDescriptorMojo.java (original)
+++ incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceUnitDescriptorMojo.java Mon Jan 15 03:16:21 2007
@@ -179,7 +179,9 @@
 					"generated : consumes " + consumes + " provides "
 							+ provides);
 
-			writer.write(descriptor, name, description, uris, consumes,
+			boolean bc = false;
+			// TODO: find if the component target is a BC ?
+			writer.write(descriptor, bc, name, description, uris, consumes,
 					provides);
 		} catch (Exception e) {
 			throw new JbiPluginException(

Modified: incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/JbiServiceUnitDescriptorWriter.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/JbiServiceUnitDescriptorWriter.java?view=diff&rev=496275&r1=496274&r2=496275
==============================================================================
--- incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/JbiServiceUnitDescriptorWriter.java (original)
+++ incubator/servicemix/trunk/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/JbiServiceUnitDescriptorWriter.java Mon Jan 15 03:16:21 2007
@@ -41,7 +41,7 @@
 		this.encoding = encoding;
 	}
 
-	public void write(File descriptor, String name, String description,
+	public void write(File descriptor, boolean bc, String name, String description,
 			List uris, List consumes, List provides) throws JbiPluginException {
 		FileWriter w;
 		try {
@@ -57,6 +57,7 @@
 		writer.addAttribute("version", "1.0");
 
 		writer.startElement("services");
+		writer.addAttribute("binding-component", bc ? "true" : "false");
 
 		// We need to get all the namespaces into a hashmap so we
 		// can get the QName output correctly