[MPlayer-dev-eng] [PATCH] Correctly detect sockaddr_storage on OpenBSD
Attila Kinali
attila at kinali.ch
Wed Aug 4 17:55:08 CEST 2010
On Tue, 03 Aug 2010 16:30:54 +0200
Diego Biurrun <diego at biurrun.de> wrote:
> On Tue, Aug 03, 2010 at 03:08:52PM +0100, Edd Barrett wrote:
> > <sys/socket.h> requires <sys/types.h> on OpenBSD.
>
> Then why does sys/socket.h not #include sys/types.h on OpenBSD?
>
> Again, I think you are patching the wrong side. Fix the header on
> OpenBSD once and you will make hundreds of packages work immediately
> instead of manually submitting patches to a trillion projects...
We have bigger hacks for other systems. I think that including
an additional headerfile is quite harmless. Our ranting here,
that *BSD has too many anachronisms wont help, as you cannot
change the way an old elephant walks on. You can only scare him,
which will get you stamped on.
Beside, socket(2) explicitly notes:
---schnapp---
POSIX.1-2001 does not require the inclusion of <sys/types.h>, and this
header file is not required on Linux. However, some historical (BSD)
implementations required this header file, and portable applications
are probably wise to include it.
---schnapp---
As i don't see any problems caused by including sys/types.h, we should
IMHO include it unconditionally.
Attila Kinali
--
If you want to walk fast, walk alone.
If you want to walk far, walk together.
-- African proverb
More information about the MPlayer-dev-eng
mailing list