Re: CVS commit: pkgsrc/mk/tools

[Roland Illig <roland.illig%gmx.de@localhost>]

Am 21.04.2017 um 17:49 schrieb David Holland:
> On Fri, Apr 21, 2017 at 03:20:44PM +0000, maya%netbsd.org@localhost wrote:
>  > > -.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
> 
> also, it should be -i386, not -386...

By the way, pkglint can not only check packages, it can also be used to
check for infrastructure files. It hasn't been tested extensively on
infrastructure files but is still useful in some cases. This case would
have been one.

$ pkglint mk/tools/tools.NetBSD.mk
WARN: mk/tools/tools.NetBSD.mk:92--93: The pattern "386" cannot match ...

Roland