[ 
https://issues.apache.org/jira/browse/AXIS2-5782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15323567#comment-15323567
 ] 

Hudson commented on AXIS2-5782:
-------------------------------

SUCCESS: Integrated in axis2-1.7 #74 (See 
[https://builds.apache.org/job/axis2-1.7/74/])
AXIS2-5782: Merge r1747575 and r1747578 to the 1.7 branch. (veithen: rev 
1747599)
* axis2
* axis2/modules/distribution/pom.xml
* axis2/modules/tool/axis2-repo-maven-plugin/pom.xml
* axis2/modules/tool/axis2-repo-maven-plugin/src/it
* axis2/modules/tool/axis2-repo-maven-plugin/src/it/AXIS2-5782/pom.xml
* 
axis2/modules/tool/axis2-repo-maven-plugin/src/main/java/org/apache/axis2/maven2/repo/AbstractCreateRepositoryMojo.java
* axis2/pom.xml


> Maven axis2-repo-maven-plugin does not allow module names to be specified on 
> multiple lines.
> --------------------------------------------------------------------------------------------
>
>                 Key: AXIS2-5782
>                 URL: https://issues.apache.org/jira/browse/AXIS2-5782
>             Project: Axis2
>          Issue Type: Improvement
>          Components: Tools
>    Affects Versions: 1.7.3
>            Reporter: Jeff Thomas
>            Priority: Minor
>             Fix For: 1.7.4, 1.8.0
>
>
> Currently the 'axis2-repo-maven-plugin' Maven Plugin does not permit you to 
> place the named modules on more than one line.  If you are configuring a lot 
> of modules, then this leads to a very long line.  Furthermore, if a 
> line-break is added it breaks the plugin.
> {code:xml}
> <plugin>
>   <groupId>org.apache.axis2</groupId>
>   <artifactId>axis2-repo-maven-plugin</artifactId>
>   <version>${axis2.version.pwc}</version>
>   <executions>
>     <execution>
>       <id>axis2-modules-server</id>
>       <phase>generate-resources</phase>
>       <goals><goal>create-repository</goal></goals>
>       <configuration>
>         
> <outputDirectory>${project.build.directory}/webResources/WEB-INF</outputDirectory>
>         <modules>addressing, rampart, soapmonitor, someModuleA, someModuleB, 
> someModuleC, someModuleD</modules>
>       </configuration>
>     </execution>
>   </executions>
> </plugin>
> {code}
> In my patched version, I have added three lines to 
> "AbstractCreateRepositoryMojo.java" to simply remove whitespace between the 
> defined module names at the start of the execution.
> {code:java}
> public void execute() throws MojoExecutionException, MojoFailureException {
>         Set<Artifact> artifacts = new HashSet<Artifact>();
>         if (modules != null) {
>             modules = modules.replaceAll("\\s", "");
>         }
> {code}
> Thus I can specify:
> {code:xml}
>         <modules>
>           addressing, 
>           rampart, 
>           soapmonitor, 
>           someModuleA, 
>           someModuleB, 
>           someModuleC, 
>           someModuleD
>         </modules>
> {code}
> IMHO an even better configuration may be to replace the "modules" String in 
> the MOJO with a "List<String>" for:
> {code:xml}
>         <modules>
>           <module>addressing</module>
>           <module>rampart</module>
>           <module>soapmonitor</module>
>           <module>someModuleA</module>
>           <module>someModuleB</module>
>           <module>someModuleC</module> 
>           <module>someModuleD</module>
>         </modules>
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@axis.apache.org
For additional commands, e-mail: java-dev-h...@axis.apache.org

Reply via email to