Re: CVS commit: pkgsrc/mk/tools

[maya%netbsd.org@localhost]

On Fri, Apr 21, 2017 at 02:45:07PM +0000, Ryo ONODERA wrote:
> Index: pkgsrc/mk/tools/tools.NetBSD.mk
> diff -u pkgsrc/mk/tools/tools.NetBSD.mk:1.59 pkgsrc/mk/tools/tools.NetBSD.mk:1.60
> --- pkgsrc/mk/tools/tools.NetBSD.mk:1.59	Thu Nov 10 20:58:46 2016
> +++ pkgsrc/mk/tools/tools.NetBSD.mk	Fri Apr 21 14:45:07 2017
> @@ -1,4 +1,4 @@
> -# $NetBSD: tools.NetBSD.mk,v 1.59 2016/11/10 20:58:46 joerg Exp $
> +# $NetBSD: tools.NetBSD.mk,v 1.60 2017/04/21 14:45:07 ryoon Exp $
>  #
>  # System-supplied tools for the NetBSD operating system.
>  
> @@ -89,8 +89,8 @@ TOOLS_PLATFORM.nroff?=		/usr/bin/nroff
>  TOOLS_PLATFORM.openssl?=	/usr/bin/openssl
>  TOOLS_PLATFORM.patch?=		/usr/bin/patch
>  TOOLS_PLATFORM.pax?=		/bin/pax
> -.if empty(MACHINE_PLATFORM:MNetBSD-[^0-3].*-386) || \
> -	empty(MACHINE_PLATFORM:MNetBSD-[^0-3].*-x86_64)
> +.if !empty(MACHINE_PLATFORM:MNetBSD-[4-9].*-386) || \
> +	!empty(MACHINE_PLATFORM:MNetBSD-[4-9].*-x86_64)
>  TOOLS_PLATFORM.paxctl?=		/usr/sbin/paxctl
>  .endif
>  TOOLS_PLATFORM.printf?=		/usr/bin/printf
> 

Will something like
.if exists(/usr/sbin/paxctl)
TOOLS...
.endif
Work better?
At the very least, it shouldn't be limited to x86 platforms.