Alexey Serbin has posted comments on this change. Change subject: [build-support] fixed IWYU no-op configuration ......................................................................
Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7833/1/build-support/jenkins/build-and-test.sh File build-support/jenkins/build-and-test.sh: Line 242: echo "IWYU is a no-op configuration in this branch" > I think this needs to actually go into the iwyu.log file, so there is an ar Uh-oh, my bad. I still feel some deficiency in the noggin department today :( Yes, you are right -- it should be explicitly stored into the log. -- To view, visit http://gerrit.cloudera.org:8080/7833 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id349a1ee83ad6b9e5fb6f35c4d044487e6aff1c3 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
