Summary: | media-libs/freetype:2 freetype-config broken under some locales | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Julian Ospald <hasufell> |
Component: | New packages | Assignee: | Gentoo Fonts Team <fonts> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://savannah.nongnu.org/bugs/index.php?38227 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 437488 | ||
Attachments: |
freetype-config.diff
freetype-config.diff freetype-config.diff |
Created attachment 337454 [details, diff]
freetype-config.diff
Created attachment 337622 [details, diff] freetype-config.diff according to some voices (such as https://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=f2dbe03dccc95f41429d60e4221b02fc0f112cc4 ) it seems that "[:alnum:]" is also locale-dependant, so forcing LC_ALL=C for sed seems to be a safer method upstream fixed it slightly different http://git.savannah.gnu.org/cgit/freetype/freetype2.git/commit/?id=3b1f206b04a05f07a86944669e36e67b0145e60b Wouldn't LC_COLLATE=C be enough? IMO it's safer to force LC_ALL instead of running scripts under a mixed locale setup that's what autoconf does and kernel hackers as well anyway, the patch is upstream, comment there if you think it's wrong https://savannah.nongnu.org/bugs/index.php?38227 I will fix this in 2 weeks if the maintainer does not respond. (In reply to comment #7) > I will fix this in 2 weeks if the maintainer does not respond. Please feel free to commit this at your convenience. I currently have too little time and too many things to do... + 05 Apr 2013; Ben de Groot <yngwin@gentoo.org> + +files/freetype-2.4.11-locale.patch, freetype-2.4.11.ebuild: + Add upstream patch to fix locale breakage (bug #454804). Punt unnecessary + epunt_cxx. |
Created attachment 337426 [details, diff] freetype-config.diff under locale et_EE.utf8 you will get: # freetype-config --prefix=/la --cflags -I--prefix/la/include/freetype2 -I--prefix/la/include # freetype-config --exec-prefix=/la --libs -L--exec-prefix/la/lib -lfreetype -lz -lbz2 attached patch was tested under different locales and also with dash