Skip to content

Incorrect docutils library version resolved for sphinx when rebuilding eval images from scratch #9

Description

@alexlovchikovya

Describe the bug

We have to build eval images ourselves. We got situation, when golden patch gives about 91% of resolved issue comparing to 98% gived by sb-cli.
This looked strange to us, and we tried to find the root of problem.
After little investigation we saw that exist eval images for sphinx repo issue give use version of docutils 1.21.2, but version resolved during rebuild of images from scratch gives use 1.22.0, because repo contains such code in setup.py

install_requires = [
    'sphinxcontrib-applehelp',
    'sphinxcontrib-devhelp',
    'sphinxcontrib-jsmath',
    'sphinxcontrib-htmlhelp',
    'sphinxcontrib-serializinghtml',
    'sphinxcontrib-qthelp',
    'Jinja2>=2.3',
    'Pygments>=2.0',
    'docutils>=0.12',  <- here is a problem

This brings error to exist test in python source
from docutils.utils.roman import toRoman
cause 1.22 does not contains required code and seems incompatible with issue base commit in repo.

We temporary fixed it with adding here
line like "sed -i \"s/'docutils>=0.12'/'docutils>=0.12,<=0.21.2'/\" setup.py",
may be it is not appliable to all issues.

Steps/Code to Reproduce

rebuild eval images from scratch

Expected Results

same behaviour as on prebuild images from docker hub

Actual Results

falling of tests

System Information

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions