Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-68492

'javadoc:javadoc' can no longer be executed with 'install'

XMLWordPrintable

    • 2.348, 2.346.1

      This used to work before https://github.com/jenkinsci/jenkins/pull/6366 :

      mvn clean javadoc:javadoc install

      AbstractGroovyViewModule references generated code (taglib-interface), so it's understandable that it now fails.

      Trying

      mvn clean install javadoc:javadoc

      fails because of Checkstyle violations in hudson/cli/client/Messages.java (33 in recent releases, 1 in older releases)

      We run javadoc:javadoc explicitly on cert.ci to ensure we do not introduce Javadoc errors that fail the release build (although the impact PR 6366 makes me think it didn't actually accomplish that before).

      Filing as a bug because I think one of the invocations above should work (in particular, Checkstyle looking at generated sources seems wrong), but open to alternatives.

          [JENKINS-68492] 'javadoc:javadoc' can no longer be executed with 'install'

          Daniel Beck created issue -
          Daniel Beck made changes -
          Description Original: This used to work before [https://github.com/jenkinsci/jenkins/pull/6366] :
          {noformat}
          mvn clean javadoc:javadoc install {noformat}
          {{AbstractGroovyViewModule}} references generated code ({{{}taglib-interface{}}}), so it's understandable that it now fails.

          Trying
          {noformat}
          mvn clean install javadoc:javadoc {noformat}
          fails because of Checkstyle violations in {{hudson/cli/client/Messages.java}} (33 in recent releases, 1 in older releases)
          ----
          We run {{javadoc:javadoc}} explicitly on cert.ci to ensure we do not introduce Javadoc errors that fail the release build (although the impact PR 6366 makes me think it didn't actually accomplish that before).

          Filing as a bug because I think one of the invocations above should work, but open to alternatives.           		New: This used to work before [https://github.com/jenkinsci/jenkins/pull/6366] :
          {noformat}
          mvn clean javadoc:javadoc install {noformat}
          {{AbstractGroovyViewModule}} references generated code ({{{}taglib-interface{}}}), so it's understandable that it now fails.

          Trying
          {noformat}
          mvn clean install javadoc:javadoc {noformat}
          fails because of Checkstyle violations in {{hudson/cli/client/Messages.java}} (33 in recent releases, 1 in older releases)
          ----
          We run {{javadoc:javadoc}} explicitly on cert.ci to ensure we do not introduce Javadoc errors that fail the release build (although the impact PR 6366 makes me think it didn't actually accomplish that before).

          Filing as a bug because I think one of the invocations above should work (in particular, Checkstyle looking at generated sources seems wrong), but open to alternatives.
          Mark Waite made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Mark Waite made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Mark Waite made changes -
          Remote Link New: This issue links to "Core PR 6560 to resolve the issue (Web Link)" [ 27755 ]
          Daniel Beck made changes -
          Labels Original: cert.ci regression New: cert.ci lts-candidate regression
          Basil Crow made changes -
          Released As New: 2.348
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Fixed but Unreleased [ 10203 ]
          Basil Crow made changes -
          Assignee New: Basil Crow [ basil ]
          Alexander Brandes made changes -
          Labels Original: cert.ci lts-candidate regression New: 2.346.1-fixed cert.ci lts-candidate regression
          Mark Waite made changes -
          Status Original: Fixed but Unreleased [ 10203 ] New: Closed [ 6 ]
          Alexander Brandes made changes -
          Labels Original: 2.346.1-fixed cert.ci lts-candidate regression New: 2.346.1-fixed cert.ci regression

            basil Basil Crow
            danielbeck Daniel Beck
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: