Re: CVS commit: pkgsrc/sysutils/libpciaccess

["OBATA Akio" <obata%lins.jp@localhost>]

On Thu, 08 May 2014 11:15:24 +0900, David Holland 
<dholland-pkgchanges%netbsd.org@localhost> wrote:

> 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?

Why?
NetBSD-5 has pciaccess.pc, and x11-links-0.88 and later support libpciaccess.

--
OBATA Akio / obata%lins.jp@localhost