search

teemtee / tmt

Test Management Tool
MIT License
80 stars 122 forks source link

ReST rendering for CLI forced CRB dependency #2690

Closed happz closed 7 months ago

happz commented 7 months ago

As reported by @martinhoyer, "after the introduction of "Simple ReST renderer for CLI help texts", tmt now relies on CRB repos on c9s and el9. Just checking this was evaluated/expected, as prior it was just EPEL"

# dnf install tmt-all --enablerepo=* |grep -i crb
 python3-docutils       noarch  0.16-6.el9         crb        1.6 M
 python3-pygments       noarch  2.7.4-4.el9        crb        2.0 M

The solution would be making these packages optional requirements, and provide dummy stand-ins to allow tmt work correctly without them.

psss commented 7 months ago

For the record, the crb repository was needed before (at some point) as well, see the installation instructions.

happz commented 7 months ago

I would be perfectly fine with keeping it as it is - my impression was CRB was introduced by my change, and it's not supposed to be there, hence the issue. But if it's acceptable, I'd be perfectly happy not doing anything :)

martinhoyer commented 7 months ago

For the record, the crb repository was needed before (at some point) as well, see the installation instructions.

Indeed, apparently crb was required for ruamel-yaml packages, up until el-9.3. On CentOS Stream 9, there is an updated ruamel-yaml.clib and both packages are now in AppStream repos. Since this is recent change, the docutils and pygments have not introduced dependency on CRB, it does look like a non-issue.

If tmt ever wants to get rid of CRB and be EPEL-only, there is a way :)

martinhoyer commented 7 months ago

Somehow I missed crb is a requirement for EPEL. Sorry for the noise.