Bug 54111 - подозрительные зависимости
Summary: подозрительные зависимости
Status: NEW
Alias: None
Product: Sisyphus
Classification: Development
Component: python3-module-imageio (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: alexvk@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-05-02 18:12 MSK by Кирилл Уницаев
Modified: 2025-05-09 21:58 MSK (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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/