You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/08/08 18:36:42 UTC

kudu git commit: [CMakeLists.txt] dot is mandatory for doxygen

Repository: kudu
Updated Branches:
  refs/heads/master 647f904b5 -> 5f98cdb8c


[CMakeLists.txt] dot is mandatory for doxygen

Require dot (graphviz) support for doxygen since the dot is needed
when generating Kudu C++ client API documentation.

This addresses the following JIRA issue(s):
  KUDU-1543: doxygen: graphviz dependency not documented

Change-Id: I3466a587f98a528d9aa57edd4f4e96684b6922a2
Reviewed-on: http://gerrit.cloudera.org:8080/3855
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/5f98cdb8
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/5f98cdb8
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/5f98cdb8

Branch: refs/heads/master
Commit: 5f98cdb8c7235ee937618149d618a73e9adc67f6
Parents: 647f904
Author: Alexey Serbin <as...@cloudera.com>
Authored: Fri Aug 5 19:34:18 2016 -0700
Committer: Todd Lipcon <to...@apache.org>
Committed: Mon Aug 8 18:36:23 2016 +0000

----------------------------------------------------------------------
 CMakeLists.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/5f98cdb8/CMakeLists.txt
----------------------------------------------------------------------
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 01c9f3b..76ef662 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -976,8 +976,8 @@ endif (UNIX)
 ############################################################
 if (UNIX)
   find_package(Doxygen)
-  if (NOT DOXYGEN_FOUND)
-    message(WARNING "Doxygen not found: 'doxygen' target is not available")
+  if (NOT (DOXYGEN_FOUND AND DOXYGEN_DOT_FOUND))
+    message(WARNING "Doxygen with Dot support (graphviz) not found: 'doxygen' target is not available")
   else ()
     set(DOXY_SUBDIR ${CMAKE_CURRENT_BINARY_DIR}/docs/doxygen)
     set(DOXY_CLIENT_DESTDIR ${DOXY_SUBDIR}/tmp.client)