The build will fail because we use checkstyle to enforce some important style rules. Trailing new lines are important for most diff tools including git diff, so it's not totally insane. I'm of the mind that checkstyle actually facilitates accepting contributions since the build will catch stupid style mistakes and we don't have to nitpick patches for style adherence.