Re: pkg/49986: print/xdvik fails to build on NetBSD-6.1.5/amd64

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

The following reply was made to PR pkg/49986; it has been noted by GNATS.

From: David Holland <dholland-pbugs%netbsd.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/49986: print/xdvik fails to build on NetBSD-6.1.5/amd64
Date: Fri, 19 Jun 2015 02:56:37 +0000

 On Fri, Jun 19, 2015 at 01:30:01AM +0000, Mark Davies wrote:
  > The following reply was made to PR pkg/49986; it has been noted by GNATS.
  > 
  > From: Mark Davies <mark%ecs.vuw.ac.nz@localhost>
  > To: gnats-bugs%netbsd.org@localhost
  > Cc: 
  > Subject: Re: pkg/49986: print/xdvik fails to build on NetBSD-6.1.5/amd64
  > Date: Fri, 19 Jun 2015 13:27:53 +1200
  > 
  >  On Fri, 19 Jun 2015 00:50:01 you wrote:
  >  >   >  "#if HAVE_XI21", so presumably that should be evaluating false for
  >  >   >  native
  >  >   >  X11 on 6.1.x and isn't?
  >  > 
  >  >  Yes, looks that way. The inputproto in -6 is 2.0.2.
  >  
  >  And the bit of configure that tests it is:
  >  
  >  [...]
  >  #include <X11/extensions/XInput2.h>
  >  int
  >  main ()
  >  {
  >  Display *DISP; int ndevices;
  >  XIDeviceInfo *info = XIQueryDevice(DISP, XIAllDevices, &ndevices);
  >    ;
  >    return 0;
  >  }
  >  
  >  Any suggestions for how to fix the test?
 
 .... that seems to be checking purely libXi, which is not even at 2.0
 yet, and ignoring inputproto.
 
 But then I've never understood what the hell xorg thought it was doing
 with these *proto packages.
 
 This should serve to check for XIScrollClass:
 
    #include <X11/extensions/XI2.h>
    #ifndef XIScrollClass
    bollocks
    #endif
    int main(void) { return 0; }
 
 Probably it's ok to stuff the #include and #ifdef parts of that into
 the existing test.
 
 -- 
 David A. Holland
 dholland%netbsd.org@localhost