You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Mike Perham (JIRA)" <ji...@codehaus.org> on 2006/06/05 22:34:43 UTC

[jira] Commented: (MPMD-33) do not fail build on PMD failures, include them in pmd report

    [ http://jira.codehaus.org/browse/MPMD-33?page=comments#action_66648 ] 

Mike Perham commented on MPMD-33:
---------------------------------

PMD 3.7 needs to appear in the central repo before this patch can be processed.

> do not fail build on PMD failures, include them in pmd report
> -------------------------------------------------------------
>
>          Key: MPMD-33
>          URL: http://jira.codehaus.org/browse/MPMD-33
>      Project: Maven 2.x Pmd Plugin
>         Type: Improvement

>     Versions: 2.1
>     Reporter: Doug Douglass
>     Priority: Minor
>  Attachments: include-failures.patch
>
>
> When PMD fails to find or successfuly process a file (e.g., throws NPE during processFile method) maven-pmd-plugin fails, which fails the site generation and the build. Instead of trying/having to configure the PMD plugin to either exclude specific files, or to skip execution altogether (in the case of a multi-module build), I think it may be better to include the error in the PMD report, but continue processing files.
> Attached is a patch against trunk that modifies the exception handling behavior to include PMD processing failures in the PMD report. The exceptions are also logged as WARNINGs.
> This patch also updates maven-pmd-plugin to use PMD 3.7.
> See http://www.nabble.com/-m2--NPE-with-maven-pmd-plugin-t1718122.html for some more background on the original problem related to PMD and Jva 5 Generics.

-- 
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