ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Ivan A. Melnikov" <iv@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] U: Безусловный LC_ALL=C в /usr/lib/rpm/functions
Date: Fri, 22 Feb 2013 10:07:24 +0400
Message-ID: <CANR-Vvw=Ri1S-xtStJCchkDm5+keQqFN+g9V7MesQ53bpbb5BA@mail.gmail.com> (raw)
In-Reply-To: <5124EED4.6090005@solin.spb.ru>

2013/2/20 Aleksey Avdeev <solo@solin.spb.ru>:
> 20.02.2013 10:25, Ivan A. Melnikov пишет:
[...]
>
>> Мне кажется, надо учить
>> 096-bytecompile_python3.brp понимать -*- encoding: ... -*-, и
>> использовать utf-8 по умолчанию.
>
>   Кто возьмётся за отображение множества python`их лангов на системное?
> Насколько я знаю, они перекликаются, но не совпадают.

Это и не обязательно, имхо локаль вообще трогать не нужно, нужно
трогать /usr/lib/rpm/python3.compileall.py.

Вот traceback  из логов сборки:

Traceback (most recent call last):
  File "/usr/lib/rpm/python3.compileall.py", line 131, in <module>
    exit_status = bool(main())
  File "/usr/lib/rpm/python3.compileall.py", line 128, in main
    return compile_path(**dopt)
  File "/usr/lib/rpm/python3.compileall.py", line 93, in compile_path
    compile_all(join_prefix(prefix,item),[join_prefix(prefix,x) for x
in exclude],deep,skip_x,clean,prefix)
  File "/usr/lib/rpm/python3.compileall.py", line 81, in compile_all
    compile_all(p,exclude,deep-1,skip_x,clean,prefix)
  File "/usr/lib/rpm/python3.compileall.py", line 81, in compile_all
    compile_all(p,exclude,deep-1,skip_x,clean,prefix)
  File "/usr/lib/rpm/python3.compileall.py", line 84, in compile_all
    if not (skip_x and executable(p)) :
  File "/usr/lib/rpm/python3.compileall.py", line 60, in executable
    line = open(file).readline()
  File "/usr/lib/python3.3/encodings/ascii.py", line 26, in decode
    return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position
14: ordinal not in range(128)
error: Bad exit status from /usr/src/tmp/rpm-tmp.41412 (%install)

Посмотрим на /usr/lib/rpm/python3.compileall.py, строку 60

http://git.altlinux.org/people/solo/packages/?p=rpm-build-python3.git;a=blob;f=rpm-build-python3/python3.compileall.py;h=3c8fb4e3023d686d926b43744bc8a6ee3d0896ff;hb=0d8ba0d9dff3be076604aa59accd673cd7056f71#l57

  57 def executable(file):
  58     import stat
  59     if stat.S_IEXEC & (os.stat(file)[stat.ST_MODE]) :
  60         line = open(file).readline()
  61         if "#!" == line[:2] :
  62             return 1

То есть, LANG вообще не нужен, нужно передать правильный encoding в
стандартный питоновский open. Я подозреваю, что если заменить строку
60 на

  60         line = open(file, encoding='utf-8').readline()

то всё заработает прямо сейчас (проверить до завтра не могу, на работе
пока что не Alt), и танцы вокруг локали будут не нужны.

Ещё более правильным решением будет работать с файлами так, как это
делает сам питон (см tokenize.py, там есть и функция detect_encoding):

http://git.altlinux.org/people/solo/packages/python3.git?p=python3.git;a=blob;f=python3/Lib/tokenize.py;h=d669412db3eb0c7eb89c316acec89d4acb541380;hb=20543a67c862d487199f572f29e9b9a67aafdb06

Если никто не сделает раньше, попробую за выходные предложить решение.

Так что не трогайте локаль, она не виновата.

[...]
>
>   Но как, опять таки, показала практика, в стадо python пакетов
> затесалась минимум одна паршивая овца: UTF8 на этапе байткомпиляции
> нужен как минимум для python3-module-pycairo.

Внезапно, крамольная мысль: пропатчить pycairo при помощи sed
's/©/(c)/g'  и спокойно жить дальше с существующей локалью и
rpm-build-python3. Пока овца одна ;)

-- 
WBR,
Ivan A. Melnikov

  reply	other threads:[~2013-02-22  6:07 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-18 11:41 ` [devel] Fwd: python3-module-pycairo-1.10.1-alt3.git20120819: Sisyphus/x86_64 test rebuild failed REAL
2013-01-18 11:43   ` REAL
2013-01-18 11:11     ` Sergei Epiphanov
2013-01-18 12:18       ` REAL
2013-01-18 11:17     ` Dmitry V. Levin
2013-02-19 11:04   ` [devel] U: Безусловный LC_ALL=C в /usr/lib/rpm/functions (was: Fwd: python3-module-pycairo-1.10.1-alt3.git20120819: Sisyphus/x86_64 test rebuild failed) Aleksey Avdeev
2013-02-19 12:18     ` Ivan A. Melnikov
2013-02-19 12:41       ` [devel] U: Безусловный LC_ALL=C в /usr/lib/rpm/functions Aleksey Avdeev
2013-02-19 13:50     ` REAL
2013-02-19 14:02     ` Aleksey Avdeev
2013-02-19 19:25       ` Dmitry V. Levin
2013-02-19 19:46         ` Aleksey Avdeev
2013-02-19 20:02           ` Aleksey Avdeev
2013-02-20  6:25             ` Ivan A. Melnikov
2013-02-20 15:42               ` Aleksey Avdeev
2013-02-22  6:07                 ` Ivan A. Melnikov [this message]
2013-02-22  7:13                   ` REAL
2013-02-22  6:20                     ` Ivan A. Melnikov
2013-02-22  9:12                       ` Aleksey Avdeev
2013-02-22  9:32                         ` Aleksey Avdeev
2013-02-22 21:47                       ` Dmitry V. Levin
2013-02-22  6:21                     ` Ivan A. Melnikov
2013-02-22  9:40                   ` Aleksey Novodvorsky
2013-02-22 10:32                     ` Aleksey Avdeev
2013-02-20  5:02           ` REAL
2013-02-20  6:11             ` Aleksey Avdeev
2013-02-21 14:16     ` [devel] U: [rpm-build-python3] LC_ALL=en_US.utf8 для python3.compileall.py (was: Безусловный LC_ALL=C в /usr/lib/rpm/functions) Aleksey Avdeev
2013-02-21 17:27       ` [devel] U: [rpm-build-python3] LC_ALL=en_US.utf8 для python3.compileall.py Aleksey Avdeev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CANR-Vvw=Ri1S-xtStJCchkDm5+keQqFN+g9V7MesQ53bpbb5BA@mail.gmail.com' \
    --to=iv@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git