[x264-devel] [PATCH] add support for FreeBSD in common/cpu.c

[Guillaume POIRIER]

Hello,

On Jan 4, 2008 4:20 AM, Igor Mozolevsky <igor at hybrid-lab.co.uk> wrote:
> On 03/01/2008, Guillaume POIRIER <poirierg at gmail.com> wrote:
> > Hello,
> >
> > On Dec 30, 2007 10:59 PM, Igor Mozolevsky <igor at hybrid-lab.co.uk> wrote:
> > > --- common/cpu.c.orig   2007-12-30 21:40:50.000000000 +0000
> > > +++ common/cpu.c        2007-12-30 21:44:20.000000000 +0000
> > > @@ -28,7 +28,7 @@
> > >  #ifdef SYS_BEOS
> > >  #include <kernel/OS.h>
> > >  #endif
> > > -#ifdef SYS_MACOSX
> > > +#if defined(SYS_MACOSX) || defined(__FreeBSD__)
> >
> > __FreeBSD__ is not defined by the configure script, which may harm
> > cross-compiling, or using compilers different from GCC.
> > The right way to do it, I believe, is to use SYS_FREEBSD instead.
> >
> > Please try attached patch, which is more correct IMHO.
>
> Fair point, I prefer to fix the real problem and let people in the
> know figure the fix into the whole system, otherwise it takes way too
> much time.

Ok, but you didn't confirm if the new patch worked for you or not...

:-))

Guillaume
-- 
A soldier will fight long and hard for a bit of colored ribbon.
 -- Napoleon Bonaparte