Bug 41042 - [FR] python3-module-typogrify should not depend on django and jinja2
Summary: [FR] python3-module-typogrify should not depend on django and jinja2
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: python3-module-typogrify (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-01 18:18 MSK by Ivan A. Melnikov
Modified: 2021-10-01 19:17 MSK (History)
1 user (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 Ivan A. Melnikov 2021-10-01 18:18:06 MSK 
В README у python3-module-typogrify есть такой текст:

Installing Jinja or Django is only required if you
intend to use the optional template filters that are
included for those frameworks.

Я смотрел в код -- они действительно не нужны для нормальной работы.

При этом

$ rpm -qpR python3-module-typogrify-2.0.7-alt1.noarch.rpm  | fgrep -e django -e jinja
python3(django) < 0
python3(django.conf) < 0
python3(django.utils.encoding) < 0
python3(django.utils.safestring) < 0
python3(jinja2) < 0
python3(jinja2.exceptions) < 0

Предлагаю отровать эти зависимости. Тогда, например, гораздо меньше всего ненужного будет оказываться в сборочной среде у клиентов gi-docgen.
Comment 1 Ivan A. Melnikov 2021-10-01 18:18:57 MSK 
(In reply to Ivan A. Melnikov from comment #0)
> Предлагаю отровать эти зависимости.

Например, так:

http://git.altlinux.org/people/iv/packages/?p=python3-module-typogrify.git;a=commitdiff;h=a9b57af28ce20918b29bb9386a2dc6d183befc2d
Comment 2 Yuri N. Sedunov 2021-10-01 18:55:13 MSK 
Одобрю, если сможете оформить в src.rpm
$ girar-get-upload-method python3-module-typogrify
srpm
Comment 3 Ivan A. Melnikov 2021-10-01 19:05:48 MSK 
id=286272 locked=no shared=no fail_early=yes test_only=no repo=sisyphus owner=iv state=EPERM try=1 iter=1 age=0w build_time=2021-Oct-01/16:05:05
 100:srpm=python3-module-typogrify-2.0.7-alt2.src.rpm
Comment 4 Repository Robot 2021-10-01 19:13:54 MSK 
python3-module-typogrify-2.0.7-alt2 -> sisyphus:

 Fri Oct 01 2021 Ivan A. Melnikov <iv@altlinux> 2.0.7-alt2
 - get rid of django and jinja dependencies (closes: #41042)
Comment 5 Ivan A. Melnikov 2021-10-01 19:17:29 MSK 
Спасибо.