You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2013/06/24 23:43:57 UTC

svn commit: r1496229 - /maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/

Author: rfscholte
Date: Mon Jun 24 21:43:56 2013
New Revision: 1496229

URL: http://svn.apache.org/r1496229
Log:
Deprecate all public fields

Modified:
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractPropertyEnforcerRule.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractStandardEnforcerRule.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractVersionEnforcer.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BannedDependencies.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireActiveProfile.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireEnvironmentVariable.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireNoRepositories.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireOS.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProperty.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseDeps.java
    maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseVersion.java

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractPropertyEnforcerRule.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractPropertyEnforcerRule.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractPropertyEnforcerRule.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractPropertyEnforcerRule.java Mon Jun 24 21:43:56 2013
@@ -35,11 +35,19 @@ public abstract class AbstractPropertyEn
 
     /**
      * Match the property value to a given regular expression. Defaults to <code>null</code> (any value is ok).
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setRegex(String)}
+     * @see {@link #getRegex()}
      */
     public String regex = null;
 
     /**
      * Specify a warning message if the regular expression is not matched.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setRegexMessage(String)}
+     * @see {@link #getRegexMessage()}
      */
     public String regexMessage = null;
 
@@ -48,15 +56,46 @@ public abstract class AbstractPropertyEn
         super();
     }
     
+    /**
+     * Set the property value to a given regular expression. Defaults to <code>null</code> (any value is ok).
+     * 
+     * @param the regular expression
+     */
     public final void setRegex( String regex )
     {
         this.regex = regex;
     }
+
+    /**
+     * Get the property value to a given regular expression. Defaults to <code>null</code> (any value is ok).
+     * 
+     * @return the regular expression
+     */
+    public final String getRegex()
+    {
+        return regex;
+    }
     
+    /**
+     * Set a warning message if the regular expression is not matched.
+     * 
+     * @param regexMessage the regex message
+     */
     public final void setRegexMessage( String regexMessage )
     {
         this.regexMessage = regexMessage;
     }
+    
+    /**
+     * Get a warning message if the regular expression is not matched.
+     * 
+     * @return the regex message
+     */
+    public final String getRegexMessage()
+    {
+        return regexMessage;
+    }
+    
 
     /**
      * Execute the rule.

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractStandardEnforcerRule.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractStandardEnforcerRule.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractStandardEnforcerRule.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractStandardEnforcerRule.java Mon Jun 24 21:43:56 2013
@@ -28,7 +28,13 @@ public abstract class AbstractStandardEn
     implements EnforcerRule
 {
 
-    /** Specify a friendly message if the rule fails. */
+    /** Specify a friendly message if the rule fails.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setMessage(String)}
+     * @see {@link #getMessage()}
+ 
+     */
     public String message = null;
 
     public final void setMessage( String message )

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractVersionEnforcer.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractVersionEnforcer.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractVersionEnforcer.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractVersionEnforcer.java Mon Jun 24 21:43:56 2013
@@ -49,6 +49,11 @@ public abstract class AbstractVersionEnf
      * <li><code>[2.0.5,)</code> Versions 2.0.5 and higher</li>
      * <li><code>(,2.0.5],[2.1.1,)</code> Versions up to 2.0.5 (included) and 2.1.1 or higher</li>
      * </ul>
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setVersion(String)}
+     * @see {@link #getVersion()}
+
      */
     public String version = null;
 

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BannedDependencies.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BannedDependencies.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BannedDependencies.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BannedDependencies.java Mon Jun 24 21:43:56 2013
@@ -45,6 +45,10 @@ public class BannedDependencies
      * Specify the banned dependencies. This can be a list of artifacts in the format <code>groupId[:artifactId][:version]</code>.
      * Any of the sections can be a wildcard by using '*' (ie group:*:1.0) <br>
      * The rule will fail if any dependency matches any exclude, unless it also matches an include rule.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setExcludes(List)}
