public inbox for binutils-cvs@sourceware.org
help / color / mirror / Atom feed
From: Vladimir Mezentsev <vmezents@sourceware.org>
To: bfd-cvs@sourceware.org, gdb-cvs@sourceware.org
Subject: [binutils-gdb/binutils-2_39-branch] gprofng: check for the makeinfo version
Date: Tue, 26 Jul 2022 23:51:27 +0000 (GMT)	[thread overview]
Message-ID: <20220726235127.ABD943858412@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=fc9672939576752123aebfa3e6f4d4f1b9674dd2

commit fc9672939576752123aebfa3e6f4d4f1b9674dd2
Author: Vladimir Mezentsev <vladimir.mezentsev@oracle.com>
Date:   Mon Jul 25 23:57:46 2022 -0700

    gprofng: check for the makeinfo version
    
    gprofng/ChangeLog
    2022-07-25  Vladimir Mezentsev  <vladimir.mezentsev@oracle.com>
    
            PR gprofng/29368
            * configure.ac: Check for the makeinfo version.
            * configure: Rebuild.

Diff:
---
 gprofng/configure    | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++--
 gprofng/configure.ac | 22 +++++++++++++++---
 2 files changed, 81 insertions(+), 5 deletions(-)

diff --git a/gprofng/configure b/gprofng/configure
index 9ea2416f41e..2cddf9dc88f 100755
--- a/gprofng/configure
+++ b/gprofng/configure
@@ -16709,13 +16709,73 @@ fi
 
 
 # Generate manpages, if possible.
+build_man=false
 if test $cross_compiling = no; then
 
 HELP2MAN=${HELP2MAN-"${am_missing_run}help2man"}
 
-  build_man=true
+  for ac_prog in makeinfo
+do
+  # Extract the first word of "$ac_prog", so it can be a program name with args.
+set dummy $ac_prog; ac_word=$2
+{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for $ac_word" >&5
+$as_echo_n "checking for $ac_word... " >&6; }
+if ${ac_cv_prog_MAKEINFO+:} false; then :
+  $as_echo_n "(cached) " >&6
+else
+  if test -n "$MAKEINFO"; then
+  ac_cv_prog_MAKEINFO="$MAKEINFO" # Let the user override the test.
 else
-  build_man=false
+as_save_IFS=$IFS; IFS=$PATH_SEPARATOR
+for as_dir in $PATH
+do
+  IFS=$as_save_IFS
+  test -z "$as_dir" && as_dir=.
+    for ac_exec_ext in '' $ac_executable_extensions; do
+  if as_fn_executable_p "$as_dir/$ac_word$ac_exec_ext"; then
+    ac_cv_prog_MAKEINFO="$ac_prog"
+    $as_echo "$as_me:${as_lineno-$LINENO}: found $as_dir/$ac_word$ac_exec_ext" >&5
+    break 2
+  fi
+done
+  done
+IFS=$as_save_IFS
+
+fi
+fi
+MAKEINFO=$ac_cv_prog_MAKEINFO
+if test -n "$MAKEINFO"; then
+  { $as_echo "$as_me:${as_lineno-$LINENO}: result: $MAKEINFO" >&5
+$as_echo "$MAKEINFO" >&6; }
+else
+  { $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5
+$as_echo "no" >&6; }
+fi
+
+
+  test -n "$MAKEINFO" && break
+done
+test -n "$MAKEINFO" || MAKEINFO=""@echo makeinfo missing; true""
+
+  case "$MAKEINFO" in
+    *true)
+      { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: gprofng: makeinfo is missing. Info documentation will not be built." >&5
+$as_echo "$as_me: WARNING: gprofng: makeinfo is missing. Info documentation will not be built." >&2;}
+      ;;
+    *)
+      case x"`$MAKEINFO --version | grep 'GNU texinfo'`" in
+	x*\ [1-5].*|x*\ 6.[0-4].* )
+	  { $as_echo "$as_me:${as_lineno-$LINENO}: WARNING: gprofng: $MAKEINFO is too old. Info documentation will not be built." >&5
+$as_echo "$as_me: WARNING: gprofng: $MAKEINFO is too old. Info documentation will not be built." >&2;}
+	  MAKEINFO="@echo $MAKEINFO is too old, 6.5 or newer required; true"
+	  ;;
+	x* )
+	  build_man=true
+	  ;;
+	esac
+      ;;
+    esac
+
 fi
  if test x$build_man = xtrue; then
   BUILD_MAN_TRUE=
diff --git a/gprofng/configure.ac b/gprofng/configure.ac
index 5a0fe1ea477..e9453109602 100644
--- a/gprofng/configure.ac
+++ b/gprofng/configure.ac
@@ -184,11 +184,27 @@ AM_CONDITIONAL(TCL_TRY, test "${ac_cv_libctf_tcl_try}" = yes)
 AM_ZLIB
 
 # Generate manpages, if possible.
+build_man=false
 if test $cross_compiling = no; then
   AM_MISSING_PROG(HELP2MAN, help2man)
-  build_man=true
-else
-  build_man=false
+  AC_CHECK_PROGS([MAKEINFO], makeinfo, ["@echo makeinfo missing; true"])
+  case "$MAKEINFO" in
+    *true)
+      AC_MSG_WARN([gprofng: makeinfo is missing. Info documentation will not be built.])
+      ;;
+    *)
+      case x"`$MAKEINFO --version | grep 'GNU texinfo'`" in
+	x*\ [[1-5]].*|x*\ 6.[[0-4]].* )
+	  AC_MSG_WARN([gprofng: $MAKEINFO is too old. Info documentation will not be built.])
+	  MAKEINFO="@echo $MAKEINFO is too old, 6.5 or newer required; true"
+	  ;;
+	x* )
+	  build_man=true
+	  ;;
+	esac
+      ;;
+    esac
+  AC_SUBST(MAKEINFO)
 fi
 AM_CONDITIONAL([BUILD_MAN], [test x$build_man = xtrue])


                 reply	other threads:[~2022-07-26 23:51 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220726235127.ABD943858412@sourceware.org \
    --to=vmezents@sourceware.org \
    --cc=bfd-cvs@sourceware.org \
    --cc=gdb-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).