This is the mail archive of the cygwin-patches mailing list for the Cygwin project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Feb 29 13:19, mail@patrick-bendorf.de wrote: > Hi Corinna, > > Am 2016-02-29 11:33, schrieb Corinna Vinschen: > >Hi Patrick, > > > >On Feb 29 08:12, Patrick Bendorf wrote: > >>/winsup/ > >>* ccwrap: fix build with non-english locale set > > > >First of all, why fix it? Without at least a short explanation what you > >observe without this patch, this change seems arbitrary. > > > short explanation: after setting up cygwin on my systems the default locale > is set to "de_DE.UTF-8". this leads to ccwrap not picking up certain > "-isystem" arguments, which in turn leads to "stddef.h: no such file or > directory". this breaks the build process for systems having non english > locale. > > consider the following two pastebins from a system with an english locale > and mine. a whole bunch of "-isystem" can not be found on my system using > german locale. > http://pastebin.com/ip5L7dZY > http://pastebin.com/wZBc2cqr > > ccwrap scans the output of the first compiler invocation (line 21) for some > specific english output on and around line 43. > output of first invocation on german locale system: > http://pastebin.com/ZZzVGReh > [...] > >Ideally the above would test for the current build system being Cygwin > >and use "C.UTF-8" on Cygwin, "C" otherwise. > > > thanks for pointing that out. > i changed the patch to check uname -o for cygwin string and set the locale > to either C or C.UTF-8 Patch applied. Thanks, Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Maintainer cygwin AT cygwin DOT com Red Hat
Attachment:
signature.asc
Description: PGP signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |