shithub: opus

Download patch

ref: 76fa939bde92a868c2a7e51018b79e56d1484285
parent: 6d0628493550f4ef0eaecf3058a502404bc6dca2
author: Ralph Giles <[email protected]>
date: Thu Oct 27 06:23:29 EDT 2016

Make use of dot optional in generating documentation.

Different distributions of doxygen have different default
values of HAVE_DOT setting, so we need to pick a specific
setting to avoid 'missing dot' warnings on some platforms.

Doxygen uses it to generate inclusion graphs for our various
header files, which is somewhat useful, but not essential.
We therefore enable dot if it's present (usually through
the parent graphviz package) but disable it if it's not
available, silencing the warning, but not giving uniform
results.

--- a/configure.ac
+++ b/configure.ac
@@ -781,6 +781,7 @@
 
 AS_IF([test "$enable_doc" = "yes"], [
   AC_CHECK_PROG(HAVE_DOXYGEN, [doxygen], [yes], [no])
+  AC_CHECK_PROG(HAVE_DOT, [dot], [yes], [no])
 ],[
   HAVE_DOXYGEN=no
 ])
--- a/doc/Doxyfile.in
+++ b/doc/Doxyfile.in
@@ -318,3 +318,18 @@
                          OPUS_CUSTOM_EXPORT_STATIC= \
                          OPUS_WARN_UNUSED_RESULT= \
                          OPUS_ARG_NONNULL(_x)=
+
+#---------------------------------------------------------------------------
+# Configuration options related to the dot tool
+#---------------------------------------------------------------------------
+
+# If you set the HAVE_DOT tag to YES then doxygen will assume the dot tool is
+# available from the path. This tool is part of Graphviz (see:
+# http://www.graphviz.org/), a graph visualization toolkit from AT&T and Lucent
+# Bell Labs.
+
+# Debian defaults to YES here, while Fedora and Homebrew default to NO.
+# So we set this based on whether the graphviz package is available at
+# configure time.
+#
+HAVE_DOT = @HAVE_DOT@