Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 439108 - sys-firmware/seabios-1.7.0 LC_ALL=ru_RU.UTF-8 fails because ./tools/layoutrom.py screenscrapes objdump output
Summary: sys-firmware/seabios-1.7.0 LC_ALL=ru_RU.UTF-8 fails because ./tools/layoutrom...
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Doug Goldstein (RETIRED)
URL:
Whiteboard:
Keywords: EBUILD, PATCH
Depends on:
Blocks:
 
Reported: 2012-10-20 21:46 UTC by Anton Suvorov
Modified: 2012-10-21 14:50 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
new ebuild (seabios-1.7.0-r1.ebuild,1.33 KB, text/plain)
2012-10-20 21:50 UTC, Anton Suvorov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anton Suvorov 2012-10-20 21:46:10 UTC
sys-firmware/seabios-1.7.0 failed to build due to localization(on my system LC_ALL is set to ru_RU.UTF-8), the python script "./tools/layoutrom.py" screenscapes objdump that's the problem. Also emailed bug to developers.

Reproducible: Always

Steps to Reproduce:
1. set LC_ALL to somthing non-english
2. emerge seabios
3.
Actual Results:  
build failed

Expected Results:  
build successful

attaching new ebuild to merge into main tree
Comment 1 Anton Suvorov 2012-10-20 21:50:57 UTC
Created attachment 327016 [details]
new ebuild

ebuild just sets LC_ALL=C instead of LANG=C
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2012-10-21 14:31:10 UTC
Comment on attachment 327016 [details]
new ebuild

--- seabios-1.7.0.ebuild        2012-10-17 16:44:08.322520751 +0200
+++ -   2012-10-21 16:31:04.351216242 +0200
@@ -52,7 +52,7 @@
 
 src_compile() {
        if use amd64 || use x86 ; then
-               LANG=C emake out/bios.bin
+               LC_ALL=C emake out/bios.bin
        fi
 }
Comment 3 Doug Goldstein (RETIRED) gentoo-dev 2012-10-21 14:50:05 UTC
We explicitly say that setting LC_ALL is not supported due to the inability to override any other LC_ or LANG variables. Please read the localization guide:

http://www.gentoo.org/doc/en/guide-localization.xml#doc_chap3