[
https://issues.apache.org/jira/browse/OFBIZ-12171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17289973#comment-17289973 ]
Daniel Watford commented on OFBIZ-12171:
----------------------------------------
Hi [~mbrohl],
There wasn't any difference in how files were checked out in git-bash (Windows) vs docker (Linux).
I also built the R18 PRs in WSL2 and has results matching the docker build.
As a test I was able to run unix2dos on one of the files containing the additional NewlineAtEndOfFileCheck errors and the error was removed. This suggested the problem was related to how checkstyle was determining valid line-endings on different platforms and that I always checkout sources with LF line-endings.
A quick check of trunk's config/checkstyle/checkstyle.xml file and we see that LF line-endings have been specified explicitly in trunk, but the same is missing in the R18 pull request.
Trunk:
{noformat}
<!-- General file conventions -->
<module name="NewlineAtEndOfFile">
<property name="lineSeparator" value="lf" />
</module>
{noformat}
PR276
{noformat}
<!-- General file conventions -->
<module name="NewlineAtEndOfFile"/>
{noformat}
Please could you add the additional lineSeparator property to the NewlineAtEndOfFile module in the PR's config/checkstyle/checkstyle.xml file.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)