Re: [newbies] Удаление лишних зависимостей из python-пакета

Re: [newbies] Удаление лишних зависимостей из python-пакета

[Alexey Appolonov]

17.09.2024 23:24, Александр Лубягин пишет:
> Собрал пакет python-damegender:
> http://git.altlinux.ru/people/dfpl/packages/?p=python3-module-damegender.git;a=summary
> Но в тестах (tests) были зависимости от модулей SPARQL и python-perceval.
> Чтобы избежать dependency hell, я просто ненужные пару утилит (тестов) удалил на
> этапе %install.
> Допустимо ли такое при сборке пакетов? Не критичные для работы модуля тесты.

Как вы определяете критичность тестов?

Re: [newbies] Удаление лишних зависимостей из python-пакета

[Alexey Appolonov]

18.09.2024 02:16, Александр Лубягин пишет:
> 18.09.2024, 02:06, "Alexey Appolonov" <alexey@basealt.ru>:
>   >> Допустимо ли такое при сборке пакетов? Не критичные для работы модуля тесты.
>
>   > Как вы определяете критичность тестов?
> Отсутствие его в зависимостях самого питоновского модуля.

Не понял вас. Отсутсвие кого в зависимостях? Если в тесте
есть зависимость на некоторый модуль, на который нет зависимости
в основных модулях пакета (не являющихся тестами), то вы считаете
этот тест подлежащим удалению? Если так, то, в общем случае,
мне такой подход видится неверным.

> Например, среди тестов - вызов jupyter notebooks, которые рисуют графики через bokeh
> (такой случай встречается в tests/ питоновского пакета pandora, предназначенного
> для 3D-реконструции
> объекта по "плоским" фотографиям)
> Здесь я полагаюсь на то, "графические" тесты не выявляют ничего дополнительно
> существенного,
> и достаточно покрытия основным набором тестов.

Re: [newbies] Удаление лишних зависимостей из python-пакета

[Ivan A. Melnikov]

On Tue, Sep 17, 2024 at 11:24:49PM GMT, Александр Лубягин wrote:
>    Собрал пакет python-damegender:
>    http://git.altlinux.ru/people/dfpl/packages/?p=python3-module-damegender.git;a=summary
>     
>    Но в тестах (tests) были зависимости от модулей SPARQL и python-perceval.
>    Чтобы избежать dependency hell, я просто ненужные пару утилит (тестов)
>    удалил на этапе %install.
>    Допустимо ли такое при сборке пакетов? Не критичные для работы модуля
>    тесты.

Если речь идёт именно о юнит тестах, которые используются для проверки
работоспособности пакета, то зачем их вообще включать в пакет?
Их кто-то когда-нибудь будет запускать?

Тесты нужно запусать при сборке, в секции %check. В системе пользователя
пакета они не нужны. Я бы их удалил все, не избирательно.

Хотя я видел, что иногда тесты пакуют в отдельный подпакет.

-- 
  wbr,
    iv m.

Re: [newbies] Удаление лишних зависимостей из python-пакета

[Alexey Appolonov]

18.09.2024 06:55, Ivan A. Melnikov пишет:
> On Tue, Sep 17, 2024 at 11:24:49PM GMT, Александр Лубягин wrote:
>>     Собрал пакет python-damegender:
>>     http://git.altlinux.ru/people/dfpl/packages/?p=python3-module-damegender.git;a=summary
>>      
>>     Но в тестах (tests) были зависимости от модулей SPARQL и python-perceval.
>>     Чтобы избежать dependency hell, я просто ненужные пару утилит (тестов)
>>     удалил на этапе %install.
>>     Допустимо ли такое при сборке пакетов? Не критичные для работы модуля
>>     тесты.
> Если речь идёт именно о юнит тестах, которые используются для проверки
> работоспособности пакета, то зачем их вообще включать в пакет?
> Их кто-то когда-нибудь будет запускать?
>
> Тесты нужно запусать при сборке, в секции %check. В системе пользователя
> пакета они не нужны. Я бы их удалил все, не избирательно.
>
> Хотя я видел, что иногда тесты пакуют в отдельный подпакет.

Мне представилось, что речь идёт о src.rpm.

Общую рекомендацию могу такую дать:

Если очень хочется избавиться от какого-либо теста, то его отключение следует
осуществить отдельным commit-ом (с пояснением причин) и далее указать об этом
в changelog.