Date:        Thu, 4 May 2017 14:07:33 +0000
    From:        "Roy Marples" <r...@netbsd.org>
    Message-ID:  <20170504140733.b327df...@cvs.netbsd.org>

  | Module Name:        src
  | Committed By:       roy
  | Date:               Thu May  4 14:07:33 UTC 2017
  | 
  | Modified Files:
  |     src/usr.bin/tic: tic.c
  | 
  | Log Message:
  | Remove __VALGRIND__ guard .... just clean up.


This breaks building on systems that have hcreate() but not hdestroy1()
(like say, NetBSD 6).   tic is (apparently) required as a host tool, the
build fails now as ...

#      link  tic/tic
cc -O -I/usr/obj/testing/tools/include/compat 
-I/release/testing/src/tools/compat -DHAVE_NBTOOL_CONFIG_H=1 
-D_FILE_OFFSET_BITS=64 
-I/release/testing/src/tools/tic/../../usr.bin/tic/../../lib/libterminfo   -o 
tic tic.lo compile.lo hash.lo -L/usr/obj/testing/tools/lib -lnbcompat -lrt -lz
tic.lo: In function `main':
tic.c:(.text+0xf5b): undefined reference to `hdestroy1'

Please revert this change, there doesn't seem to be a point to it
(it is certainly just a waste of time to race around freeing memory
and then immediately call exit() ... unless you are debugging memory
leaks, say with valgrind).

kre

Reply via email to