Maven Javadoc Plugin
  1. Maven Javadoc Plugin
  2. MJAVADOC-304

failOnError is ignored if javadoc:javadoc is called via Site Plugin 3.0-beta

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.8
    • Labels:
      None
    • Testcase included:
      yes
    • Patch Submitted:
      Yes
    • Number of attachments :
      2

      Description

      failOnError works for the case that javadoc:jar is executed. This invocation calls execute() and then generate(sink, locale). The generate is surrounded with a try/catch-block catching any exception thrown by the javadoc generation. In case of failOnError==false the exception does not interrupt the build. That perfectly ok!

      failOnError does not work if it is called during site generation. During site generation only generate(sink, locale is called which misses the try/catch guard for failures during generation of javadoc.

      Effectively you cannot ignore failures during site generation.

      1. MJAVADOC-304_IT_Test.patch
        6 kB
        Sebastian Annies
      2. MJAVADOC-304_Patch.patch
        0.8 kB
        Sebastian Annies

        Activity

        Hide
        Sebastian Annies added a comment -

        Integration Test (IT Test) and Patch.

        Show
        Sebastian Annies added a comment - Integration Test (IT Test) and Patch.
        Hide
        Felix Simmendinger added a comment -

        This bug is very nasty since you have to configure your release plugin to use preparationGoals "clean install" otherwise thirdparty javadoc references will fail the build as those dependencies aren't present in the classpath of javadoc.

        Show
        Felix Simmendinger added a comment - This bug is very nasty since you have to configure your release plugin to use preparationGoals "clean install" otherwise thirdparty javadoc references will fail the build as those dependencies aren't present in the classpath of javadoc.
        Hide
        Herve Boutemy added a comment -

        fixed in r1098423

        Show
        Herve Boutemy added a comment - fixed in r1098423

          People

          • Assignee:
            Herve Boutemy
            Reporter:
            Sebastian Annies
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: