[Ffmpeg-devel] FreeBSD 6.1 amd64 and current SVN

[Diego Biurrun]

On Sun, Oct 29, 2006 at 01:10:21AM +0300, Tatu Kilappa wrote:
> >> # if __FreeBSD__ >= 502100 || __FreeBSD__ == 6
> > Actually, this is GCC dependent I'm afraid, as __FreeBSD__ is in the
> > default defines of the compiler. Most likely the GCC 3.4 shipped with
> > base system... ...Using it to desume the version is not correct
> > anyway, instead one should use the __FreeBSD_version macro from
> > <osreldate.h>:
> > #define __FreeBSD_version 602100
> > this is from a 6.2-BETA2 insteall.
> 
> So how should this be fixed? Shall I just check if the system is
> FreeBSD (for example by checking that __FreeBSD__ is defined at all),
> and if yes, then include <osreldate.h>, possibly with #undef to
> __FreeBSD__ beforehand?
> 
> Personally, I'd just write Autotools -style cheks that produce output
> of the like #define HAVE_SYS_???_IOCTL_METEOR_H and we could forget the
> whole FreeBSD dependency stuff and focus on finding the headers, that's
> were the problem was anyway.
> 
> Tell your preferred approach and I'll make a patch and submit wherever
> you like, this mailing list perhaps?

Check for the functionality, not for the system.

Diego