ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
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:12:28 +0400
Message-ID: <20130217001228.GA15615@altlinux.org> (raw)
In-Reply-To: <51201D63.1020303@solin.spb.ru>

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

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
напрямую с тем же результатом.

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

Изменение правильное, но commit message ужасный.  Помимо грамматики, текст
просто сложно понять, не зная заранее, о чем идет речь.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2013-02-17  0:12 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 [this message]
2013-02-17  0:48   ` Aleksey Avdeev
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=20130217001228.GA15615@altlinux.org \
    --to=ldv@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