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