Bug 54111

Summary:	подозрительные зависимости
Product:	Sisyphus	Reporter:	Кирилл Уницаев <fiersik>
Component:	python3-module-imageio	Assignee:	alexvk <alexvk>
Status:	RESOLVED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P5	CC:	alexvk72, grenka, lav, rider
Version:	unstable
Hardware:	x86_64
OS:	Linux

Description Кирилл Уницаев 2025-05-02 18:12:58 MSK

Имеет зависимость на python3(pytest), что выглядит очень странно

Comment 1 Alexander Kovalev 2025-05-08 21:25:10 MSK

(Ответ для Кирилл Уницаев на комментарий #0)
> Имеет зависимость на python3(pytest), что выглядит очень странно

В imageio.testing используется import pytest

Comment 2 Кирилл Уницаев 2025-05-08 21:35:57 MSK

(Ответ для Alexander Kovalev на комментарий #1)
> (Ответ для Кирилл Уницаев на комментарий #0)
> > Имеет зависимость на python3(pytest), что выглядит очень странно
> 
> В imageio.testing используется import pytest

Такое ощущение, что в пакете ему не место.
Файл только для тестов

Comment 3 Vitaly Lipatov 2025-05-08 22:05:14 MSK

(Ответ для Alexander Kovalev на комментарий #1)
> (Ответ для Кирилл Уницаев на комментарий #0)
> > Имеет зависимость на python3(pytest), что выглядит очень странно
> 
> В imageio.testing используется import pytest
А где используется imageio.testing ?

Comment 4 Alexander Kovalev 2025-05-09 18:16:26 MSK

(Ответ для Vitaly Lipatov на комментарий #3)
> (Ответ для Alexander Kovalev на комментарий #1)
> > (Ответ для Кирилл Уницаев на комментарий #0)
> > > Имеет зависимость на python3(pytest), что выглядит очень странно
> > 
> > В imageio.testing используется import pytest
> А где используется imageio.testing ?

Если в спеке сделать прогон тестов, то в них и будет использоваться. В пакетах, у которых есть зависимость на imageio (а их в Сизифе 4), использование testing я не заметил. Понадобится ли файл кому-то в будущем -- неизвестно. Как убрать файл из пакета, если в этом есть необходимость, я не знаю, он же прописан в setup.py, получается, сделать это непросто, требуется определенная аккуратность.

Comment 5 Vitaly Lipatov 2025-05-09 18:38:46 MSK

(Ответ для Alexander Kovalev на комментарий #4)
> (Ответ для Vitaly Lipatov на комментарий #3)
> > (Ответ для Alexander Kovalev на комментарий #1)
> > > (Ответ для Кирилл Уницаев на комментарий #0)
> > > > Имеет зависимость на python3(pytest), что выглядит очень странно
> > > 
> > > В imageio.testing используется import pytest
> > А где используется imageio.testing ?
> 
> Если в спеке сделать прогон тестов, то в них и будет использоваться. В
> пакетах, у которых есть зависимость на imageio (а их в Сизифе 4),
> использование testing я не заметил. Понадобится ли файл кому-то в будущем --
> неизвестно.
В общем случае известно, что не понадобится.

 Как убрать файл из пакета, если в этом есть необходимость, я не
> знаю, он же прописан в setup.py, получается, сделать это непросто, требуется
> определенная аккуратность.
Файл убрать просто: удалить его в секции install

Как бы то ни было, в пакете не должны быть упакованы тесты. К сожалению, для модулей python плохо проработан вопрос с тестами, поэтому файл недальновидно вписали в setup.py.

Comment 6 Alexander Kovalev 2025-05-09 21:58:43 MSK

Пересобрал пакет, требуется одобрение: https://git.altlinux.org/tasks/383649/

Comment 7 Repository Robot 2025-05-13 23:41:18 MSK

python3-module-imageio-2.37.0-alt2 -> sisyphus:

 Fri May 09 2025 Alexander Kovalev <alexvk@altlinux> 2.37.0-alt2
 - Removed testing.py (ALT #54111).