+     * @see {@link #getExcludes()}
      */
     public List<String> excludes = null;
 
@@ -54,6 +58,10 @@ public class BannedDependencies
      * Includes override the exclude rules. It is meant to allow wide exclusion rules with wildcards and still allow a
      * smaller set of includes. <br>
      * For example, to ban all xerces except xerces-api -> exclude "xerces", include "xerces:xerces-api"
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setIncludes(List)}
+     * @see {@link #getIncludes()}
      */
     public List<String> includes = null;
 

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/EvaluateBeanshell.java Mon Jun 24 21:43:56 2013
@@ -19,6 +19,8 @@ package org.apache.maven.plugins.enforce
  * under the License.
  */
 
+import java.util.List;
+
 import org.apache.maven.enforcer.rule.api.EnforcerRuleException;
 import org.apache.maven.enforcer.rule.api.EnforcerRuleHelper;
 import org.apache.maven.plugin.logging.Log;
@@ -40,7 +42,12 @@ public class EvaluateBeanshell
     /** Beanshell interpreter. */
     private static final Interpreter bsh = new Interpreter();
 
-    /** The condition to be evaluated. */
+    /** The condition to be evaluated.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setCondition(String)}
+     * @see {@link #getCondition()}
+     * */
     public String condition;
 
     public final void setCondition( String condition )

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireActiveProfile.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireActiveProfile.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireActiveProfile.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireActiveProfile.java Mon Jun 24 21:43:56 2013
@@ -38,11 +38,41 @@ public class RequireActiveProfile
     extends AbstractNonCacheableEnforcerRule
 {
 
-    /** Comma separated list of profiles to check. */
+    /** Comma separated list of profiles to check.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setProfiles(String)}
+     * @see {@link #getProfiles()}
+     */
     public String profiles = null;
 
-    /** If all profiles must be active. If false, only one must be active */
+    /** If all profiles must be active. If false, only one must be active
+     *
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAll(boolean)}
+     * @see {@link #isAll()}
+     */
     public boolean all = true;
+    
+    public final String getProfiles()
+    {
+        return profiles;
+    }
+    
+    public final void setProfiles( String profiles )
+    {
+        this.profiles = profiles;
+    }
+    
+    public final boolean isAll()
+    {
+        return all;
+    }
+    
+    public final void setAll( boolean all )
+    {
+        this.all = all;
+    }
 
     /*
      * (non-Javadoc)

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireEnvironmentVariable.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireEnvironmentVariable.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireEnvironmentVariable.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireEnvironmentVariable.java Mon Jun 24 21:43:56 2013
@@ -36,10 +36,22 @@ public class RequireEnvironmentVariable
      */
     public String variableName = null;
 
+    /**
+     * @param variableName the variable name
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setVariableName(String)}
+     * @see {@link #getVariableName()}
+     */
     public final void setVariableName( String variableName )
     {
         this.variableName = variableName;
     }
+    
+    public final String getVariableName()
+    {
+        return variableName;
+    }
 
     @Override
     public String resolveValue( EnforcerRuleHelper helper )

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireNoRepositories.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireNoRepositories.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireNoRepositories.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireNoRepositories.java Mon Jun 24 21:43:56 2013
@@ -47,31 +47,49 @@ public class RequireNoRepositories
 {
     /**
      * Whether to ban non-plugin repositories. By default they are banned.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanRepositories(boolean)}
      */
     public boolean banRepositories = true;
 
     /**
      * Whether to ban plugin repositories. By default they are banned.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanPluginRepositories(boolean)}
      */
     public boolean banPluginRepositories = true;
 
     /**
      * Specify explicitly allowed non-plugin repositories. This is a list of ids.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAllowedRepositories(List)}
      */
     public List<String> allowedRepositories = Collections.emptyList();
 
     /**
      * Specify explicitly allowed plugin repositories. This is a list of ids.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAllowedPluginRepositories(List)}
      */
     public List<String> allowedPluginRepositories = Collections.emptyList();
 
     /**
      * Whether to allow repositories which only resolve snapshots. By default they are banned.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAllowSnapshotRepositories(boolean)}
      */
     public boolean allowSnapshotRepositories = false;
 
     /**
      * Whether to allow plugin repositories which only resolve snapshots. By default they are banned.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAllowSnapshotPluginRepositories(boolean)}
      */
     public boolean allowSnapshotPluginRepositories = false;
 

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireOS.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireOS.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireOS.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireOS.java Mon Jun 24 21:43:56 2013
@@ -58,22 +58,43 @@ public class RequireOS
      * <li>z/os</li>
      * <li>os/400</li>
      * </ul>
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setFamily(String)}
+     * @see {@link #getFamily()}
      */
     public String family = null;
 
-    /** The OS name desired. */
+    /** The OS name desired.
+     *
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setName(String)}
+     * @see {@link #getName()}
+     */
     public String name = null;
 
-    /** The OS version desired. */
+    /** The OS version desired.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setVersion(String)}
+     * @see {@link #getVersion()}
+     */
     public String version = null;
 
-    /** The OS architecture desired. */
+    /** The OS architecture desired.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setArch(String)}
+     * @see {@link #getArch()} 
+     */
     public String arch = null;
 
-    /** Specify an optional message to the user if the rule fails. */
-    public String message = "";
-
-    /** Display detected OS information. */
+    /** Display detected OS information.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setDisplay(boolean)}
+     * @see {@link #isDisplay()}
+     * */
     public boolean display = false;
 
     /**
@@ -104,6 +125,7 @@ public class RequireOS
         {
             if ( !isAllowed() )
             {
+                String message = getMessage();
                 if ( StringUtils.isEmpty( message ) )
                 {
                     message =
@@ -325,6 +347,16 @@ public class RequireOS
     {
         this.version = theVersion;
     }
+    
+    public final void setDisplay( boolean display )
+    {
+        this.display = display;
+    }
+    
+    public final boolean isDisplay()
+    {
+        return display;
+    }
 
     /*
      * (non-Javadoc)

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequirePluginVersions.java Mon Jun 24 21:43:56 2013
@@ -84,21 +84,45 @@ public class RequirePluginVersions
     extends AbstractNonCacheableEnforcerRule
 {
 
-    /** Don't allow the LATEST identifier. */
+    /** Don't allow the LATEST identifier. 
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanLatest(boolean)}
+     * @see {@link #isBanLatest()}
+     */
     public boolean banLatest = true;
 
-    /** Don't allow the RELEASE identifier. */
+    /** Don't allow the RELEASE identifier.
+     *  
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanRelease(boolean)}
+     * @see {@link #isBanRelease()}
+     */
     public boolean banRelease = true;
 
-    /** Don't allow snapshot plugins. */
+    /** Don't allow snapshot plugins. 
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanSnapshots(boolean)}
+     * @see {@link #isBanSnapshots()}
+     */
     public boolean banSnapshots = true;
 
-    /** Don't allow timestamp snapshot plugins. */
+    /** Don't allow timestamp snapshot plugins. 
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setBanTimestamps(boolean)}
+     * @see {@link #isBanTimestamps()}
+     */
     public boolean banTimestamps = true;
 
     /**
      * The comma separated list of phases that should be used to find lifecycle plugin bindings. The default value is
      * "clean,deploy,site".
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setPhases(String)}
+     * @see {@link #getPhases()}
      */
     public String phases = "clean,deploy,site";
 
@@ -106,13 +130,20 @@ public class RequirePluginVersions
      * Additional plugins to enforce have versions. These are plugins that may not be in the poms but are used anyway,
      * like help, eclipse etc. <br>
      * The plugins should be specified in the form: <code>group:artifactId</code>.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setAdditionalPlugins(List)}
+     * @see {@link #getAdditionalPlugins()}
      */
     public List<String> additionalPlugins;
 
     /**
      * Plugins to skip for version enforcement. The plugins should be specified in the form:
      * <code>group:artifactId</code>. NOTE: This is deprecated, use unCheckedPluginList instead.
-     * @deprecated
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setUnCheckedPlugins(List)}
+     * @see {@link #getUnCheckedPlugins()}
      */
     public List unCheckedPlugins;
 
@@ -120,6 +151,10 @@ public class RequirePluginVersions
      * Same as unCheckedPlugins but as a comma list to better support properties. Sample form:
      * <code>group:artifactId,group2:artifactId2</code>
      * @since 1.0-beta-1
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setUnCheckedPlugins(List)}
+     * @see {@link #getUnCheckedPlugins()}
      */
     public String unCheckedPluginList;
 
@@ -1225,4 +1260,24 @@ public class RequirePluginVersions
     {
         this.unCheckedPlugins = unCheckedPlugins;
     }
+    
+    public final void setPhases( String phases )
+    {
+        this.phases = phases;
+    }
+    
+    public final String getPhases()
+    {
+        return phases;
+    }
+    
+    public final void setAdditionalPlugins( List<String> additionalPlugins )
+    {
+        this.additionalPlugins = additionalPlugins;
+    }
+    
+    public final List<String> getAdditionalPlugins()
+    {
+        return additionalPlugins;
+    }
 }
\ No newline at end of file

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProperty.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProperty.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProperty.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireProperty.java Mon Jun 24 21:43:56 2013
@@ -19,6 +19,8 @@ package org.apache.maven.plugins.enforce
  * under the License.
  */
 
+import java.util.List;
+
 import org.apache.maven.enforcer.rule.api.EnforcerRuleException;
 import org.apache.maven.enforcer.rule.api.EnforcerRuleHelper;
 import org.codehaus.plexus.component.configurator.expression.ExpressionEvaluationException;
@@ -32,8 +34,15 @@ public class RequireProperty
     extends AbstractPropertyEnforcerRule
 {
 
-    /** Specify the required property. */
+    /** Specify the required property. 
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setProperty(String)}
+     * @see {@link #getPropertyName()}
+     */
     public String property = null;
+    
+    
     public final void setProperty( String property )
     {
         this.property = property;

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseDeps.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseDeps.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseDeps.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseDeps.java Mon Jun 24 21:43:56 2013
@@ -47,6 +47,11 @@ public class RequireReleaseDeps
      * Allows this rule to execute only when this project is a release.
      *
      * @parameter
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setOnlyWhenRelease(boolean)}
+     * @see {@link #isOnlyWhenRelease()}
+
      */
     public boolean onlyWhenRelease = false;
 
@@ -54,18 +59,30 @@ public class RequireReleaseDeps
      * Allows this rule to fail when the parent is defined as a snapshot.
      *
      * @parameter
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setFailWhenParentIsSnapshot(boolean)}
+     * @see {@link #isFailWhenParentIsSnapshot()}
      */
     public boolean failWhenParentIsSnapshot = true;
 
     /**
      * Dependencies to ignore when checking for release versions.  For example, inter-module dependencies 
      * can be excluded from the check and therefore allowed to contain snapshot versions.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setExcludes(List)}
+     * @see {@link #getExcludes()}
      */
     public List<String> excludes = null;
 
     /**
      * Dependencies to include when checking for release versions.  If any of the included dependencies
      * have snapshot versions, the rule will fail.
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setIncludes(List)}
+     * @see {@link #getIncludes()}
      */
     public List<String> includes = null;
 
@@ -204,4 +221,24 @@ public class RequireReleaseDeps
     {
         this.failWhenParentIsSnapshot = failWhenParentIsSnapshot;
     }
+    
+    public final void setExcludes( List<String> excludes )
+    {
+        this.excludes = excludes;
+    }
+    
+    public final List<String> getExcludes()
+    {
+        return excludes;
+    }
+    
+    public void setIncludes( List<String> includes )
+    {
+        this.includes = includes;
+    }
+    
+    public List<String> getIncludes()
+    {
+        return includes;
+    }
 }

Modified: maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseVersion.java
URL: http://svn.apache.org/viewvc/maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseVersion.java?rev=1496229&r1=1496228&r2=1496229&view=diff
==============================================================================
--- maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseVersion.java (original)
+++ maven/enforcer/trunk/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireReleaseVersion.java Mon Jun 24 21:43:56 2013
@@ -38,6 +38,10 @@ public class RequireReleaseVersion
      * Allows this rule to fail when the parent is defined as a snapshot.
      *
      * @parameter
+     * 
+     * @deprecated the visibility will be reduced to private with the next major version
+     * @see {@link #setFailWhenParentIsSnapshot(boolean)}
+     * @see {@link #isFailWhenParentIsSnapshot()}
      */
     public boolean failWhenParentIsSnapshot = true;