Hi Tedd, On Tue, May 11, 2021 at 11:02 AM Tedd Ho-Jeong An <hj.tedd.an@xxxxxxxxx> wrote: > > From: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx> > > This patch adds a check if the conversion tool is installed when the > manpages is enabled, and show error if it is not installed. > --- > configure.ac | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/configure.ac b/configure.ac > index f445589b0..2b044f8e5 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -328,6 +328,9 @@ AC_ARG_ENABLE(manpages, AC_HELP_STRING([--enable-manpages], > [enable_manpages=${enableval}]) > if (test "${enable_manpages}" != "no"); then > AC_CHECK_PROGS(RST2MAN, [rst2man rst2man.py], "no") > + if (test "${RST2MAN}" = "no" ); then > + AC_MSG_ERROR([rst2man is required]) I guess we can mention that it is typically packaged with python-docutils e.g: rstman is required: install python-docutils or alternatively we check python-docutils with pkgconfig since it will be a hard dependency anyway. > + fi > fi > AM_CONDITIONAL(MANPAGES, test "${enable_manpages}" = "yes") > AM_CONDITIONAL(RUN_RST2MAN, test "${enable_manpages}" = "yes" && test "${RST2MAN}" != "no") > -- > 2.26.3 > -- Luiz Augusto von Dentz