On 03-03-2009 02:29:21 -0600, Johan Hattne wrote:
> > Anyway, what is you host os version -- Tiger by chance? And the
> > corresponding error message would be nice to have (judging from the
> > patch I'd say missing _SC_NPROCESSORS_CONF).
> 
> Yes, it is.  I don't have a Leopard yet.  Can't reproduce the exact
> error message at the moment, but you're right--_SC_NPROCESSORS_CONF
> isn't defined.

Tiger here as well.

> > About the patch: I would like to see the "#elif
> > defined(_SC_NPROCESSORS_CONF)" case in front of the self-coded one (i.e.
> > right after the _WIN32 case like before), since it's the xvid-default
> > and working on 10.5.
> 
> Makes perfect sense to me. I wrongly assumed that you couldn't have
> __APPLE__ and __MACH__ and _SC_NPROCESSORS_CONF at the same time. Grobian?

Well, I found it a bit strange, but the best thing IMO would be to
configure check or #ifdef check for it.

Without configure-foo, probably something like

#ifndef _SC_NPROCESSORS_CONF
# if defined(__MACH__) || defined(__BSD__)
#  include <blah>
#  doit();
# else
#  error don't know how to get the number of processors, should Google for it
# endif
#else
# doit(_SC_NPROCESSORS_CONF);
#endif


-- 
Fabian Groffen
Gentoo on a different level