[Patch v4 1/3] doc: build and install doxygen api docs

Tomi Ollila tomi.ollila at iki.fi
Wed Jul 9 13:22:49 PDT 2014


On Wed, Jul 09 2014, David Bremner <david at tethera.net> wrote:

> In order to support out of tree builds, generate `doc/config.dox` from
> configure.
>
> In order to avoid hardcoding version in doxygen.cfg, generate
> doc/version.dox at build time.

is this commit message accurate ?

> ---
>  configure          | 12 ++++++++++++
>  doc/Makefile.local | 27 +++++++++++++++++++++++++--
>  doc/doxygen.cfg    |  5 ++---
>  3 files changed, 39 insertions(+), 5 deletions(-)
>
> diff --git a/configure b/configure
> index 9514d4d..fa44188 100755
> --- a/configure
> +++ b/configure
> @@ -417,6 +417,15 @@ else
>      have_emacs=0
>  fi
>  
> +printf "Checking if doxygen is available... "
> +if doxygen -v > /dev/null 2>&1; then

Doxygen 1.6.1 does not know -v -- and exits w/ value 1. If we wanted to
allow man generation using that version (out-of-the-box) then the line
above should be `if command -v doxygen >/dev/null; then`

otherwise this series looks good and generation seems to work.

Tomi

> +    printf "Yes.\n"
> +    have_doxygen=1
> +else
> +    printf "No (so will not install api docs)\n"
> +    have_doxygen=0
> +fi
> +


More information about the notmuch mailing list