ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Vladimir D. Seleznev" <vseleznv@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] pyproject.toml and new rpm macroses
Date: Tue, 21 Jun 2022 18:22:25 +0300
Message-ID: <YrHiMfXhQG5Ap6l4@portlab> (raw)
In-Reply-To: <d34ad272-00fb-3cab-55ad-2ff11906fd20@altlinux.org>

On Tue, Jun 21, 2022 at 01:54:31PM +0300, Stanislav Levin wrote:
> 
> 
> 20.06.2022 19:03, Grigory Ustinov пишет:
> > 20.06.2022 17:40, Stanislav Levin пишет:
> >> Здравствуйте,

Hi, Stanislav!

> >> в сборочном задании 301407 [0] предложены RPM макросы и их реализация 
> >> для поддержки сборки и установки современных Python проектов 
> >> (setup.py-less).
> >>
> >> Больше информации на соответствующей wiki странице [1].

Спасибо за модернизацию сборки модулей Python!

> >> Замечания и предложения приветствуются.

Реализацию (пока) не смотрел, есть маленькое замечание по вики:
выделенные голубым цветом слова производят впечатление ссылок. Обычно
слова в данном контексте принято выделять моноширинным шрифтом.

> > Круто! Но предлагаю рассмотреть переименование %tox_check_pyproject в 
> > %pyproject_tox_check, чтобы все макросы были в одном стиле и люди не 
> > спотыкались о то, что в этом макросе два слова поменяны местами.
> Спасибо за отклик.
> 
> '_' - это разделение по области применяемости, в первую очередь это 
> макрос про tox, а не pyproject. Другими словами, один стиль в данном 
> случае - это "%tox_". Передаем опции не pyproject, а именно tox.

Вот это вопрос: стоит ли привязывать имена макросов к названиям
фреймворков, которые находятся "под капотом"? Ведь важно, что происходит
тестирование модуля, а не чем он тестируется.

> > 
> > В коммите 
> > https://git.altlinux.org/tasks/301407/gears/300/git?p=git;a=commitdiff;h=3415741d2820d9f6680ac0e037b5363bd06d162d 
> > 
> > присутствуют странные ручки _disable_test и _without_test которые в 
> > наших питоновских пакетах не используются. Может убрать?
> 
> Эти ручки - стандартные варианты отключения секции '%check' (можно 
> посмотреть реализацию RPM макроса "%__spec_check_pre"). В данном случае
> добавлены для проверки необходимости установки tox и co (выключен check 
> - не ставим). Ручки могут быть заданы не только через RPM specfile, но и 
> через опции командной строки rpmbuild.
> 
> > 
> > Можно ещё подумать о макросе типа %pyproject_distinfo, который можно 
> > будет добавлять в секцию %files.
> 
> Да, имеет смысл.
> Добавил, теперь можно написать так:
> -%python3_sitelibdir/%pypi_name-%version.dist-info/
> +%python3_sitelibdir/%{pyproject_distinfo %pypi_name}

-- 
   WBR,
   Vladimir D. Seleznev


  reply	other threads:[~2022-06-21 15:22 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-20 14:40 Stanislav Levin
2022-06-20 16:03 ` Grigory Ustinov
2022-06-21 10:54   ` Stanislav Levin
2022-06-21 15:22     ` Vladimir D. Seleznev [this message]
2022-06-20 17:52 ` Yuri Sedunov
2022-06-21 12:14 ` Ivan Zakharyaschev
2022-06-23  5:34 ` Michael Shigorin
2022-06-23 14:23 ` Stanislav Levin
2022-06-23 16:08 ` Dmitry V. Levin
2022-06-24 14:05   ` Stanislav Levin
2022-06-29 11:48 ` Stanislav Levin
2022-07-18  8:06 ` Stanislav Levin
2022-07-19 10:21   ` Ivan Zakharyaschev
2022-07-19 10:28     ` Anton Farygin
2022-07-19 10:57     ` Stanislav Levin
2022-07-19 11:09       ` Arseny Maslennikov
2022-07-19 12:30         ` Stanislav Levin
2022-07-19 11:12       ` Ivan Zakharyaschev
2022-07-20  8:00         ` Stanislav Levin
2022-07-20 15:08 ` Grigory Ustinov
2022-07-20 15:15   ` Stanislav Levin
2022-07-20 16:03     ` Grigory Ustinov
2022-07-20 20:13       ` Stanislav Levin
2022-07-20 21:43         ` Grigory Ustinov
2022-07-21  9:12           ` Stanislav Levin
2022-08-11 18:01     ` Vitaly Lipatov
2022-08-11 18:15       ` Yuri Sedunov
2022-08-12  7:22       ` Stanislav Levin
2022-07-23 12:06   ` Andrey Cherepanov

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=YrHiMfXhQG5Ap6l4@portlab \
    --to=vseleznv@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