Re: [HACKERS] CVS -Tip compile issue -- FreeBSD 4.8

From: "Andrew Dunstan" <andrew(at)dunslane(dot)net>
To: <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <peter_e(at)gmx(dot)net>, <rbt(at)rbt(dot)ca>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] CVS -Tip compile issue -- FreeBSD 4.8
Date: 2003-06-17 10:21:45
Message-ID: 34301.199.90.235.43.1055859705.squirrel@www.dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


The man page for socket on Linux says you need both, but you can get away
without it.

It's easy enough to test if a platform does the sane thing:

[andrew(at)Thor andrew]$ echo '#include <sys/socket.h>' > blurfl.c
[andrew(at)Thor andrew]$ gcc -E -M blurfl.c
blurfl.o: blurfl.c /usr/include/sys/socket.h /usr/include/features.h \
/usr/include/sys/cdefs.h /usr/include/gnu/stubs.h \
/usr/include/sys/uio.h /usr/include/sys/types.h \
[snip]

Probably the best bet is just include sys/types.h just before any place
sys/socket.h is included, no? Should be harmless enough.

cheers

andrew

Tom Lane wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> Rod Taylor writes:
>>>> First thought that comes to mind is that <sys/socket.h> may require
>>>> something else to be included first --- maybe <sys/types.h> ?
>>>
>>> That did it. It's a little curious that sys/socket.h doesn't mention
>>> that anywhere though.
>
>> Because it shouldn't be so. Can you look in config.log what the real
>> reason of the failure was? Maybe it's a bug in FreeBSD.
>
> It's been true for years that <sys/socket.h> requires <sys/types.h> on
> some platforms. How do you think I knew what to recommend? There's no
> point in arguing about it, it's just a fact.
>
> regards, tom lane
>

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Curt Sampson 2003-06-17 10:34:50 Re: O_DIRECT in freebsd
Previous Message Oleg Bartunov 2003-06-17 10:02:27 Re: confused with encodings

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2003-06-17 12:35:27 Re: [HACKERS] CVS -Tip compile issue -- FreeBSD 4.8
Previous Message Shridhar Daithankar 2003-06-17 07:47:29 DBLink patch