[PATCH] Include "xlocale.h", not <xlocale.h>.

Corinna Vinschen vinschen@redhat.com
Wed Jun 12 08:05:00 GMT 2019

Hi Michael,

On Jun 11 19:43, Michael Haubenwallner wrote:
> libX11 does provide an <X11/Xlocale.h>, and while building libX11
> itself, they add their include/X11 to the compiler's include path.
> Now when newlib headers include <xlocale.h> on the case insensitive
> file system, the compiler does pick their Xlocale.h.
> So we better include "xlocale.h" in newlib headers instead, to prefer
> newlib xlocale.h from within newlib headers.

Commit 90e35b1eb3df renamed _locale.h to xlocale.h in March 2017 under
the assumption that we should provide the locale_t type in the same file
as on Linux, FreeBSD, and Darwin.

A few weeks later (June 2017), glibc removed the xlocale.h file in favor
of bits/types/locale_t.h, which shouldn't be included directly anyway.

For reference and the reasoning, see

Given that glibc and thus Linux stopped providing the non-standard
xlocale.h header, I'd favor reverting 90e35b1eb3df.



Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20190612/036b3938/attachment.sig>

More information about the Newlib mailing list