[Mesa-dev] [PATCH] [RFC] include/pipe: all major Linux libc support endian.h

[Erik Faye-Lund]

On Tue, Mar 18, 2014 at 3:18 PM, David Heidelberger
<david.heidelberger at ixit.cz> wrote:
> Dne 2014-03-18 13:43, Erik Faye-Lund napsal:
>
>> On Fri, Mar 14, 2014 at 7:57 PM, David Heidelberger
>> <david.heidelberger at ixit.cz> wrote:
>>>
>>> ---
>>>  src/gallium/include/pipe/p_config.h | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/src/gallium/include/pipe/p_config.h
>>> b/src/gallium/include/pipe/p_config.h
>>> index d603681..cd6f560 100644
>>> --- a/src/gallium/include/pipe/p_config.h
>>> +++ b/src/gallium/include/pipe/p_config.h
>>> @@ -130,7 +130,7 @@
>>>   * Endian detection.
>>>   */
>>>
>>> -#ifdef __GLIBC__
>>> +#if defined(PIPE_OS_LINUX) || defined(__GLIBC__)
>>>  #include <endian.h>
>>>
>>>  #if __BYTE_ORDER == __LITTLE_ENDIAN
>>
>>
>> Doesn't this do something else than the subject suggests? All *major*
>> linux libcs isn't the same as all linux libcs...
>
>
> In this moment I'm not aware any libc without endian.h on linux,
> where is Mesa supposed to build. If there is any, let me know and I'll add
> it to #ifdef.

Oh, OK. That's not what I read from the commit message, but thanks for
clearing that up. I'm not aware of any either. At least not on
platforms where Mesa with Gallium is a reasonable option.