maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hbout...@apache.org
Subject svn commit: r1339666 - in /maven/plugin-tools/trunk: maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/ maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/ maven-plugin-tools-java/src/site/apt/
Date Thu, 17 May 2012 15:52:19 GMT
Author: hboutemy
Date: Thu May 17 15:52:18 2012
New Revision: 1339666

URL: http://svn.apache.org/viewvc?rev=1339666&view=rev
Log:
removed readonly and required attributes from component configuration, both in java 5 annotations
or javadoc tags documentation, since they are ignored

Modified:
    maven/plugin-tools/trunk/maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/Component.java
    maven/plugin-tools/trunk/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java
    maven/plugin-tools/trunk/maven-plugin-tools-java/src/site/apt/index.apt

Modified: maven/plugin-tools/trunk/maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/Component.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/Component.java?rev=1339666&r1=1339665&r2=1339666&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/Component.java
(original)
+++ maven/plugin-tools/trunk/maven-plugin-annotations/src/main/java/org/apache/maven/plugins/annotations/Component.java
Thu May 17 15:52:18 2012
@@ -49,16 +49,4 @@ public @interface Component
      * @return the role-hint
      */
     String roleHint() default "";
-
-    /**
-     * is the component required?
-     * @return <code>true</code> if the Mojo should fail when the component cannot
be injected
-     */
-    boolean required() default true;
-
-    /**
-     * ignored...
-     * @return
-     */
-    boolean readonly() default true;
 }

Modified: maven/plugin-tools/trunk/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java?rev=1339666&r1=1339665&r2=1339666&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java
(original)
+++ maven/plugin-tools/trunk/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java
Thu May 17 15:52:18 2012
@@ -510,11 +510,11 @@ public class JavaAnnotationsMojoDescript
                 parameter.setName( componentAnnotationContent.getFieldName() );
                 parameter.setRequirement(
                     new Requirement( componentAnnotationContent.role(), componentAnnotationContent.roleHint()
) );
-                parameter.setEditable( false );
                 parameter.setDeprecated( componentAnnotationContent.getDeprecated() );
                 parameter.setSince( componentAnnotationContent.getSince() );
+
                 // same behaviour as JavaMojoDescriptorExtractor
-                //parameter.setRequired( componentAnnotationContent.required() );
+                //parameter.setRequired( ... );
                 parameter.setEditable( false );
                 mojoDescriptor.addParameter( parameter );
             }

Modified: maven/plugin-tools/trunk/maven-plugin-tools-java/src/site/apt/index.apt
URL: http://svn.apache.org/viewvc/maven/plugin-tools/trunk/maven-plugin-tools-java/src/site/apt/index.apt?rev=1339666&r1=1339665&r2=1339666&view=diff
==============================================================================
--- maven/plugin-tools/trunk/maven-plugin-tools-java/src/site/apt/index.apt (original)
+++ maven/plugin-tools/trunk/maven-plugin-tools-java/src/site/apt/index.apt Thu May 17 15:52:18
2012
@@ -72,8 +72,6 @@ public class MyMojo
 
     /**
      * @component role="..." roleHint="..."
-     * @readonly
-     * @required
      * @since <since-text>
      * @deprecated <deprecated-text>
      */
@@ -93,4 +91,4 @@ public class MyMojo
 
  * {{{/developers/mojo-api-specification.html#The_Descriptor_and_Annotations}Mojo API Specification}}
 
- * {{{/ref/current/maven-plugin-api/plugin.html}META-INF/maven/plugin.xml plugin descriptor}}
+ * {{{/ref/current/maven-plugin-api/plugin.html}<<<META-INF/maven/plugin.xml>>>
plugin descriptor}}



Mime
View raw message