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

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

[Alexey Shabalin]

Всем привет!
С приехавшим в сизиф 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: хорошобы мантейнеру о таких изменениях предупреждать заранее.

-- 
Alexey Shabalin

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

[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: уместно?

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

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

[Lenar Shakirov]

30.05.2017 20:22, Alexey Shabalin пишет:
> Всем привет!
> С приехавшим в сизиф 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: хорошобы мантейнеру о таких изменениях предупреждать заранее.
>
Прошу прощения, не оповестил

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

[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

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

[Lenar Shakirov]

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
>

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

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

[Ivan Zakharyaschev]

[-- 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

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

[Vitaly Lipatov]

Ivan Zakharyaschev писал 14.6.17 22:49:
...>>> >  Кстати, не проще смержить обратно python-module-setuptools и
>>>  python-module-setuptools-tests?
>>> 
>>>  https://bugzilla.altlinux.org/29279
> 
> Это создаст меньшую гибкость при bootsrap-е/пересборке питоновских
> пакетов, потому что добавит лишних зависимостей, легко не отключаемых
> ручкой --without-check или т.п. (Не знаю, принципиально ли они
> тяжёлые. Посмотрим ниже на сравнение зависимостей.)
> 
> Ещё у меня есть сомнения о том, что этот пакет
> python3-module-setuptools-tests правильно назван. Ведь если он нужен
> для сборки других пакетов, то речь не идёт просто о тестах для самого
> пакета, как это принято для *-tests.
Прошу прощения, не успел заметить письмо в devel и уже отправил на 
сборку,
внеся -tests внутрь. Если он будет чем-то мешать, то там легко будет 
добавить
ручки для отключения упаковки тестов.

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

Кстати, есть сильно более новые setuptools (сейчас версия 18, а есть — 
30), кто-нибудь
одобрит рискнуть и собрать её?

-- 
С уважением,
Виталий Липатов,
Etersoft

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

[Ivan Zakharyaschev]

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

On Thu, 15 Jun 2017, Vitaly Lipatov wrote:

> Ivan Zakharyaschev писал 14.6.17 22:49:
> ...>>> >  Кстати, не проще смержить обратно python-module-setuptools и
>> > >   python-module-setuptools-tests?
>> > > 
>> > >   https://bugzilla.altlinux.org/29279
>>
>>  Это создаст меньшую гибкость при bootsrap-е/пересборке питоновских
>>  пакетов, потому что добавит лишних зависимостей, легко не отключаемых
>>  ручкой --without-check или т.п. (Не знаю, принципиально ли они
>>  тяжёлые. Посмотрим ниже на сравнение зависимостей.)
>>
>>  Ещё у меня есть сомнения о том, что этот пакет
>>  python3-module-setuptools-tests правильно назван. Ведь если он нужен
>>  для сборки других пакетов, то речь не идёт просто о тестах для самого
>>  пакета, как это принято для *-tests.
> Прошу прощения, не успел заметить письмо в devel и уже отправил на сборку,
> внеся -tests внутрь. Если он будет чем-то мешать, то там легко будет добавить
> ручки для отключения упаковки тестов.

Да ничего страшного, я думаю. Можно будет при необходимости подумать и о 
лучшем имени, и о том, как делить.

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

Скорее всего, наоборот не будет проблем. Потому что слияние setuptools и 
setuptools-tests было бы самым сильным решением из этих трёх вариантов, 
одним махом, насколько я понимаю.

> Кстати, есть сильно более новые setuptools (сейчас версия 18, а есть — 30), 
> кто-нибудь
> одобрит рискнуть и собрать её?

Лучше делать это в состоянии, когда в FTBFS почти нет пакетов python*. А 
то потом совсем не разгребёшь последствий.

-- 
Best regards,
Ivan