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

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On Jun 13 19:00, Yaakov Selkowitz wrote:
> On Wed, 2019-06-12 at 10:05 +0200, Corinna Vinschen wrote:
> > 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
> > https://sourceware.org/git/?p=glibc.git;a=commit;h=f0be25b6336d
> > 
> > Given that glibc and thus Linux stopped providing the non-standard
> > xlocale.h header, I'd favor reverting 90e35b1eb3df.
> 
> I concur, please proceed.  Any packages which still expect <xlocale.h>
> should have a patch in Fedora by now.

Thank you, done.


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAl0DVTQACgkQ9TYGna5E
T6AcABAAh7mZ6Jno9ULVa23ktj8UgV6Y+1X1XdFXUy2ESvEDXj1udLmnapzksMwo
CSKXJSnP+GPyNMJ2JErtL4sX8K2czTUqnq8aBVN/kwkV8EGPzIQMjIOO7kgRGfKO
mikuE/LmO7NJmx4N+btAycEAYJcsaC81snpxXn92VmzkW2LYZk1UAxJNt0Lo7qGG
Gfrp2rq+Jj9/XSGBDah5zfsmkRfJPAPaQtgrkxzHWlA1iHmotLIq+49RJ0kFlX1p
61+gNz0afNvxlzaGobFt96Fj0FywNoEkkV+cSLhvXh03kmHPAPaPuxfTgkeXFLZp
mfwEE0Kl6QEfCbyqKLM7c+EuHBtXUGbm8Uwh9gmYG6UVzwWSlgRmm2AXekFSTTWk
0zOQB2UMXRgcQitpAWFqdSKfcEdSKDKIHpNu+jxpAzjKKVZ90Xzcyj//gU/2Nx4/
bMvtgjvw2H+dzEvJVgo9qlKBkehwnl+H5adTI5MgMQYjC3hG2tVa2BU8n9L64o0g
cLd0q0vWi6+jyShhMptueIR5SjbQkqgzOXlK43Xqak0XBjogGLfGstKj22DcaVC+
JHJdQEWzI0mMafjwx+BU0oOBVqLa9mJMEdf05Fdb3fGWiXcoyf1BkcBUsLIhVX0E
jpqAZy1pa84svN1Wl5vss1v4CDOR1CqWUTqT+ejJmkZDs9cFSao=
=5cCi
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.