Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/16715 )
Change subject: [docs] Update doxygen requirements in docs ...................................................................... Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/16715/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/16715/1//COMMIT_MSG@12 PS1, Line 12: which is not enough now > Ah, I see. Thank you for the clarification. I forgot those older releases Done http://gerrit.cloudera.org:8080/#/c/16715/1/docs/installation.adoc File docs/installation.adoc: http://gerrit.cloudera.org:8080/#/c/16715/1/docs/installation.adoc@135 PS1, Line 135: graphviz > nit: I guess libgraphviz-dev would be required in addition if building doxy I don't think that's true. I built doxygen on my CentOS 7 server without graphviz-devel being installed (only graphviz) and it worked and was able to build the docs too. http://gerrit.cloudera.org:8080/#/c/16715/1/docs/installation.adoc@140 PS1, Line 140: built from source > Is graphviz support is required by doxygen config by default when building I don't think so, I built without graphviz-devel and any extra options. http://gerrit.cloudera.org:8080/#/c/16715/1/docs/installation.adoc@142 PS1, Line 142: CentOS older than 8.0 > It seems CentOS 8 has doxygen-1.8.14: https://centos.pkgs.org/8/centos-powe Hm I guess it is a bit ambiguous. I meant devtoolset is required in pre-8.0 to build Doxygen. -- To view, visit http://gerrit.cloudera.org:8080/16715 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie0f92f185490ad15d8f4f71d9c7fc5f6a6073fdc Gerrit-Change-Number: 16715 Gerrit-PatchSet: 2 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 13 Nov 2020 08:39:23 +0000 Gerrit-HasComments: Yes
