Re: [PATCH] build: Don???t assume the paper size???s case.

[Bjarni Ingi Gislason]

On Sun, Mar 30, 2025 at 11:26:34PM +0000, Lennart Jablonka wrote:
> libpaper reads /etc/papersize case-insensitively; `configure PAGE=???`
> shouldn???t be case-sensitive either.
> 
  The file "/etc/papersize" is no longer on Debian, testing, but
"/etc/paperspecs" (package libpaper2) is, with all lines beginning with a 
capital letter or a
digit.

> I haven???t looked very closely, but I suspect the comment in Makefile.am
> on config.h is wrong: I don???t see where PAGE would be #defined.
> ---
>  Makefile.am           | 2 +-
>  font/devdvi/devdvi.am | 2 +-
>  font/devlbp/devlbp.am | 2 +-
>  font/devlj4/devlj4.am | 2 +-
>  font/devpdf/devpdf.am | 2 +-
>  font/devps/devps.am   | 2 +-
>  m4/groff.m4           | 8 +++++---
>  7 files changed, 11 insertions(+), 9 deletions(-)
> 
> diff --git a/Makefile.am b/Makefile.am
> index 3e1fee78d..d6756946e 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -353,7 +353,7 @@ man7dir=$(manroot)/man$(man7ext)
>  # uintmax_t=<value>          define to `unsigned long' or `unsigned long
>  #                            long' if <inttypes.h> does not exist
>  #
> -# PAGE=A4                    if the printer's page size is A4
> +# PAGE=a4                    if the printer's page size is A4
>  # GHOSTSCRIPT=gs             the name (and directory if required) of the
>  #                            ghostscript program
>  
[...]
> diff --git a/m4/groff.m4 b/m4/groff.m4
> index 142820737..f28332b8b 100644
> --- a/m4/groff.m4
> +++ b/m4/groff.m4
> @@ -971,10 +971,10 @@ AC_DEFUN([GROFF_PAGE], [
>      then
>        if grep -q ['^paperlength[      ]\+841890'] "$descfile"
>        then
> -     PAGE=A4
> +     PAGE=a4
>        elif grep -q ['^papersize[      ]\+[aA]4'] "$descfile"
>        then
> -     PAGE=A4
> +     PAGE=a4
>        fi
>        test -n "$PAGE" && whence=$descfile
>      fi
> @@ -1010,7 +1010,7 @@ AC_DEFUN([GROFF_PAGE], [
>        [*.[Uu][Ss]|*.[Cc][Aa])]
>          ;;
>        [*.[A-Za-z][A-Za-z])]
> -        PAGE=A4 ;;
> +        PAGE=a4 ;;
>        esac
>      done
>    fi
> @@ -1024,6 +1024,8 @@ AC_DEFUN([GROFF_PAGE], [
>      PAGE=letter
>    fi
>  
> +  PAGE=`echo "$PAGE" | tr A-Z a-z`
> +
>    AC_MSG_RESULT([$PAGE$message])
>    AC_SUBST([PAGE])
>  ])
> -- 
> 2.48.1
> 
> 
  This is the other way around.
The correct and official name of the paper format is A4, not a4, see for
example the standard ISO 216.

So the spelling a4 should be changed to A4.

For example in the font/dev*.am files, papersize should be A4 not a4.