Uploaded image for project: 'ManifoldCF'
  1. ManifoldCF
  2. CONNECTORS-1618

Maven javadoc build fails due to linter errors

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • ManifoldCF 2.14
    • Build
    • None

    Description

      mvn javadoc:javadoc fails with due to an awful lot of linter errors. (unescaped html chars end entities, invalid parameter names, return tags without description, self closing paragraph tags)

      Attachments

        Issue Links

          Activity

            schuch Markus Schuch added a comment -

            The javadoc build produces warnings and errors. The errors break the javadoc build. I will only address the errors.

            schuch Markus Schuch added a comment - The javadoc build produces warnings and errors. The errors break the javadoc build. I will only address the errors.
            schuch Markus Schuch added a comment - - edited

            r1865235 (branches/CONNECTORS-1618)

            • removes needles/empty javadoc statements causing errors
            • escapes html characters in javadoc comments
            • fixes invalid selfclosing paragraphs
            • fixes not matching parameter names
            • fixes other javadoc syntax errors
            schuch Markus Schuch added a comment - - edited r1865235 (branches/ CONNECTORS-1618 ) removes needles/empty javadoc statements causing errors escapes html characters in javadoc comments fixes invalid selfclosing paragraphs fixes not matching parameter names fixes other javadoc syntax errors
            schuch Markus Schuch added a comment -

            r1865267 (branches/CONNECTORS-1618)

            schuch Markus Schuch added a comment - r1865267 (branches/ CONNECTORS-1618 ) fixes travis ci builds (were failing due to build image changes - see https://travis-ci.community/t/install-of-oracle-jdk-8-failing/3038/8 ) adds a mvn javadoc:javadoc build task to travis ci
            schuch Markus Schuch added a comment - - edited

            Travis Maven Build and Maven Javadoc Build were successful on the branch build: https://travis-ci.org/apache/manifoldcf/builds/572616423

            The ant build failed due to failing tests... but this is another story.

            daddywri i would like to merge this to master. Any objections?
            Maybe we should add a jenkins job for for building javadoc with maven to avoid finding build issues when trying to release the maven artifacts.

            schuch Markus Schuch added a comment - - edited Travis Maven Build and Maven Javadoc Build were successful on the branch build: https://travis-ci.org/apache/manifoldcf/builds/572616423 The ant build failed due to failing tests... but this is another story. daddywri i would like to merge this to master. Any objections? Maybe we should add a jenkins job for for building javadoc with maven to avoid finding build issues when trying to release the maven artifacts.
            schuch Markus Schuch added a comment -

            merged to trunk to make progress on the blocked maven delivery issue.

            r1865954 (trunk)

            schuch Markus Schuch added a comment - merged to trunk to make progress on the blocked maven delivery issue. r1865954 (trunk)
            schuch Markus Schuch added a comment -

            According to https://cwiki.apache.org/confluence/display/INFRA/Jenkins PMC chairs can grant access to jenkins to allow configuration ob jobs. I would like to add the javadoc build as a separate job.

            daddywri how are you handling this? Do you like to create the job. Are you willing to grant configuration access to me?

            schuch Markus Schuch added a comment - According to https://cwiki.apache.org/confluence/display/INFRA/Jenkins PMC chairs can grant access to jenkins to allow configuration ob jobs. I would like to add the javadoc build as a separate job. daddywri how are you handling this? Do you like to create the job. Are you willing to grant configuration access to me?

            People

              schuch Markus Schuch
              schuch Markus Schuch
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: