Re: pkg/52431: Gargoyle 2011.1 (new package)

[Wes Frazier <wes.frazier%members.fsf.org@localhost>]

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

From: Wes Frazier <wes.frazier%members.fsf.org@localhost>
To: gnats-bugs%NetBSD.org@localhost, pkg-manager%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
 pkgsrc-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/52431: Gargoyle 2011.1 (new package)
Date: Thu, 27 Jul 2017 06:42:05 -0400

 On 07/27/17 05:10 AM, Thomas Klausner wrote:
 > The following reply was made to PR pkg/52431; it has been noted by GNATS.
 >
 > From: Thomas Klausner <tk%giga.or.at@localhost>
 > To: NetBSD bugtracking <gnats-bugs%NetBSD.org@localhost>
 > Cc:
 > Subject: Re: pkg/52431: Gargoyle 2011.1 (new package)
 > Date: Thu, 27 Jul 2017 11:08:05 +0200
 >
 >   On Wed, Jul 26, 2017 at 06:05:00PM +0000, wes.frazier%members.fsf.org@localhost wrote:
 >   > Gargoyle is an IF player that supports all the major interactive fiction formats.
 >   
 >   Thanks for packaging!
 >   
 >   I've improved the github part of it a bit, and commented out the
 >   LICENSE (since a partial and thus incorrect license is worse than no
 >   license mentioned). However, it does not build for me on
 >   NetBSD-8.99.1/amd64:
 >   
 >   Link build/netbsd.release/nitfol/nitfol
 >   build/netbsd.release/garglk/libgarglk.a(cgfref.o): In function `glk_fileref_create_temp':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/garglk/cgfref.c:140: warning: warning: tempnam() possibly used unsafely, use mkstemp() or mkdtemp()
 >   build/netbsd.release/nitfol/infix.o: In function `infix_get_val':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/infix.c:687: undefined reference to `get_var'
 >   build/netbsd.release/nitfol/decode.o: In function `decode':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/decode.c:92: undefined reference to `get_var'
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/decode.c:70: undefined reference to `get_var'
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/decode.c:71: undefined reference to `get_var'
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/decode.c:62: undefined reference to `get_var'
 >   build/netbsd.release/nitfol/decode.o:/scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/decode.c:55: more undefined references to `get_var' follow
 >   build/netbsd.release/nitfol/op_math.o: In function `op_dec_chk':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:98: undefined reference to `set_var'
 >   build/netbsd.release/nitfol/op_math.o: In function `op_inc':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:108: undefined reference to `get_var'
 >   build/netbsd.release/nitfol/op_math.o: In function `op_inc_chk':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:114: undefined reference to `get_var'
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:115: undefined reference to `set_var'
 >   build/netbsd.release/nitfol/op_math.o: In function `op_store':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:30: undefined reference to `set_var'
 >   build/netbsd.release/nitfol/op_math.o: In function `op_dec':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:92: undefined reference to `set_var'
 >   build/netbsd.release/nitfol/op_math.o: In function `op_inc':
 >   /scratch/nih/gargoyle/work/garglk-stable-2011.1/terps/nitfol/op_math.c:109: undefined reference to `set_var'
 >   
 >   I've added the modified package to wip/gargoyle. Can you please take a
 >   look at it?
 >   
 >   Instructions for using wip are at http://pkgsrc.org/wip/ and http://pkgsrc.org/wip/users/
 >   
 >   Thanks,
 >    Thomas
 >   
 Frustratingly enough the wip/gargoyle package builds fine for me on 
 NetBSD 7.1 / amd64.
 
 Does it have any errors before it enters the nitfol directory? What 
 version of gcc is being used in your version of NetBSD?