[ http://jira.codehaus.org/browse/MPMD-33?page=comments#action_68236 ]
Doug Douglass commented on MPMD-33: ----------------------------------- pmd-3.7 is now available on central. Please consider this patch for inclusion in the pmd plugin. In thinking about this issue some more, it may be worth providiing a configuration option to fail-on-error (the current behavior, should probably be the default) or include errors in the report. If this is desired, I may be able to provide a new patch in a few days. > 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
