On Mon, Apr 01, 2013 at 07:32:07PM -0700, Colin Percival wrote: > On 04/01/13 19:24, James Turner wrote: > > On Mon, Apr 01, 2013 at 07:15:05PM -0700, Colin Percival wrote: > >> On 04/01/13 18:56, James Turner wrote: > >>> So I checked out spiped from svn then ran VERSION=1.3.0 make publish. > >>> I then tested building from there on an OpenBSD amd64 snapshot from 4/1. > >>> With the new addition of 200809L anywhere sys/sockets.h was added I also > >>> had to add sys/types.h. > >> > >> It seems that guenther's 1.82 in sys/sys/socket.h fixed the <sys/types.h> > >> inclusion but only with __BSD_VISIBLE, which is disabled due to spiped > >> having _POSIX_C_SOURCE set. What's the precise error you see? Perhaps > >> OpenBSD needs more #ifndef/#define/typedef/#endif clauses in socket.h > >> like it already has for socklen_t, sa_family_t, size_t, and ssize_t to > >> handle the !__BSD_VISIBLE case? > > > > /usr/include/sys/socket.h:189: error: expected specifier-qualifier-list > > before 'u_int8_t' > > /usr/include/sys/socket.h:207: error: expected specifier-qualifier-list > > before 'u_int8_t' > > Looks to me like you need to apply s/u_int/__uint/g to sys/socket.h -- can > you try that and see if the errors go away? > > -- > Colin Percival > Security Officer Emeritus, FreeBSD | The power to serve > Founder, Tarsnap | www.tarsnap.com | Online backups for the truly paranoid >
Yup, that fixed the problem. socket.h happens to reference u_int8_t twice and u_int64_t once. I'll see about getting them changed to __uint instead. -- James Turner
