Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14995 )
Change subject: [build] Fix IWYU dependencies ...................................................................... [build] Fix IWYU dependencies IWYU is failing with a missing `version_defines.h`. This patch fixes the dependency chain by adding `gen_version_info` to the generated-headers target. This ensures it’s run for the iwyu and tidy targets. Change-Id: I2521a1d54b695971afbe445729e746beeb5db595 Reviewed-on: http://gerrit.cloudera.org:8080/14995 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M CMakeLists.txt 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14995 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2521a1d54b695971afbe445729e746beeb5db595 Gerrit-Change-Number: 14995 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
