Re: spiped 1.3.0 coming soon

[James Turner <james@calminferno.net>]

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