Re: CVS commit: pkgsrc/sysutils/libpciaccess

[David Holland <dholland-pkgchanges%netbsd.org@localhost>]

On Tue, Apr 29, 2014 at 06:18:42AM +0000, OBATA Akio wrote:
 > Modified Files:
 >      pkgsrc/sysutils/libpciaccess: buildlink3.mk
 > Added Files:
 >      pkgsrc/sysutils/libpciaccess: builtin.mk
 > 
 > Log Message:
 > Add builtin support for libpciaccess.

On one of my -5 boxes this causes it to hallucinate the existence of a
nonexistent native libpciaccess. It doesn't even get as far as
checking with builtin.mk; instead this logic:

   +.if ${X11_TYPE} != "modular" && \
   +       !exists(${X11BASE}/lib/pkgconfig/pciaccess.pc) && \
   +       !exists(${X11BASE}/lib${LIBABISUFFIX}/pkgconfig/pciaccess.pc)
   +.include "../../mk/x11.buildlink3.mk"
   +.else

causes it to imagine that libpciaccess is provided via x11-links,
which it isn't.

Perhaps those exists() tests shouldn't have ! on them?

-- 
David A. Holland
dholland%netbsd.org@localhost