Maven 1.x Site Plugin

a way to ignore report failures

Details

  • Type: Wish Wish
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.6.1
  • Fix Version/s: 1.7
  • Component/s: plugin
  • Labels:
    None
  • Number of attachments :
    3

Description

It would be great if there was a way to ignore report failures. Something like a proprety called "maven.site.reports.failonerror".

Attachments

  1. Text File
    MPSITE-50.patch
    04/Apr/06 12:25 PM
    1 kB
    Shinobu Kawai
  2. Text File
    MPSITE-50.patch
    03/Apr/06 5:18 PM
    1 kB
    Shinobu Kawai
  3. Text File
    MPSITE-50-with-xdocs.patch
    05/Apr/06 11:13 AM
    2 kB
    Shinobu Kawai

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Lukas Theussl added a comment -

This can only be implemented in the corresponding reports, eg the pmd plugin has a maven.pmd.failonerror property, checkstyle has maven.checkstyle.fail.on.violation, etc.

Show
Lukas Theussl added a comment - This can only be implemented in the corresponding reports, eg the pmd plugin has a maven.pmd.failonerror property, checkstyle has maven.checkstyle.fail.on.violation, etc.
Hide
Permalink
Shinobu Kawai added a comment -

I was thinking more along the lines of using the jelly:catch tag, and swallowing the BuildException (or whatever gets thrown).

Show
Shinobu Kawai added a comment - I was thinking more along the lines of using the jelly:catch tag, and swallowing the BuildException (or whatever gets thrown).
Hide
Permalink
Shinobu Kawai added a comment -

Patch to ignore report failures.

Show
Shinobu Kawai added a comment - Patch to ignore report failures.
Hide
Permalink
Shinobu Kawai added a comment -

Provide a better message.

Show
Shinobu Kawai added a comment - Provide a better message.
Hide
Permalink
Lukas Theussl added a comment -

Can you provide a patch for the docs as well?
What will happen if you set eg maven.checkstyle.fail.on.violation=true but maven.site.reports.failonerror=false?

Show
Lukas Theussl added a comment - Can you provide a patch for the docs as well? What will happen if you set eg maven.checkstyle.fail.on.violation=true but maven.site.reports.failonerror=false?
Hide
Permalink
Shinobu Kawai added a comment -

Patch including xdocs.

Show
Shinobu Kawai added a comment - Patch including xdocs.
Hide
Permalink
Shinobu Kawai added a comment -

>> What will happen if you set eg maven.checkstyle.fail.on.violation=true but maven.site.reports.failonerror=false?

Haven't tried, but the checkstyle report will fail, but the site generation should continue. (The checkstyle report should become a broken link.)

The Use Case for this issue is when I don't need all of the reports to succeed before the site gets generated/deployed. For example, even if the source violated checkstyle, I still want to see my javadoc. As you commented earlier, not all plugins support don't-fail-on-error options.

Show
Shinobu Kawai added a comment - >> What will happen if you set eg maven.checkstyle.fail.on.violation=true but maven.site.reports.failonerror=false? Haven't tried, but the checkstyle report will fail, but the site generation should continue. (The checkstyle report should become a broken link.) The Use Case for this issue is when I don't need all of the reports to succeed before the site gets generated/deployed. For example, even if the source violated checkstyle, I still want to see my javadoc. As you commented earlier, not all plugins support don't-fail-on-error options.
Hide
Permalink
Lukas Theussl added a comment -

Patch applied with slight modifications. I renamed the property to maven.site.reports.ignoreErrors which appeared more logical
(eg the build did not fail with maven.checkstyle.fail.on.violation=false but maven.site.reports.failonerror=true). Thanks!

Show
Lukas Theussl added a comment - Patch applied with slight modifications. I renamed the property to maven.site.reports.ignoreErrors which appeared more logical (eg the build did not fail with maven.checkstyle.fail.on.violation=false but maven.site.reports.failonerror=true). Thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.