ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Ivan Zakharyaschev <imz@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: [devel] check_*_pkg? Was: Re:  RFC: *-install-test
Date: Sun, 24 Dec 2017 13:08:45 +0300 (MSK)
Message-ID: <alpine.LFD.2.20.1712241243510.7714@imap.altlinux.org> (raw)
In-Reply-To: <20171224000448.GC13502@altlinux.org>

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


On Sun, 24 Dec 2017, Dmitry V. Levin wrote:

>> Да, сделаю. Надо только написать в описании пакета, что он существует
>> только для install-test в сборочнице и его не нужно ставить в реальную
>> систему. Мне не слишком нравится, что в Сизифе будет такие пакеты, но
>
> Мне кажется, что это, наоборот, очень перспективная тема.
>
> Для того, чтобы пакеты *-install-test не были установлены случайно, можно
> - индексировать их отдельно, по аналогии с *-debuginfo;
> - подключать этот индекс только при тестировании установки пакетов
>  *-install-test;
> - завести фиксированный формат %description таких пакетов,
>  и проверять их на стадии sisyphus_check.

Да, я тоже хотел такое начать делать! Не помню, как я назвал первую 
попытку (кажется: *-checkpkg), потом обдумывал, как это вообще может быть 
устроено в целом, и заодно родились схемы названий (Не буду утверждать, 
что они лучше):

1) check_*_pkg

2)
check_NAME0_pkg_with_NAME1
check_NAME0_pkg_with_NAME1_NAME2

или check_NAME0_NAME1_NAME2_pkgs

Второе -- на случаи, если захочется тестировать каждый раз, например, при 
обновлении NAME1, NAME2 (поставить зависимость на их EVR).

Разделитель другой (_), чтобы легче было читать NAME* обычных пакетов 
(вычленять), учитывая, что обычный разделитель -- это - (rpm-build так 
оформляет подпакеты).

Чтобы не спутатьс пакетами, просто пакующими тесты, я решил, что нужно 
использовать на слово test в названии, а check. (Чтобы ни люди не 
спутали, ни автоматика, которая их будет откладывать в отдельный 
подрепозиторий.) К тому же это больше похоже на команду "Check smth!", что 
отражает суть: поставить этот пакет == проверить что-то (это действие 
произойдёт в системе).

Обрамлять название пакета с двух сторон (check_*_pkg) мне захотелось, 
чтобы с большей вероятностью избежать совпадений с обычными пакетами.

Единственное, что мне в этой схеме не очень нравится -- это то, что они 
все будут иметь одинаковое начало, а это мешает сортировке, работе со 
спсиком (как со "словарём") пакетов отдельного подрепозитория...

Есть ещё мысль: следует иметь в виду разные возможности:

1) более сильные install-time проверки check_NAME0_NAME1_NAME2_pkgs с
    зависимостями на EVR всех пакетов. Будут вызывать unmet-ы при,
    требовать прохождения при сборке новых релизов сразу же.

2) более слабые (информационные) аналоги check_NAME0_NAME1_NAME2_pkgs с 
build-time проверками; они в BuildPreReq ставят проверяемые пакеты, и в 
%build делают их тесты. Тогда о нерохождении мы будем информироваться 
просто раз в сутки в рамках FTBFS от beehive. Схему именования я ещё не 
придумал.

-- 
Best regards,
Ivan

  parent reply	other threads:[~2017-12-24 10:08 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-21 15:48 [devel] RFC: ca-certificates a la Fedora Mikhail Efremov
2017-12-21 23:21 ` Mikhail Efremov
2017-12-29 15:38   ` Mikhail Efremov
2017-12-29 15:53     ` Dmitry V. Levin
2017-12-29 16:08       ` Dmitry V. Levin
2017-12-29 16:29         ` Mikhail Efremov
2018-01-09 23:22     ` Mikhail Efremov
2018-01-09 23:33       ` Mikhail Efremov
2018-01-10  0:41         ` Ivan Zakharyaschev
2017-12-22  0:36 ` Alexey Gladkov
2017-12-22  1:28   ` Dmitry V. Levin
2017-12-22  2:26     ` Alexey Gladkov
2017-12-22 16:33       ` Mikhail Efremov
2017-12-22 22:37         ` Dmitry V. Levin
2017-12-22 23:53           ` Alexey Gladkov
2017-12-22 23:58             ` Dmitry V. Levin
2017-12-23  0:30               ` Alexey Gladkov
2017-12-23  0:05         ` Alexey Gladkov
2017-12-23  0:22           ` Dmitry V. Levin
2017-12-23  0:35             ` Alexey Gladkov
2017-12-23  0:58               ` Dmitry V. Levin
2017-12-23  1:07                 ` Alexey Gladkov
2017-12-23 11:54                   ` Mikhail Efremov
2017-12-23 12:00                     ` Dmitry V. Levin
2017-12-23 23:58                       ` Mikhail Efremov
2017-12-24  0:04                     ` [devel] RFC: *-install-test Dmitry V. Levin
2017-12-24  1:05                       ` Alexey Gladkov
2017-12-24  1:09                         ` Dmitry V. Levin
2018-01-06 23:22                           ` Dmitry V. Levin
2018-01-07  0:23                             ` Alexey Tourbin
2018-01-07  0:46                               ` Dmitry V. Levin
2018-01-07  6:25                                 ` Alexey Tourbin
2018-01-07 10:49                                   ` Dmitry V. Levin
2017-12-24 10:08                       ` Ivan Zakharyaschev [this message]
2017-12-23 11:21             ` [devel] RFC: ca-certificates a la Fedora Mikhail Efremov
2017-12-23 11:58               ` Dmitry V. Levin
2017-12-23 14:54                 ` Alexey V. Vissarionov

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=alpine.LFD.2.20.1712241243510.7714@imap.altlinux.org \
    --to=imz@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