Windows Support: Checkstyle

[kadaan]

Presto fails to build on Windows due to checkstyle enforcing that lines end in \n. This is not the line separator in windows. Can the checkstyle checks be changed to allow any of: \r\n, \n, \r? Alternately, can a .gitattributes file be added which specifies that LF is the correct newline character by: text eol=lf

[ArturGajowy]

I think the latter (.gitattributes) would be a better solution. @electrum WDYT?

[findepi]

@kadaan, the repository doesn't contain source files with \r\n. If your git client creates such line endings upon checkout, it's a matter of your git client's configuration. Did you set core.autocrlf?

[blackhawkbigpunch]

Setting core.autocrlf to false bypassed the fail with maven-checkstyle-plugin issue in windows.

[findepi]

@blackhawkbigpunch thanks for sharing this