[ http://jira.codehaus.org/browse/MNG-830?page=all ]

allan ramirez updated MNG-830:
------------------------------

    Attachment: MNG-830.patch

>From the clover plugin, I found the usage of expression to be
     /**
      * @parameter expression=""
      */
     private String licenseFile;


Since expression value is null, I removed the expression word to be like this
     /**
      * @parameter
      */
     private String licenseFile;




> review plugins use of expression and change to default-value/component where 
> appropriate
> ----------------------------------------------------------------------------------------
>
>          Key: MNG-830
>          URL: http://jira.codehaus.org/browse/MNG-830
>      Project: Maven 2
>         Type: Improvement
>   Components: maven-plugins
>     Reporter: Brett Porter
>     Assignee: allan ramirez
>      Fix For: 2.0-beta-4
>  Attachments: MNG-830.patch
>
> Original Estimate: 2 hours
>         Remaining: 2 hours
>
> we also need a good explanation written down of the difference.
> * default-value is used when expression is null
> * expression is intended to be used to get the user's override, eg from a 
> system property
> Perhaps if we could cover that other use case differently (eg by a standard 
> system property mechanism that went [plugin-prefix.goal.configuration]), we 
> could eliminate expression altogether in a future version of Maven.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to