Return-Path: X-Original-To: apmail-maven-commits-archive@www.apache.org Delivered-To: apmail-maven-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 26830175CA for ; Sun, 9 Nov 2014 23:27:53 +0000 (UTC) Received: (qmail 86082 invoked by uid 500); 9 Nov 2014 23:27:52 -0000 Delivered-To: apmail-maven-commits-archive@maven.apache.org Received: (qmail 86022 invoked by uid 500); 9 Nov 2014 23:27:52 -0000 Mailing-List: contact commits-help@maven.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@maven.apache.org Delivered-To: mailing list commits@maven.apache.org Received: (qmail 86013 invoked by uid 99); 9 Nov 2014 23:27:52 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 09 Nov 2014 23:27:52 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 09 Nov 2014 23:27:50 +0000 Received: from eris.apache.org (localhost [127.0.0.1]) by eris.apache.org (Postfix) with ESMTP id 7957123888E4; Sun, 9 Nov 2014 23:26:30 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r1637762 - in /maven/plugin-tools/trunk/maven-plugin-plugin/src: it/plugin-info-jdk-default-property/ it/plugin-info-jdk-default-property/src/ it/plugin-info-jdk-default-property/src/main/ it/plugin-info-jdk-default-property/src/main/java/ ... Date: Sun, 09 Nov 2014 23:26:30 -0000 To: commits@maven.apache.org From: hboutemy@apache.org X-Mailer: svnmailer-1.0.9 Message-Id: <20141109232630.7957123888E4@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: hboutemy Date: Sun Nov 9 23:26:29 2014 New Revision: 1637762 URL: http://svn.apache.org/r1637762 Log: [MPLUGIN-279] use default properties when no configuration in m-compiler-p Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties (with props) maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml (with props) maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java (with props) maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml (with props) maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy (with props) Modified: maven/plugin-tools/trunk/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/PluginReport.java Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties?rev=1637762&view=auto ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties (added) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties Sun Nov 9 23:26:29 2014 @@ -0,0 +1,18 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +invoker.goals = clean site -DskipTests Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties ------------------------------------------------------------------------------ svn:keywords = Author Date Id Revision Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/invoker.properties ------------------------------------------------------------------------------ svn:mime-type = text/plain Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml?rev=1637762&view=auto ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml (added) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml Sun Nov 9 23:26:29 2014 @@ -0,0 +1,87 @@ + + + + + + 4.0.0 + + org.apache.maven.its + jdk-default-property + 1.0-SNAPSHOT + maven-plugin + + + UTF-8 + 1.5 + ${maven.compiler.source} + + + + + org.apache.maven + maven-plugin-api + 2.0 + + + + + + + + org.apache.maven.plugins + maven-compiler-plugin + 2.3.2 + + + + org.apache.maven.plugins + maven-plugin-plugin + @project.version@ + + + org.apache.maven.plugins + maven-site-plugin + @sitePluginVersion@ + + + + + + + + + org.apache.maven.plugins + maven-project-info-reports-plugin + 2.4 + + + org.apache.maven.plugins + maven-plugin-plugin + @project.version@ + + + + Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml ------------------------------------------------------------------------------ svn:keywords = Author Date Id Revision Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/pom.xml ------------------------------------------------------------------------------ svn:mime-type = text/plain Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java?rev=1637762&view=auto ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java (added) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java Sun Nov 9 23:26:29 2014 @@ -0,0 +1,81 @@ +package org.apache.maven.plugins.issues.plugin; + +/* + * Copyright 2001-2005 The Apache Software Foundation. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +import org.apache.maven.plugin.AbstractMojo; +import org.apache.maven.plugin.MojoExecutionException; + +import java.io.File; +import java.io.FileWriter; +import java.io.IOException; + +/** + * Goal which touches a timestamp file. + * + * @goal touch + * + * @phase process-sources + */ +public class MyMojo + extends AbstractMojo +{ + /** + * Location of the file. + * @parameter expression="${project.build.directory}" + * @required + */ + private File outputDirectory; + + public void execute() + throws MojoExecutionException + { + File f = outputDirectory; + + if ( !f.exists() ) + { + f.mkdirs(); + } + + File touch = new File( f, "touch.txt" ); + + FileWriter w = null; + try + { + w = new FileWriter( touch ); + + w.write( "touch.txt" ); + } + catch ( IOException e ) + { + throw new MojoExecutionException( "Error creating file " + touch, e ); + } + finally + { + if ( w != null ) + { + try + { + w.close(); + } + catch ( IOException e ) + { + // ignore + } + } + } + } +} Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java ------------------------------------------------------------------------------ svn:keywords = Author Date Id Revision Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/main/java/org/apache/maven/plugins/issues/plugin/MyMojo.java ------------------------------------------------------------------------------ svn:mime-type = text/plain Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml?rev=1637762&view=auto ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml (added) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml Sun Nov 9 23:26:29 2014 @@ -0,0 +1,30 @@ + + + + + + + + + + + + Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml ------------------------------------------------------------------------------ svn:keywords = Author Date Id Revision Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/src/site/site.xml ------------------------------------------------------------------------------ svn:mime-type = text/plain Added: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy?rev=1637762&view=auto ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy (added) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy Sun Nov 9 23:26:29 2014 @@ -0,0 +1,27 @@ + +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ +assert new File( basedir, 'target/site' ).exists(); + +content = new File( basedir, 'target/site/plugin-info.html' ).text; + +assert content.contains( '1.5' ); +assert !content.contains( 'Default target for maven-compiler-plugin version' ); + +return true; \ No newline at end of file Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/plugin-tools/trunk/maven-plugin-plugin/src/it/plugin-info-jdk-default-property/verify.groovy ------------------------------------------------------------------------------ svn:keywords = Author Date Id Revision Modified: maven/plugin-tools/trunk/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/PluginReport.java URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/PluginReport.java?rev=1637762&r1=1637761&r2=1637762&view=diff ============================================================================== --- maven/plugin-tools/trunk/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/PluginReport.java (original) +++ maven/plugin-tools/trunk/maven-plugin-plugin/src/main/java/org/apache/maven/plugin/plugin/PluginReport.java Sun Nov 9 23:26:29 2014 @@ -54,6 +54,7 @@ import java.util.Iterator; import java.util.List; import java.util.Locale; import java.util.Map; +import java.util.Properties; import java.util.ResourceBundle; import java.util.Set; @@ -667,11 +668,13 @@ public class PluginReport String jdk = requirements.getJdk(); if ( jdk == null ) { - jdk = discoverJdkRequirementFromPlugins( project.getBuild().getPluginsAsMap() ); + jdk = discoverJdkRequirementFromPlugins( project.getBuild().getPluginsAsMap(), project.getProperties() ); } if ( jdk == null && project.getPluginManagement() != null ) { - jdk = discoverJdkRequirementFromPlugins( project.getPluginManagement().getPluginsAsMap() ); + jdk = + discoverJdkRequirementFromPlugins( project.getPluginManagement().getPluginsAsMap(), + project.getProperties() ); } if ( jdk == null ) { @@ -685,7 +688,7 @@ public class PluginReport * @param pluginsAsMap could be null * @return the value of the target in the configuration of maven-compiler-plugin. */ - private static String discoverJdkRequirementFromPlugins( Map pluginsAsMap ) + private static String discoverJdkRequirementFromPlugins( Map pluginsAsMap, Properties props ) { if ( pluginsAsMap == null ) { @@ -704,6 +707,9 @@ public class PluginReport Object value = entry.getValue(); Xpp3Dom pluginConf = null; + // default value + jdk = props.getProperty( "maven.compiler.target" ); + backupJdk = "Default version for maven-compiler-plugin"; if ( value instanceof Plugin ) {