ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: devel@lists.altlinux.org
Subject: Re: [devel] python*-tests; was: Re: [cyber] I: Sisyphus-20180201 i586 beehive_status: +1151 -6 (1510)
Date: Mon, 5 Feb 2018 02:35:40 +0300
Message-ID: <20180204233540.GB22820@altlinux.org> (raw)
In-Reply-To: <alpine.LFD.2.20.1802040239060.7714@imap.altlinux.org>

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

On Sun, Feb 04, 2018 at 02:55:30AM +0300, Ivan Zakharyaschev wrote:
> On Sun, 4 Feb 2018, Dmitry V. Levin wrote:
> > On Sat, Feb 03, 2018 at 07:41:04AM +0300, Anton Farygin wrote:
> >> 03.02.2018 03:14, Dmitry V. Levin пишет:
> >>> On Fri, Feb 02, 2018 at 07:44:20AM +0300, Anton Farygin wrote:
> >>>> 02.02.2018 04:39, Alexei Takaseev пишет:
> >>>>> Добрый день!
> >>>>>
> >>>>> ----- Исходное сообщение -----
> >>>>>> От: "ALT beekeeper"<hiver@altlinux.org>
> >>>>>> Кому:sisyphus-cybertalk@lists.altlinux.org
> >>>>>> Отправлено: Пятница, 2 Февраль 2018 г 0:42:58
> >>>>>> Тема: [cyber] I: Sisyphus-20180201 i586 beehive_status: +1151 -6 (1510)
> >>>>> В Сизифе что-то большое рвануло?
> >>>> Когда-то давным давно в пакеты python пробралась паразитная сборочная
> >>>> зависимость на пакет python-module-setuptools-tests (кто-то в офисе в
> >>>> Москве предположил, что это могло произойти через некорректное
> >>>> использование buildreq)
> >>>>
> >>>> Сейчас этого пакета нет, но зависимость осталась. Чинится пересборкой с
> >>>> заменой сборочной зависимости на python-module-setuptools. И я
> >>>> предполагаю, что это исправят в ближайшее время скриптом (тот же кто и
> >>>> сломал).
> >>> Может, проще было бы добавить provides для обратной совместимости,
> >>> чем пересобирать сотни пакетов?
> >>
> >> Может быть и проще, но правильнее и честнее пересобрать.
> >
> > Почему правильнее?  Станут ли Пакеты, полученные в результате пересборки,
> > лучше прежних?
> 
> По идее (без рассмотрения конкретно этого случая), пакеты python*-tests 
> никому не должны быть нужны ни в runtime, ни при сборке (за исключением, 
> возможно, каких-то хитрых %check). (Просто тесты, которые можно запустить 
> в системе.)

По идее да.

> > Правильнее пересобирать все компилируемые пакеты после обновления
> > тулчейна, мы готовы к этому?
> 
> Убрать такую сборочную зависимость -- более существенное улучшение в 
> структуре репозитория Sisyphus, чем просто пересборка отдельных пакетов.

Убрать конкретно эту, или все python*-tests?

> По идее, их можно было бы сложить в отдельную компоненту все и не 
> нагружать обычные pkglists, если уж сложилась практика их паковать. Но 
> плодить компоненты, про которые никто не будет знать, тоже не хочется. 
> Можно было бы одну компоненту junk завести.
> 
> Помимо предположения о том, что какие-то хитрые %check могут использовать 
> чужие python*-tests, можно сказать, что пакеты *-checkinstall должны иметь 
> возможность использоввать python*-tests (не в виде исключения, а как 
> обычное дело).
> 
> (Отличие python*-tetss от *-checkinstall в том, что первые просто 
> содержат тексты тестов, а вторые их запускают при установке пакета.)
> 
> По теме: Provides: python*-tests в нормальном пакете был бы некрасивым 
> костылём (каким-то обманом, потому что вообще-то такая сущность не должна 
> исользоваться в BuildRequires).

Это всё хорошо, конечно, но какое это имеет отношение к данному
конкретному случаю?  Я не вижу, чтобы в результате автоматизированной
слепой пересборки с удалением атавистической сборочной зависимости
полученные пакеты были бы лучше прежних.  Нагрузку на сборочницу вижу,
трафик вижу, а в чём польза-то?


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

  reply	other threads:[~2018-02-04 23:35 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-02  1:39 ` [devel] " Alexei Takaseev
2018-02-02  4:44   ` Anton Farygin
2018-02-02  7:36     ` Dmitry V. Levin
2018-02-02 12:05     ` Levin Stanislav
2018-02-02 12:43       ` Igor Vlasenko
2018-02-03  0:14     ` Dmitry V. Levin
2018-02-03  4:41       ` Anton Farygin
2018-02-03 23:27         ` Dmitry V. Levin
2018-02-03 23:55           ` [devel] python*-tests; was: " Ivan Zakharyaschev
2018-02-04 23:35             ` Dmitry V. Levin [this message]
2018-02-05 11:45               ` Ivan Zakharyaschev
2018-02-05 12:12               ` Levin Stanislav
2018-02-05 12:42                 ` Alexey Shabalin
2018-02-06  4:54                   ` Anton Farygin
2018-02-05 22:53                 ` Leonid Krivoshein
2018-02-05 23:05                 ` Leonid Krivoshein
2018-02-05 23:44                   ` Alexey Shabalin
2018-02-06 22:22                     ` Leonid Krivoshein
2018-02-04  5:29           ` [devel] " Anton Farygin
2018-02-04 10:46           ` Leonid Krivoshein
2018-02-04 20:07                 ` Leonid Krivoshein
2018-02-02  6:18   ` Sergey Afonin
2018-02-02  6:20     ` Sergey Afonin
2018-02-02  6:51       ` Sergey Afonin
2018-02-02  6:40     ` Anton Farygin
2018-02-02  7:01       ` Sergey Afonin

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=20180204233540.GB22820@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