diff --git a/NEWS.adoc b/NEWS.adoc index 388f7fe82c..4ea73c91e7 100644 --- a/NEWS.adoc +++ b/NEWS.adoc @@ -41,6 +41,13 @@ https://github.com/networkupstools/nut/milestone/9 - (expected) Bug fixes for fallout possible due to "fightwarn" effort in 2.8.0+ + - Fix fallout of development in NUT v2.8.0 and/or v2.8.1 and/or v2.8.2 and/or + v2.8.3: + * Fixed a regression in recipes of NUT v2.8.3 release (as compared to + v2.8.2), where `configure --with-docs=all` no longer failed a run + of the `configure` script when some of the required rendering tools + were not in fact available. [#2842] + - common code: * Revised common `writepid()` to use `altpidpath()` as location for the PID file creation, if the default `rootpidpath()` is not accessible diff --git a/configure.ac b/configure.ac index 87433c4065..3bcc090707 100644 --- a/configure.ac +++ b/configure.ac @@ -3463,9 +3463,6 @@ dnl is set to 'no', we may still want to build some doc targets manually dnl (so enable the Makefile recipes for those targets if tools are available) NUT_CHECK_ASCIIDOC -NUT_REPORT_FEATURE([requested to build and install documentation], [${nut_with_doc}], [], - [WITH_ASCIIDOC], [Define to enable Asciidoc support]) - DOC_INSTALL_DISTED_MANS=no KNOWN_UNABLE_MANS=no @@ -3496,6 +3493,9 @@ dnl If user passed --with-doc='' they they want nothing, right? ;; esac +NUT_REPORT_FEATURE([requested to build and install documentation], ['${nut_with_doc}' => '${nut_doc_build_list}'], [], + [WITH_ASCIIDOC], [Define to enable Asciidoc support], [-]) + if test -z "${abs_srcdir}" ; then case "${srcdir}" in /*) abs_srcdir="${srcdir}";; @@ -3574,7 +3574,7 @@ dnl not fail if we have no tools to generate it (so add to SKIP list). AC_MSG_RESULT(no) DOC_CANNOTBUILD_LIST="${DOC_CANNOTBUILD_LIST} ${nut_doc_build_target_base}" if test "${nut_doc_build_target_flag}" = "yes" ; then - AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested]) + AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested; will reflect on this below]) else DOC_SKIPBUILD_LIST="${DOC_SKIPBUILD_LIST} ${nut_doc_build_target_base}" fi @@ -3597,7 +3597,7 @@ dnl not fail if we have no tools to generate it (so add to SKIP list). AC_MSG_RESULT(no) DOC_CANNOTBUILD_LIST="${DOC_CANNOTBUILD_LIST} ${nut_doc_build_target_base}" if test "${nut_doc_build_target_flag}" = "yes" ; then - AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested]) + AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested; will reflect on this below]) else DOC_SKIPBUILD_LIST="${DOC_SKIPBUILD_LIST} ${nut_doc_build_target_base}" fi @@ -3623,7 +3623,7 @@ dnl not fail if we have no tools to generate it (so add to SKIP list). AC_MSG_RESULT(no) DOC_CANNOTBUILD_LIST="${DOC_CANNOTBUILD_LIST} ${nut_doc_build_target_base}" if test "${nut_doc_build_target_flag}" = "yes" ; then - AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested]) + AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested; will reflect on this below]) else DOC_SKIPBUILD_LIST="${DOC_SKIPBUILD_LIST} ${nut_doc_build_target_base}" fi @@ -3660,7 +3660,7 @@ dnl not fail if we have no tools to generate it (so add to SKIP list). DOC_CANNOTBUILD_LIST="${DOC_CANNOTBUILD_LIST} ${nut_doc_build_target_base}" KNOWN_UNABLE_MANS=yes if test "${nut_doc_build_target_flag}" = "yes" ; then - AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested]) + AC_MSG_WARN([Unable to build ${nut_doc_build_target_base} documentation which you requested; will reflect on this below]) else DOC_SKIPBUILD_LIST="${DOC_SKIPBUILD_LIST} ${nut_doc_build_target_base}" if test "${nut_doc_build_target_flag}" = "auto" || test "${nut_doc_build_target_flag}" = "dist-auto" ; then @@ -3684,28 +3684,30 @@ done rm -rf "${DOCTESTDIR}" AS_IF([test x"${nut_enable_configure_debug}" = xyes], [ - AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_BUILD_LIST: '${DOC_BUILD_LIST}']) - AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_CANNOTBUILD_LIST: '${DOC_CANNOTBUILD_LIST}']) - AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_SKIPBUILD_LIST: '${DOC_SKIPBUILD_LIST}']) + AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_BUILD_LIST: '${DOC_BUILD_LIST}']) + AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_CANNOTBUILD_LIST: '${DOC_CANNOTBUILD_LIST}']) + AC_MSG_NOTICE([(CONFIGURE-DEVEL-DEBUG) DOC_SKIPBUILD_LIST: '${DOC_SKIPBUILD_LIST}']) ]) case "${nut_with_doc}" in auto) if test -n "${DOC_BUILD_LIST}"; then + dnl # We can build at least one format... nut_with_doc="yes" else + dnl # We can not build any formats... nut_with_doc="no" fi ;; no) ;; -*) +*) dnl # yes, all, skip... if test -n "${DOC_CANNOTBUILD_LIST}"; then DOC_CANNOTBUILD_LIST_LINES="`echo "${DOC_CANNOTBUILD_LIST}" | tr ' ' '\n' | grep -vE '^$'`" - for DOCTYPE in ${DOC_BUILD_LIST} ; do - if echo "${DOC_CANNOTBUILD_LIST_LINES}" | grep -E "^${DOCTYPE}(|=yes)\$" ; then + for DOCTYPE in ${nut_doc_build_list} ; do + if echo "${DOC_CANNOTBUILD_LIST_LINES}" | grep -E "^${DOCTYPE}(|=yes)\$" >/dev/null 2>/dev/null ; then AC_MSG_ERROR([Unable to build${DOC_CANNOTBUILD_LIST} documentation (check for 'no' results above)]) - else if echo "${DOC_CANNOTBUILD_LIST_LINES}" | grep -E "^${DOCTYPE}=(skip|auto|dist-auto)\$" ; then + else if echo "${DOC_CANNOTBUILD_LIST_LINES}" | grep -E "^${DOCTYPE}=(skip|auto|dist-auto)\$" >/dev/null 2>/dev/null ; then AC_MSG_NOTICE([Unable to build${DOC_CANNOTBUILD_LIST} documentation (check for 'no' results above), skipping]) fi fi @@ -3727,7 +3729,7 @@ esac AM_CONDITIONAL(WITH_PDF_NONASCII_TITLES, [test x"$can_build_doc_pdf_nonascii_titles" = xyes]) NUT_REPORT_FEATURE([would build specific documentation format(s)], [${nut_with_doc}], [${DOC_BUILD_LIST}], - [WITH_DOCS], [Define to enable overall documentation generation]) + [WITH_DOCS], [Define to enable overall documentation generation], [-]) # To cater for less portable make's, precalculate the target list # for "make check" in "docs/" here... diff --git a/m4/nut_report_feature.m4 b/m4/nut_report_feature.m4 index 5b326c8deb..43ef880f4e 100644 --- a/m4/nut_report_feature.m4 +++ b/m4/nut_report_feature.m4 @@ -111,7 +111,18 @@ AC_DEFUN([NUT_REPORT_FEATURE], dnl arg#3 = value dnl arg#4 = autoconf varname dnl arg#5 = longer description (autoconf comment) - AC_MSG_CHECKING([whether to $1]) + dnl arg#6 = (OPTIONAL) word(s) after prefix "whether" before "$1", + dnl or nothing if "-"; NOTE: "whether" is not part of this + dnl consideration (see NUT REPORT used below with a partial + dnl message) + nrf_tmp="${6-}" + AS_CASE([x"${nrf_tmp}"], + [x], [nrf_tmp="to "], + [x-], [nrf_tmp=""] + ) + AC_MSG_CHECKING([whether ${nrf_tmp}$1]) + unset nrf_tmp + AC_MSG_RESULT([$2 $3]) NUT_REPORT([$1], [$2 $3])