apache / apache/maven-checkstyle-plugin

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

Open
#289 11 comments 0 reactions 1 assignee Claimed by @eolivelli View on GitHub
enhancement priority:major
Dominant language
Java
Stars
97
Forks
67
Avg merge
14h 50m
Merged PRs (30d)
6

Description

**[Tibor Digana](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=tibor17)** opened **[MCHECKSTYLE-376](https://issues.apache.org/jira/browse/MCHECKSTYLE-376?redirect=false)** and commented

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:**
- [linux.zip](https://issues.apache.org/jira/secure/attachment/12968708/linux.zip) (_1.50 kB_)
- [windows.zip](https://issues.apache.org/jira/secure/attachment/12968709/windows.zip) (_1.54 kB_)

**Remote Links:**
- [GitHub Pull Request #16
](https://github.com/apache/maven-checkstyle-plugin/pull/16)

0 votes, 6 watchers

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.