Alexey Serbin has posted comments on this change. Change subject: doxygen for C++ client API ......................................................................
Patch Set 8: (2 comments) Will send an update shortly. http://gerrit.cloudera.org:8080/#/c/3619/8/CMakeLists.txt File CMakeLists.txt: Line 989: add_custom_target(doxy_install_client_alt_destdir > Interesting. I think the kudu_client_exported problem is due a misuse of ad Perfect -- it works now. So, it was a bug, not a feature. Thank you for fixing this! Line 992: WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/src/kudu/client > Running make out of anywhere but the top-level directory isn't something mo OK, that sounds reasonable. -- To view, visit http://gerrit.cloudera.org:8080/3619 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie7d42fb1c90b83074e357dcecf42489ed9fc4f02 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-HasComments: Yes