You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2016/05/16 14:56:42 UTC

svn commit: r1744063 - in /maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war: AbstractWarMojo.java WarMojo.java

Author: khmarbaise
Date: Mon May 16 14:56:42 2016
New Revision: 1744063

URL: http://svn.apache.org/viewvc?rev=1744063&view=rev
Log:
[MWAR-376] Remove param properties that doesn't make sense for CLI usage
  o Removed the following properties:
    archiveClasses, resourceEncoding, maven.war.webxml, maven.war.containerConfigXML,
    useCache, includes, excludes, maven.war.filteringDeploymentDescriptors, 
    maven.war.escapedBackslashesInFilePath, maven.war.escapeString, 
    maven.war.supportMultiLineFiltering, maven.war.useJvmChmod, primaryArtifact, 
    failOnMissingWebXml.

Modified:
    maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java
    maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/WarMojo.java

Modified: maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java?rev=1744063&r1=1744062&r2=1744063&view=diff
==============================================================================
--- maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java (original)
+++ maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/AbstractWarMojo.java Mon May 16 14:56:42 2016
@@ -90,7 +90,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.0.1
      */
-    @Parameter( property = "archiveClasses", defaultValue = "false" )
+    @Parameter( defaultValue = "false" )
     private boolean archiveClasses;
 
     /**
@@ -98,7 +98,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.3
      */
-    @Parameter( property = "resourceEncoding", defaultValue = "${project.build.sourceEncoding}" )
+    @Parameter( defaultValue = "${project.build.sourceEncoding}" )
     private String resourceEncoding;
 
     /**
@@ -166,7 +166,7 @@ public abstract class AbstractWarMojo
     /**
      * The path to the web.xml file to use.
      */
-    @Parameter( property = "maven.war.webxml" )
+    @Parameter
     private File webXml;
 
     /**
@@ -174,7 +174,7 @@ public abstract class AbstractWarMojo
      * different servlet containers. Apache Tomcat uses a configuration file named context.xml. The file will be copied
      * to the META-INF directory.
      */
-    @Parameter( property = "maven.war.containerConfigXML" )
+    @Parameter
     private File containerConfigXML;
 
     /**
@@ -206,7 +206,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.1-alpha-1
      */
-    @Parameter( property = "useCache", defaultValue = "false" )
+    @Parameter( defaultValue = "false" )
     private boolean useCache = false;
 
     /**
@@ -233,13 +233,13 @@ public abstract class AbstractWarMojo
     /**
      * The comma separated list of tokens to include when copying the content of the warSourceDirectory.
      */
-    @Parameter( alias = "includes", defaultValue = "**" )
+    @Parameter( defaultValue = "**" )
     private String warSourceIncludes;
 
     /**
      * The comma separated list of tokens to exclude when copying the content of the warSourceDirectory.
      */
-    @Parameter( alias = "excludes" )
+    @Parameter
     private String warSourceExcludes;
 
     /**
@@ -282,7 +282,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.1-alpha-2
      */
-    @Parameter( property = "maven.war.filteringDeploymentDescriptors", defaultValue = "false" )
+    @Parameter( defaultValue = "false" )
     private boolean filteringDeploymentDescriptors = false;
 
     /**
@@ -291,7 +291,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.1-alpha-2
      */
-    @Parameter( property = "maven.war.escapedBackslashesInFilePath", defaultValue = "false" )
+    @Parameter( defaultValue = "false" )
     private boolean escapedBackslashesInFilePath = false;
 
     /**
@@ -300,7 +300,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.1-beta-1
      */
-    @Parameter( property = "maven.war.escapeString" )
+    @Parameter
     protected String escapeString;
 
     /**
@@ -323,7 +323,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.4
      */
-    @Parameter( property = "maven.war.supportMultiLineFiltering", defaultValue = "false" )
+    @Parameter( defaultValue = "false" )
     private boolean supportMultiLineFiltering = false;
 
     /**
@@ -331,7 +331,7 @@ public abstract class AbstractWarMojo
      *
      * @since 2.4
      */
-    @Parameter( property = "maven.war.useJvmChmod", defaultValue = "true" )
+    @Parameter( defaultValue = "true" )
     private boolean useJvmChmod;
 
     /**

Modified: maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/WarMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/WarMojo.java?rev=1744063&r1=1744062&r2=1744063&view=diff
==============================================================================
--- maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/WarMojo.java (original)
+++ maven/plugins/trunk/maven-war-plugin/src/main/java/org/apache/maven/plugins/war/WarMojo.java Mon May 16 14:56:42 2016
@@ -108,7 +108,7 @@ public class WarMojo
      * Whether this is the main artifact being built. Set to <code>false</code> if you don't want to install or deploy
      * it to the local repository instead of the default one in an execution.
      */
-    @Parameter( property = "primaryArtifact", defaultValue = "true" )
+    @Parameter( defaultValue = "true" )
     private boolean primaryArtifact = true;
 
     /**
@@ -118,7 +118,7 @@ public class WarMojo
      *
      * @since 2.1-alpha-2
      */
-    @Parameter( property = "failOnMissingWebXml", defaultValue = "true" )
+    @Parameter( defaultValue = "true" )
     private boolean failOnMissingWebXml = true;
 
     /**