[groff] 15/115: [build]: Clarify GROFF_TMAC Autoconf test output.

[G. Branden Robinson]

gbranden pushed a commit to branch branden-2022-06-01
in repository groff.

commit 4a81ae882935271daa7952360629dc3740510619
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
AuthorDate: Mon Mar 13 22:23:02 2023 -0500

    [build]: Clarify GROFF_TMAC Autoconf test output.
    
    * configure.ac: Correct characterization of the list of macro packages
      reported.  It is not the list of macro packages receiving a "g"
      prefix, but the list of macro packages reciving a wrapper macro file
      in groff's macro directory, each of which sources the corresponding
      system (likely AT&T-descended troff) macro package.  Fixes wording I
      introduced in commit 4d30dd7424, 31 May.
    
    * m4/groff.m4 (GROFF_TMAC): Clarify first "checking" message: we are
      looking for a file name prefix on existing system macro packages (like
      "tmac.").  This has nothing to do with the prefix applied to groff
      commands, or the "g" prefixed to groff implementations of the man, mm,
      and ms packages.  Tighten wording of second "checking" message.
      Consistently say "(none)" instead of "none" or "none found".
---
 ChangeLog    | 21 +++++++++++++++++++++
 configure.ac |  2 +-
 m4/groff.m4  |  8 ++++----
 3 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index e8a912a66..9e64aa694 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,24 @@
+2023-03-13  G. Branden Robinson <g.branden.robinson@gmail.com>
+
+       [build]: Clarify output of `GROFF_TMAC` Autoconf test.
+
+       * configure.ac: Correct characterization of the list of macro
+       packages reported.  It is not the list of macro packages
+       receiving a "g" prefix, but the list of macro packages reciving
+       a wrapper macro file in groff's macro directory, each of which
+       sources the corresponding system (likely AT&T-descended troff)
+       macro package.  Fixes wording I introduced in commit 4d30dd7424,
+       31 May.
+       * m4/groff.m4 (GROFF_TMAC): Clarify first "checking" message:
+       we are looking for a file name prefix on existing system macro
+       packages (like "tmac.").  This has nothing to do with the prefix
+       applied to groff commands, or the "g" prefixed to groff
+       implementations of the man, mm, and ms packages.  Tighten
+       wording of second "checking" message.  Consistently say "(none)"
+       instead of "none" or "none found".
+
+       Fixes <https://savannah.gnu.org/bugs/?63900>.
+
 2023-03-11  G. Branden Robinson <g.branden.robinson@gmail.com>
 
        * doc/doc.am (install-txt): Look for "groff.txt" in the source
diff --git a/configure.ac b/configure.ac
index 4b1e92fb1..6baeaaf53 100644
--- a/configure.ac
+++ b/configure.ac
@@ -222,7 +222,7 @@ then
   if test -n "$tmac_wrap"
   then
   echo "\
- prefix used for macro packages   : $tmac_wrap"
+ wrappers used for macro packages : $tmac_wrap"
   fi
 fi
 echo "\
diff --git a/m4/groff.m4 b/m4/groff.m4
index c514b6a1a..fb1157e06 100644
--- a/m4/groff.m4
+++ b/m4/groff.m4
@@ -1118,7 +1118,7 @@ fopen(0, 0);
 
 
 AC_DEFUN([GROFF_TMAC],
-  [AC_MSG_CHECKING([for prefix of system macro packages])
+  [AC_MSG_CHECKING([file name prefix of system macro packages])
    sys_tmac_prefix=
    sys_tmac_file_prefix=
    for d in /usr/share/lib/tmac /usr/lib/tmac
@@ -1138,13 +1138,13 @@ AC_DEFUN([GROFF_TMAC],
        done
      done
    done
-   sys_tmac_prefix_result=none
+   sys_tmac_prefix_result='(none)'
    test -z "$sys_tmac_prefix" \
      || sys_tmac_prefix_result="$sys_tmac_prefix"
    AC_MSG_RESULT([$sys_tmac_prefix_result])
    AC_SUBST([sys_tmac_prefix])
 
-   AC_MSG_CHECKING([which system macro packages should be made available])
+   AC_MSG_CHECKING([for system macro packages to make available])
    tmac_wrap=
    if test "$sys_tmac_file_prefix" = tmac.
    then
@@ -1192,7 +1192,7 @@ AC_DEFUN([GROFF_TMAC],
      done
      rm -f conftest.sol
    fi
-   tmac_wrap_result="none found"
+   tmac_wrap_result='(none)'
    test -z "$tmac_wrap" || tmac_wrap_result="$tmac_wrap"
    AC_MSG_RESULT([$tmac_wrap_result])
    AC_SUBST([tmac_wrap])