groff 1.17 skips paper size checking

[Andras Salamon]

                           Groff Bug Report

Please read the PROBLEMS file before sending in a bug report.

Please fill in all fields, even if you think they are not relevant.

Please delete the text in brackets before sending it in.

Please report separate bugs separately.

Send the completed form to address@hidden

GROFF VERSION: 1.17

MACHINE: Sony VAIO laptop (Intel PIII)

OS: GNU/Linux, originally RedHat 7.1

COMPILER: gcc/g++ 2.96

INPUT FILES: none

COMMAND LINE: ./configure

DESCRIPTION OF INCORRECT BEHAVIOUR:

In aclocal.m4, lines 249 and 250, paper size will always be set to
letter if no prior installation of groff exists in installation path:
the existing paper size detection code based on the domain name is
always skipped.  In addition, where domainname returns '(none)' when
no NIS domain has been set, the hostname detection code is not used.


SUGGESTED FIX:

This small fix ensures that the existing domain detection code is
not skipped.  This is fine since if PAGE remains empty even after
the domain detection code, the last part of the page size detection
code sets it to letter anyway.  An additional check is also made for
'(none)' returned by domainname.

--- aclocal.m4.orig     Sat Apr 14 16:25:12 2001
+++ aclocal.m4  Mon Jun 25 11:38:51 2001
@@ -246,8 +246,6 @@
        if test -n "$descfile" \
          && grep "^paperlength 841890" $descfile >/dev/null 2>&1; then
                PAGE=A4
-       else
-               PAGE=letter
        fi
 fi
 if test -z "$PAGE"; then
@@ -255,7 +253,8 @@
            /etc/resolv.conf 2>/dev/null`
        if test -z "$dom"; then
                dom=`(domainname) 2>/dev/null | tr -d '+'`
-               if test -z "$dom"; then
+               if test -z "$dom" \
+                 || test "$dom" = '(none)'; then
                        dom=`(hostname) 2>/dev/null | grep '\.'`
                fi
        fi