[PATCH:macros] Update check for manpage section numbers to not rely on Solaris version

Matthieu Herrb matthieu at herrb.eu
Mon Nov 6 07:07:05 UTC 2017


On Sat, Nov 04, 2017 at 05:33:56PM -0700, Alan Coopersmith wrote:
> Check for a specific file instead of a specific set of versions from
> uname, to cope with manpage section alignment coming to 11.4 instead
> of 12.0.
> 
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

Hi,

I can't test it anymore, but it looks good to me.

Reviewed-by: Matthieu Herrb <matthieu at herrb.eu>

> ---
>  xorg-macros.m4.in | 31 +++++++++++++++++++------------
>  1 file changed, 19 insertions(+), 12 deletions(-)
> 
> diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in
> index 7935426..efce888 100644
> --- a/xorg-macros.m4.in
> +++ b/xorg-macros.m4.in
> @@ -114,6 +114,17 @@ AC_DEFUN([XORG_MANPAGE_SECTIONS],[
>  AC_REQUIRE([AC_CANONICAL_HOST])
>  AC_REQUIRE([AC_PROG_SED])
>  
> +case $host_os in
> +    solaris*)
> +        # Solaris 2.0 - 11.3 use SysV man page section numbers, so we
> +        # check for a man page file found in later versions that use
> +        # traditional section numbers instead
> +        AC_CHECK_FILE([/usr/share/man/man7/attributes.7],
> +                [SYSV_MAN_SECTIONS=false], [SYSV_MAN_SECTIONS=true])
> +        ;;
> +    *) SYSV_MAN_SECTIONS=false ;;
> +esac
> +
>  if test x$APP_MAN_SUFFIX = x    ; then
>      APP_MAN_SUFFIX=1
>  fi
> @@ -129,9 +140,8 @@ if test x$LIB_MAN_DIR = x    ; then
>  fi
>  
>  if test x$FILE_MAN_SUFFIX = x    ; then
> -    case $host_os in
> -	# Solaris 2.0 - 11 use SysV man page sections
> -	solaris2.?|solaris2.1[[01]])	FILE_MAN_SUFFIX=4  ;;
> +    case $SYSV_MAN_SECTIONS in
> +	true)				FILE_MAN_SUFFIX=4  ;;
>  	*)				FILE_MAN_SUFFIX=5  ;;
>      esac
>  fi
> @@ -140,9 +150,8 @@ if test x$FILE_MAN_DIR = x    ; then
>  fi
>  
>  if test x$MISC_MAN_SUFFIX = x    ; then
> -    case $host_os in
> -	# Solaris 2.0 - 11 use SysV man page sections
> -	solaris2.?|solaris2.1[[01]])	MISC_MAN_SUFFIX=5  ;;
> +    case $SYSV_MAN_SECTIONS in
> +	true)				MISC_MAN_SUFFIX=5  ;;
>  	*)				MISC_MAN_SUFFIX=7  ;;
>      esac
>  fi
> @@ -151,9 +160,8 @@ if test x$MISC_MAN_DIR = x    ; then
>  fi
>  
>  if test x$DRIVER_MAN_SUFFIX = x    ; then
> -    case $host_os in
> -	# Solaris 2.0 - 11 use SysV man page sections
> -	solaris2.?|solaris2.1[[01]])	DRIVER_MAN_SUFFIX=7  ;;
> +    case $SYSV_MAN_SECTIONS in
> +	true)				DRIVER_MAN_SUFFIX=7  ;;
>  	*)				DRIVER_MAN_SUFFIX=4  ;;
>      esac
>  fi
> @@ -162,9 +170,8 @@ if test x$DRIVER_MAN_DIR = x    ; then
>  fi
>  
>  if test x$ADMIN_MAN_SUFFIX = x    ; then
> -    case $host_os in
> -	# Solaris 2.0 - 11 use SysV man page sections
> -	solaris2.?|solaris2.1[[01]])	ADMIN_MAN_SUFFIX=1m ;;
> +    case $SYSV_MAN_SECTIONS in
> +	true)				ADMIN_MAN_SUFFIX=1m ;;
>  	*)				ADMIN_MAN_SUFFIX=8  ;;
>      esac
>  fi
> -- 
> 2.13.0
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel

-- 
Matthieu Herrb
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 793 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20171106/53f025d0/attachment.sig>


More information about the xorg-devel mailing list