Todd Lipcon has submitted this change and it was merged. Change subject: java: allow eclipse checkstyle to work ......................................................................
java: allow eclipse checkstyle to work Eclipse's checkstyle plugin doesn't properly handle paths relative to the checkstyle configuration XML file. The workaround is that it does provide a "samedir" property which can be used to construct the proper absolute path. Of course, Maven doesn't provide that property by default, so this patch also edits the pom so that Maven passes in "./" as "samedir". Change-Id: Ibb8c2d180b8c931b4e83ec5244f4b2e54155c5e4 Reviewed-on: http://gerrit.cloudera.org:8080/5943 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu_style.xml M java/pom.xml 2 files changed, 3 insertions(+), 2 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5943 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb8c2d180b8c931b4e83ec5244f4b2e54155c5e4 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
