Author: alessio.soldano(a)jboss.com Date: 2011-12-15 18:20:45 -0500 (Thu, 15 Dec 2011) New Revision: 15399 Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractToolsMojo.java projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractWsConsumeMojo.java projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsConsumeMojo.java projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsProvideMojo.java projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsConsumeMojo.java projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsProvideMojo.java Log: [JBWS-3400] Minor updates Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractToolsMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractToolsMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractToolsMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -110,12 +110,13 @@ } /** - * Return the plugin dependencies that are required to actually call the tools + * Return the plugin dependencies that are required to actually call the tools in fork mode * (jbossws-common-tools and his transitive dependencies getopt and log4j) - //at the *end* of the list, add the required plugin dependency - //that do not come in from the stack dependency tree because: - //1) jbossws-common-tools is not a required dep for stack clients - //2) log4j and getopt need to be provided scope deps, hence are not transitive + * Those dependencies would usually be added at the end of the classpath as they do not come + * in from the stack dependency tree because: + * 1) jbossws-common-tools is not a required dep for stack clients + * 2) log4j and getopt need to be provided scope deps in jbossws-commons-tools, hence they + * are not transitive * * @return a list with the required plugin dependencies */ Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractWsConsumeMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractWsConsumeMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/AbstractWsConsumeMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -27,7 +27,6 @@ import java.util.LinkedList; import java.util.List; -import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.logging.Log; Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsConsumeMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsConsumeMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsConsumeMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -56,8 +56,8 @@ protected List<String> testClasspathElements; /** + * The plugin dependency artifacts * - * * @parameter default-value="${plugin.artifacts}" * @required * @readonly Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsProvideMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsProvideMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/TestWsProvideMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -56,8 +56,8 @@ protected List<String> testClasspathElements; /** + * The plugin dependency artifacts * - * * @parameter default-value="${plugin.artifacts}" * @required * @readonly Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsConsumeMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsConsumeMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsConsumeMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -56,8 +56,8 @@ protected List<String> classpathElements; /** + * The plugin dependency artifacts * - * * @parameter default-value="${plugin.artifacts}" * @required * @readonly Modified: projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsProvideMojo.java =================================================================== --- projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsProvideMojo.java 2011-12-15 23:15:03 UTC (rev 15398) +++ projects/plugins/maven/jaxws-tools/trunk/src/main/java/org/jboss/ws/plugins/tools/WsProvideMojo.java 2011-12-15 23:20:45 UTC (rev 15399) @@ -56,8 +56,8 @@ protected List<String> classpathElements; /** + * The plugin dependency artifacts * - * * @parameter default-value="${plugin.artifacts}" * @required * @readonly