You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2014/10/30 11:07:03 UTC
svn commit: r1635447 - in /maven/plugins/trunk/maven-toolchains-plugin:
pom.xml
src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java
src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java
Author: hboutemy
Date: Thu Oct 30 10:07:02 2014
New Revision: 1635447
URL: http://svn.apache.org/r1635447
Log:
updated parent pom
Modified:
maven/plugins/trunk/maven-toolchains-plugin/pom.xml
maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java
maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java
Modified: maven/plugins/trunk/maven-toolchains-plugin/pom.xml
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-toolchains-plugin/pom.xml?rev=1635447&r1=1635446&r2=1635447&view=diff
==============================================================================
--- maven/plugins/trunk/maven-toolchains-plugin/pom.xml (original)
+++ maven/plugins/trunk/maven-toolchains-plugin/pom.xml Thu Oct 30 10:07:02 2014
@@ -25,7 +25,7 @@ under the License.
<parent>
<artifactId>maven-plugins</artifactId>
<groupId>org.apache.maven.plugins</groupId>
- <version>25</version>
+ <version>26</version>
<relativePath>../maven-plugins/pom.xml</relativePath>
</parent>
@@ -57,7 +57,6 @@ under the License.
<properties>
<mavenVersion>2.0.9</mavenVersion>
- <mavenPluginVersion>3.2</mavenPluginVersion>
</properties>
<dependencies>
@@ -74,7 +73,6 @@ under the License.
<dependency>
<groupId>org.apache.maven.plugin-tools</groupId>
<artifactId>maven-plugin-annotations</artifactId>
- <version>${mavenPluginVersion}</version>
<scope>provided</scope>
</dependency>
<dependency>
@@ -85,37 +83,6 @@ under the License.
</dependency>
</dependencies>
- <build>
- <pluginManagement>
- <plugins>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-plugin-plugin</artifactId>
- <version>${mavenPluginVersion}</version>
- <configuration>
- <skipErrorNoDescriptorsFound>true</skipErrorNoDescriptorsFound>
- </configuration>
- </plugin>
- </plugins>
- </pluginManagement>
-
- <plugins>
- <plugin>
- <groupId>org.apache.maven.plugins</groupId>
- <artifactId>maven-plugin-plugin</artifactId>
- <executions>
- <execution>
- <id>generate-descriptor</id>
- <goals>
- <goal>descriptor</goal>
- </goals>
- </execution>
- </executions>
- </plugin>
- </plugins>
-
- </build>
-
<profiles>
<profile>
<id>run-its</id>
Modified: maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java?rev=1635447&r1=1635446&r2=1635447&view=diff
==============================================================================
--- maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java (original)
+++ maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainConverter.java Thu Oct 30 10:07:02 2014
@@ -74,20 +74,25 @@ public class ToolchainConverter
{
Map map = new HashMap();
PlexusConfiguration[] tools = configuration.getChildren();
- for (PlexusConfiguration tool : tools) {
+ for ( PlexusConfiguration tool : tools )
+ {
String type = tool.getName();
PlexusConfiguration[] params = tool.getChildren();
Map parameters = new HashMap();
- for (PlexusConfiguration param : params) {
- try {
+ for ( PlexusConfiguration param : params )
+ {
+ try
+ {
String name = param.getName();
String val = param.getValue();
- parameters.put(name, val);
- } catch (PlexusConfigurationException ex) {
- throw new ComponentConfigurationException(ex);
+ parameters.put( name, val );
+ }
+ catch ( PlexusConfigurationException ex )
+ {
+ throw new ComponentConfigurationException( ex );
}
}
- map.put(type, parameters);
+ map.put( type, parameters );
}
chain.toolchains = map;
}
Modified: maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java?rev=1635447&r1=1635446&r2=1635447&view=diff
==============================================================================
--- maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java (original)
+++ maven/plugins/trunk/maven-toolchains-plugin/src/main/java/org/apache/maven/plugin/toolchain/ToolchainMojo.java Thu Oct 30 10:07:02 2014
@@ -83,10 +83,12 @@ public class ToolchainMojo
Map params = toolchains.getParams( type );
ToolchainPrivate[] tcs = getToolchains( type );
boolean matched = false;
- for (ToolchainPrivate tc : tcs) {
- if (tc.matchesRequirements(params)) {
- getLog().info("Toolchain (" + type + ") matched:" + tc);
- toolchainManager.storeToolchainToBuildContext(tc, session);
+ for ( ToolchainPrivate tc : tcs )
+ {
+ if ( tc.matchesRequirements( params ) )
+ {
+ getLog().info( "Toolchain (" + type + ") matched:" + tc );
+ toolchainManager.storeToolchainToBuildContext( tc, session );
matched = true;
break;
}
@@ -103,22 +105,26 @@ public class ToolchainMojo
}
if ( !nonMatchedTypes.isEmpty() )
{
- //TODO add the default toolchain instance if defined??
+ // TODO add the default toolchain instance if defined??
StringBuilder buff = new StringBuilder();
buff.append( "Cannot find matching toolchain definitions for the following toolchain types:" );
- for (Object nonMatchedType : nonMatchedTypes) {
+ for ( Object nonMatchedType : nonMatchedTypes )
+ {
String type = (String) nonMatchedType;
- buff.append('\n');
- buff.append(type);
- Map params = toolchains.getParams(type);
- if (params.size() > 0) {
+ buff.append( '\n' );
+ buff.append( type );
+ Map params = toolchains.getParams( type );
+ if ( params.size() > 0 )
+ {
Iterator it2 = params.keySet().iterator();
- buff.append(" [");
- while (it2.hasNext()) {
+ buff.append( " [" );
+ while ( it2.hasNext() )
+ {
String string = (String) it2.next();
- buff.append(" ").append(string).append("='").append(params.get(string)).append("' ");
+ buff.append( " " ).append( string ).append( "='" ).append( params.get( string ) );
+ buff.append( "' " );
}
- buff.append(']');
+ buff.append( ']' );
}
}
getLog().error( buff.toString() );
@@ -145,14 +151,14 @@ public class ToolchainMojo
Method newMethod =
managerClass.getMethod( "getToolchainsForType", new Class[] { String.class, MavenSession.class } );
- return (ToolchainPrivate[]) newMethod.invoke( toolchainManager, type, session);
+ return (ToolchainPrivate[]) newMethod.invoke( toolchainManager, type, session );
}
catch ( NoSuchMethodException e )
{
// try 2.x style API
Method oldMethod = managerClass.getMethod( "getToolchainsForType", new Class[] { String.class } );
- return (ToolchainPrivate[]) oldMethod.invoke( toolchainManager, type);
+ return (ToolchainPrivate[]) oldMethod.invoke( toolchainManager, type );
}
}
catch ( NoSuchMethodException e )