Re: [devel] Осторожно, Модерн! (python-module-pbr-2.0.0-alt1)

[Ivan Zakharyaschev <imz@altlinux.org>]

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


On Wed, 14 Jun 2017, Lenar Shakirov wrote:

>
> 14.06.2017 14:57, Lenar Shakirov пишет:
>>  30.05.2017 23:20, Michael Shigorin пишет:
>> >  On Tue, May 30, 2017 at 08:22:17PM +0300, Alexey Shabalin wrote:
>> > >  С приехавшим в сизиф python-module-pbr-2.0.0-alt1, скорее всего
>> > >  сломается сборка у множества пакетов, использующих pbr для сборки.
>> > >  С ошибкой
>> > >  ImportError: No module named test
>> > > 
>> > >  Для исправления нужно добавить
>> > >  BuildRequires: python-module-setuptools-tests
>> > >  и для python3
>> > >  BuildRequires: python3-module-setuptools-tests
>> > > 
>> > >  PS: хорошобы мантейнеру о таких изменениях предупреждать заранее.
>> >  Может, это ему в Requires: уместно?
>> > 
>>
>>  Кстати, не проще смержить обратно python-module-setuptools и
>>  python-module-setuptools-tests?
>>
>>  https://bugzilla.altlinux.org/29279

Это создаст меньшую гибкость при bootsrap-е/пересборке питоновских 
пакетов, потому что добавит лишних зависимостей, легко не отключаемых 
ручкой --without-check или т.п. (Не знаю, принципиально ли они тяжёлые. 
Посмотрим ниже на сравнение зависимостей.)

Ещё у меня есть сомнения о том, что этот пакет 
python3-module-setuptools-tests правильно назван. Ведь если он нужен для 
сборки других пакетов, то речь не идёт просто о тестах для самого пакета, 
как это принято для *-tests.

> Прошу посмотреть/проверить старших товарищей:
> http://git.altlinux.org/tasks/184227/

Кажется, тут самая тяжёлая зависимость, которая добавляется , это pytest. 
Оно ещё что-то за собой потянет. Точно не могу сказать, но 
предпочтительнее так не делать. А вот этот pbr, из-за которого начались 
проблемы -- он только для тестов используется, т.е. отключался бы ручкой 
--without-check? В любом случае, я бы сказал, что вернуть в него 
зависимость чуть предпочтительнее, чем укрупнять setuptools, а ещё 
предпочтительнее обновить buildreq во всех затронутых пакетах... правда, 
по-умному: под %if-ами. (Такую фичу в buildreq мы обсуждали, когда массово 
пересобирали питон, но не реализовали.) Правда, тут вопрос, кто когда это 
сможет сделать.

--- /tmp/.private/imz/compare_packages.lOBfzgbP5p/1     2017-06-14 
22:21:44.139002654 +0300
+++ /tmp/.private/imz/compare_packages.lOBfzgbP5p/2     2017-06-14 
22:21:44.144002483 +0300
@@ -7,6 +7,7 @@
  python3(collections) < 0
  python3(contextlib) < 0
  python3(ctypes) < 0
+python3(datetime) < 0
  python3(distutils) < 0
  python3(distutils.ccompiler) < 0
  python3(distutils.cmd) < 0
@@ -42,6 +43,8 @@
  python3(inspect) < 0
  python3(io) < 0
  python3(lib2to3.refactor) < 0
+python3(locale) < 0
+python3(logging) < 0
  python3(numbers) < 0
  python3(operator) < 0
  python3(os) < 0
@@ -51,6 +54,7 @@
  python3(platform) < 0
  python3(plistlib) < 0
  python3(posixpath) < 0
+python3(pytest) < 0
  python3(random) < 0
  python3(re) < 0
  python3(shlex) < 0
@@ -58,12 +62,13 @@
  python3(site) < 0
  python3(socket) < 0
  python3(stat) < 0
+python3(string) < 0
  python3(struct) < 0
  python3(subprocess) < 0
  python3(symbol) < 0
  python3(tarfile) < 0
  python3(tempfile) < 0
-python3(textwrap) < 0
+python3(threading) < 0
  python3(token) < 0
  python3(types) < 0
  python3(unicodedata) < 0



-- 
Best regards,
Ivan