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/02/10 20:48:46 UTC
svn commit: r1729705 -
/maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java
Author: khmarbaise
Date: Wed Feb 10 19:48:46 2016
New Revision: 1729705
URL: http://svn.apache.org/viewvc?rev=1729705&view=rev
Log:
Better use Parameter defaultValue to define default values
instead hard coding them.
Index: src/main/java/org/apache/maven/plugins/rar/RarMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugins/rar/RarMojo.java (revision 1729697)
+++ src/main/java/org/apache/maven/plugins/rar/RarMojo.java (working copy)
@@ -78,8 +78,8 @@
* Specify if the generated jar file of this project should be
* included in the rar file ; default is true.
*/
- @Parameter
- private Boolean includeJar = Boolean.TRUE;
+ @Parameter( defaultValue = "true" )
+ private Boolean includeJar;
/**
* The location of the manifest file to be used within the rar file.
Modified:
maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java
Modified: maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java?rev=1729705&r1=1729704&r2=1729705&view=diff
==============================================================================
--- maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java (original)
+++ maven/plugins/trunk/maven-rar-plugin/src/main/java/org/apache/maven/plugins/rar/RarMojo.java Wed Feb 10 19:48:46 2016
@@ -78,8 +78,8 @@ public class RarMojo
* Specify if the generated jar file of this project should be
* included in the rar file ; default is true.
*/
- @Parameter
- private Boolean includeJar = Boolean.TRUE;
+ @Parameter( defaultValue = "true" )
+ private Boolean includeJar;
/**
* The location of the manifest file to be used within the rar file.