ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Aleksey Avdeev <solo@solin.spb.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] A: rpm-build-python3-0.1.5-alt1
Date: Sun, 17 Feb 2013 04:48:14 +0400
Message-ID: <512028CE.9080503@solin.spb.ru> (raw)
In-Reply-To: <20130217001228.GA15615@altlinux.org>

[-- Attachment #1: Type: text/plain, Size: 1636 bytes --]

17.02.2013 04:12, Dmitry V. Levin пишет:
> On Sun, Feb 17, 2013 at 03:59:31AM +0400, Aleksey Avdeev wrote:
>> Приветствую.
>>
>>   Прошу пропустить в Сизиф rpm-build-python3-0.1.5-alt1 (собран с
>> --test-only, см.
>> <http://git.altlinux.org/tasks/90151/logs/events.1.1.log>, чистовая
>> сборка в процессе). Изменения в пакете:
>>
>> 1. Добавлен макрос %__python3_config, определённый на основе %__python3
>> и используемый для вызова python3-config:
>>
>> %__python3_config %__python3-config
>> %_python3_abiflags %(%__python3_config --abiflags 2>/dev/null || echo
>> unknown)
>> %_python3_extension_suffix %(%__python3_config --extension-suffix
>> 2>/dev/null || echo .so)
> 
> Изменения по сути правильные, но я не уверен, что сам по себе макрос
> %__python3_config имеет смысл, ведь можно использовать %__python3-config
> напрямую с тем же результатом.

  При наличии %__python3_config его можно переопределить в спеке, если
возникнет такая надобность. Вариант с прямым использованием
%__python3-config таким качеством не обладает.

> 
>> 2. Ужесточена зависимость на file (более ранние неумеют определять
>> скрипты python3):
>>
>> Requires: file >= 4.26-alt8
> 
> Изменение правильное, но commit message ужасный.  Помимо грамматики, текст
> просто сложно понять, не зная заранее, о чем идет речь.

  В инглише писатель я никакой: это комбинация из результат
<http://translate.google.com/> и commit message от file.

  Готов пириписать историю с любым приложенным commit message. Также не
возражаю, если это сделает кто нибудь знающий инглиш лучше меня.

-- 

С уважением. Алексей.



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 897 bytes --]

  reply	other threads:[~2013-02-17  0:48 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-16 23:59 Aleksey Avdeev
2013-02-17  0:12 ` Dmitry V. Levin
2013-02-17  0:48   ` Aleksey Avdeev [this message]
2013-02-17 10:36     ` Aleksey Avdeev
2013-02-18 10:36       ` Paul Wolneykien
2013-02-19 18:00         ` Aleksey Avdeev
2013-02-21 17:30           ` Aleksey Avdeev
2013-02-22 21:44     ` Dmitry V. Levin
2013-02-25  8:22       ` 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=512028CE.9080503@solin.spb.ru \
    --to=solo@solin.spb.ru \
    --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