Subject: problems with make release on netbsd-2
To: None <tech-kern@netbsd.org>
From: Dave Tyson <Dave.Tyson@liverpool.ac.uk>
List: tech-kern
Date: 11/21/2005 14:36:14
After a recent CVS update of the netbsd-2 on both i386 and sparc architectures 
attempts to do a make release fail compiling  cap_mkdb.c. There is an earlier 
warning during the build of the tools - but the warning level then is not a 
show stopper. This module was recently pulled up to both the -2 and 3.0 
branches.

NetBSD gdtpc 2.1_STABLE NetBSD 2.1_STABLE (LIVSTD) #4: Mon Nov  7 19:59:59 GMT 
2005  root@gdtpc:/usr/obj/sys/arch/i386/compile/LIVSTD i386

./build.sh -u -U -x -T ../tools -O ../obj release
....
....
#    create  cap_mkdb/.depend
rm -f .depend
CC=/usr/src/../tools/bin/i386--netbsdelf-gcc /usr/src/../tools/bin/nbmkdep -d 
-f .depend -s .o\ .ln cap_mkdb.d
rm -f .gdbinit
echo "set solib-absolute-prefix /usr/src/../obj/destdir.i386" > .gdbinit
#   compile  cap_mkdb/cap_mkdb.o
/usr/src/../tools/bin/i386--netbsdelf-gcc -O2  -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Wno-sign-compare -Wno-traditional 
-Wno-uninitialized  -Werror    -nostdinc 
-isystem /usr/src/../obj/destdir.i386/usr/include  
-c    /usr/src/usr.bin/cap_mkdb/cap_mkdb.c
/usr/src/usr.bin/cap_mkdb/cap_mkdb.c: In function `db_build':
/usr/src/usr.bin/cap_mkdb/cap_mkdb.c:180: warning: passing arg 2 of `cgetnext' 
from incompatible pointer type

*** Failed target:  cap_mkdb.o
*** Failed command: /usr/src/../tools/bin/i386--netbsdelf-gcc -O2 -Wall 
-Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Wno-sign-compare 
-Wno-traditional -Wno-uninitialized -Werror -nostdinc 
-isystem /usr/src/../obj/destdir.i386/usr/include 
-c /usr/src/usr.bin/cap_mkdb/cap_mkdb.c
*** Error code 1

Stop.
nbmake: stopped in /usr/src/usr.bin/cap_mkdb

Stop.
nbmake: stopped in /usr/src

ERROR: Failed to make release
*** BUILD ABORTED ***

This looks like an issue with the call to cgetnext on line 180 with ifiles 
(declared as const char **)  vs the declaration of cgetnext in stdlib ( int  
cgetnext __P((char **, char **));

Easily fixed with a cast, but is anyone seeing this - Should I send-pr it?

-- 
=====================================================================
Computing Services Dept         Phone/Fax: 0151-794-3731/3759
The University of Liverpool     Email: dtyson@liv.ac.uk
Chadwick Tower, Peach Street    WWW:   http://www.liv.ac.uk/~dtyson
Liverpool L69 7ZF               Open Source O/S: www.netbsd.org
=====================================================================