On Sat, Feb 9, 2013 at 12:34 PM, Vinson Lee <v...@freedesktop.org> wrote:
> On Sat, Feb 9, 2013 at 11:38 AM, Matt Turner <matts...@gmail.com> wrote:
>> On Fri, Feb 8, 2013 at 7:19 PM, Vinson Lee <v...@freedesktop.org> wrote:
>>> Some platforms, such as Mac OS X, do not have librt.
>>
>> But Mac OS X (apparently from my quick Google search) doesn't clock_gettime.
>>
>> clock_gettime is called from two places:
>>   src/gallium/auxiliary/os/os_time.c
>>   src/mesa/drivers/dri/intel/intel_screen.c
>>
>> So this patch will just turn configure failures on platforms we do
>> care about into build failures. I don't think this is a good idea.
>>
>> Do we care about either of these pieces of code on Mac OS X? Actually,
>> I'm curious why you're doing testing on Mac OS X at all? src/glx/apple
>> hasn't even been converted to automake.
>>
>
> Would something similar to v1 of this patch be preferred? It avoided
> the clock_gettime check only for Mac OS X.

Yeah, I think that patch is okay. Sorry I missed it.

Thanks. Have an Acked-by: Matt Turner <matts...@gmail.com>

Matt
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to