Re: PR/29425 CVS commit: pkgsrc/games/netmaj

[Klaus Heinz <k.heinz%maeracht.kh-22.de@localhost>]

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

From: Klaus Heinz <k.heinz%maeracht.kh-22.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: PR/29425 CVS commit: pkgsrc/games/netmaj
Date: Tue, 4 Mar 2008 00:25:30 +0100

 Johnny C. Lam wrote:
 >  "u_char" is not part of any standard, so it's not safe to use in cui.c.
 >  Define "u_char" to "unsigned char" in cui.c.  This should fix the the
 >  problem reported by Klaus Heinz in the follow-up to PR pkg/29425.
 
 The package builds now. Whether it does something useful I cannot
 say because the output in my xterm is not readable for me. Comments from
 someone able to read it would be necessary.
 
 Two small improvements in the pkgsrc Makefile:
 
  
 +INSTALL_MAKE_FLAGS+=   PERL=${PERL5:Q}
  INSTALL_TARGET=                install install-pf
  DOCDIR=                        ${PREFIX}/share/doc/netmaj
  
  pre-configure:
         cp ${FILESDIR}/configure.sh ${WRKSRC}/configure
         chmod +x ${WRKSRC}/configure
 +       chmod -x ${WRKSRC}/auto.pf
 
 
 This will stop warnings:
 
   => Checking file permissions in netmaj-2.0.7nb2
   warning:
   
/opt/pkgsrc-current/pkgsrc/games/netmaj/work/.destdir/opt/pkg/lib/netmaj/auto.pf:
 executable bit is set on non-executable file.
   warning:
   
/opt/pkgsrc-current/pkgsrc/games/netmaj/work/.destdir/opt/pkg/lib/netmaj/player_info:
 #! without a following slash.
   warning:
   
/opt/pkgsrc-current/pkgsrc/games/netmaj/work/.destdir/opt/pkg/lib/netmaj/player_info:
 executable bit is set on non-executable file.
   
 
 ciao
      Klaus