Re: pkg/43811 (pkgsrc-2010Q2 - emulators/vice build (wrong patch))

[Matthew Mondor <mm_lists%pulsar-zone.net@localhost>]

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

From: Matthew Mondor <mm_lists%pulsar-zone.net@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/43811 (pkgsrc-2010Q2 - emulators/vice build (wrong patch))
Date: Sun, 5 Sep 2010 23:49:10 -0400

 On Sun,  5 Sep 2010 22:50:04 +0000 (UTC)
 Thomas Klausner <wiz%NetBSD.org@localhost> wrote:
 
 > The following reply was made to PR pkg/43811; it has been noted by GNATS.
 > 
 > From: Thomas Klausner <wiz%NetBSD.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: 
 > Subject: Re: pkg/43811 (pkgsrc-2010Q2 - emulators/vice build (wrong patch))
 > Date: Mon, 6 Sep 2010 00:49:10 +0200
 > 
 >  On Sun, Sep 05, 2010 at 06:40:04AM +0000, Matthew Mondor wrote:
 >  >  It appears that the interface change will last.
 >  >  
 >  >  Does the following patch work with XOrg 7.5+ (-current NetBSD)?
 >  
 >  No:
 >  In file included from ../../src/arch/unix/x11/xaw/videoarch.h:45,
 >                   from ../../src/arch/unix/videoarch.h:12,
 >                   from video-color.c:40:
 >  /scratch/emulators/vice/work/.buildlink/include/X11/Xmd.h:142: error:
 >  two or more data types in declaration specifiers
 >  /scratch/emulators/vice/work/.buildlink/include/X11/Xmd.h:142: error:
 >  two or more data types in declaration specifiers
 >  /scratch/emulators/vice/work/.buildlink/include/X11/Xmd.h:142:
 >  warning: useless type name in empty declaration
 >  
 >  Other ideas?
 
 Thanks for testing.  Hmm does that mean that the original pkgsrc-current
 patch-ad also causes this error on -current?  It also adds the Xmd.h
 include which has this odd part:
 
 #ifndef I_NEED_OS2_H
 typedef CARD8           BYTE;
 typedef CARD8           BOOL;
 #else
 #define BYTE    CARD8
 #define BOOL    CARD8
 #endif
 
 A difference I can see is that Xshm.h is included before shmproto.h in
 the original patch, this could also make a difference.  It would be
 much easier if I could actually perform those tests myself, but
 unfortunatly my -current laptop's drive died recently.  Next week I
 might be able to setup a new -current test box.
 
 Thanks,
 -- 
 Matt