Uploaded image for project: 'Maven Checkstyle Plugin'
  1. Maven Checkstyle Plugin
  2. MCHECKSTYLE-376

The IT fails MCHECKSTYLE-54 (Unix EOL files). Broken rule 'NewlineAtEndOfFile' on Windows.

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • checkstyle:check
    • None

    Description

      Afects version 3.1.0 during the release vote.

      It looks like the library expects system based EOL character.
      The checkstyle reports the issue in two files in https://github.com/apache/maven-checkstyle-plugin/tree/master/src/it/MCHECKSTYLE-54/src/main/java/org/apache/maven/plugins/checkstyle/mcheckstyle54

      I doubt it is a problem with these two files, nothing but the library itself.
      No idea why other ITs do not fail on Windows too.

      mvn -nsu checkstyle:check
      
      [INFO] --- maven-checkstyle-plugin:3.1.1-SNAPSHOT:check (default-cli) @ mcheckstyle-54 ---
      [INFO] There are 2 errors reported by Checkstyle 8.19 with sun_checks.xml ruleset.
      [ERROR] src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1] (misc) NewlineAtEndOfFile: File does not end with a newline.
      [ERROR] src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1] (misc) NewlineAtEndOfFile: File does not end with a newline.
      [INFO] ------------------------------------------------------------------------
      [INFO] BUILD FAILURE
      [INFO] ------------------------------------------------------------------------
      [INFO] Total time: 3.450 s
      [INFO] Finished at: 2019-05-13T23:42:55+02:00
      [INFO] Final Memory: 12M/193M
      [INFO] ------------------------------------------------------------------------
      [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1-SNAPSHOT:check (default-cli) on project mcheckstyle-54: You have 2 Checkstyle violations. -> [Help 1]
      [ERROR]
      [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
      [ERROR] Re-run Maven using the -X switch to enable full debug logging.
      [ERROR]
      [ERROR] For more information about the errors and possible solutions, please read the following articles:
      [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
      

      Attachments

        1. linux.zip
          1 kB
          Tibor Digana
        2. windows.zip
          2 kB
          Tibor Digana

        Issue Links

          Activity

            People

              eolivelli Enrico Olivelli
              tibordigana Tibor Digana
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated: