Alex Behm has submitted this change and it was merged. Change subject: IMPALA-3632: Add script for runnig cppclean over the BE code. ......................................................................
IMPALA-3632: Add script for runnig cppclean over the BE code. This patch adds a script bin/cppclean.sh that runs cppclean over the BE code and writes the output to stdout. cppclean is a tool for detecting various issues with C++ code: https://github.com/myint/cppclean To simplify the above step this change also modifies CMakeLists.txt to allow dumping of all include paths to a file, like this: cmake -DDUMP_INCLUDE_PATHS=output_file.txt Change-Id: I5cd064344bdf47d77058fb570f034bb324308103 Reviewed-on: http://gerrit.cloudera.org:8080/3237 Reviewed-by: Alex Behm <[email protected]> Tested-by: Alex Behm <[email protected]> --- M CMakeLists.txt A bin/cppclean.sh 2 files changed, 51 insertions(+), 0 deletions(-) Approvals: Alex Behm: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/3237 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cd064344bdf47d77058fb570f034bb324308103 Gerrit-PatchSet: 4 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Alex Behm <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
