On 2018年04月11日 16:29, Misono Tomohiro wrote: > AC_PATH_PROG won't fail even if it fails to find command path. > xmlto is required for document build and we should report error > if it doesn't exist at configure time. > > Signed-off-by: Tomohiro Misono <misono.tomoh...@jp.fujitsu.com>
Looks good. Reviewed-by: Qu Wenruo <w...@suse.com> Thanks, Qu > --- > configure.ac | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index af13a959..a4c98b31 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -93,7 +93,11 @@ AC_SUBST([DISABLE_DOCUMENTATION]) > dnl detect tools to build documentation > ASCIIDOC_TOOL="none" > if test "x$enable_documentation" = xyes; then > - AC_PATH_PROG([XMLTO], [xmlto], [xmlto]) > + AC_PATH_PROG([XMLTO], [xmlto]) > + if test -z "$XMLTO"; then > + AC_MSG_ERROR([cannot find xmlto, cannot build documentation]) > + fi > + > AC_PATH_PROG([GZIP], [gzip], [gzip]) > AC_PATH_PROG([MV], [mv], [mv]) > AC_PROG_SED > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html