https://gcc.gnu.org/bugzilla/show_bug.cgi?id=122638

--- Comment #8 from Eric Gallager <egallager at gcc dot gnu.org> ---
(In reply to Eric Gallager from comment #7)
> (In reply to [email protected] from comment #6)
> > However, such a requirement and merely documenting it in install.texi is
> > meaningless unless it's enforced with a version check and makeinfo
> > treated as missing if too old
> 
> True; I think I've seen an autoconf macro to perform just such a check
> somewhere before; I will see if I can track it down...

It looks like libiberty's configure.ac already has this:

# Do we have a single-tree copy of texinfo?  Even if we do, we can't
# rely on it - libiberty is built before texinfo.
AC_CHECK_PROG(MAKEINFO, makeinfo, makeinfo, )
if test "x$MAKEINFO" = "x"; then
    MAKEINFO="@echo makeinfo missing; true"
    BUILD_INFO=
else
    BUILD_INFO=info
    case "$MAKEINFO" in
      */missing\ makeinfo*)
        BUILD_INFO=
        AC_MSG_WARN([
*** Makeinfo is missing. Info documentation will not be built.])
        ;;
      *)
        case x"`$MAKEINFO --version | grep 'GNU texinfo'`" in
          x*\ [[1-3]].* )
            MAKEINFO="@echo $MAKEINFO is too old, 4.0 or newer required; true"
            BUILD_INFO=
            AC_MSG_WARN([
*** Makeinfo is too old. Info documentation will not be built.])
          ;;
        esac
        ;;
    esac
fi
AC_SUBST(MAKEINFO)
AC_SUBST(BUILD_INFO)

Also, in my fork of gdb, I have this in gdb's configure.ac file:

# We never need to detect it in this sub-configure.
# But preserve it for config.status --recheck.
AC_ARG_VAR([MAKEINFO],
           [Parent configure detects if it is of sufficient version.])dnl
AC_ARG_VAR([MAKEINFOFLAGS],
           [Parameters for MAKEINFO.])dnl

MAKEINFO_EXTRA_FLAGS=""
if test "x${MAKEINFO}" != "x"; then
  test -n "${MAKEINFO}" && echo "MAKEINFO is: '${MAKEINFO}'"
  AC_CACHE_CHECK([whether ${MAKEINFO} supports @click],
                 [gdb_cv_have_makeinfo_click],
    [echo '@clicksequence{a @click{} b}' >conftest.texinfo
     if eval "${MAKEINFO} conftest.texinfo >&5 2>&5"; then
       gdb_cv_have_makeinfo_click=yes
     else
       gdb_cv_have_makeinfo_click=no
     fi])
  if test x"${gdb_cv_have_makeinfo_click}" = x"yes"; then
    MAKEINFO_EXTRA_FLAGS="${MAKEINFO_EXTRA_FLAGS} -DHAVE_MAKEINFO_CLICK"
  fi
  AC_SUBST([MAKEINFO_EXTRA_FLAGS])dnl
  ## end makeinfo stuff
fi

Reply via email to