Bug 2005976 - python-sphinx-tabs FTBFS with python-docutils 0.17.1 in Rawhide
Summary: python-sphinx-tabs FTBFS with python-docutils 0.17.1 in Rawhide
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-sphinx-tabs
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Richard Shaw
QA Contact:
URL:
Whiteboard:
: 2006935 (view as bug list)
Depends On:
Blocks: F36FailsToInstall
TreeView+ depends on / blocked
 
Reported: 2021-09-20 15:25 UTC by Karolina Surma
Modified: 2021-10-02 11:56 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-10-02 11:56:13 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github executablebooks sphinx-tabs issues 140 0 None open Tests fail with docutils 0.17.1 on Fedora Rawhide 2021-09-22 12:54:35 UTC

Description Karolina Surma 2021-09-20 15:25:02 UTC
Description of problem:
python-sphinx-tabs FTBFS with python-docutils 0.17.1 in Rawhide.
python-sphinx-tabs pins docutils to 0.16 upstream which will cause FTI.


Version-Release number of selected component (if applicable):
3.1.0-4

How reproducible:
Always, when building python-sphinx-tabs with python-docutils 0.17.1

See full build logs: https://copr.fedorainfracloud.org/coprs/ksurma/docutils-0.17.1/package/python-sphinx-tabs/


Actual results:
== 3 failed, 6 passed, 1 skipped, 1 deselected, 3 warnings, 4 errors in 2.48s ==


_____________ ERROR at setup of test_conditional_assets[no_tabs2] ______________
<snip>
    @pytest.fixture(autouse=True)
    def auto_build_and_check(
        app,
        status,
        warning,
        check_build_success,
        get_sphinx_app_doctree,
        regress_sphinx_app_output,
        request,
    ):
        """
        Build and check build success and output regressions.
        Currently all tests start with this.
        Disable using a `noautobuild` mark.
        """
        if "noautobuild" in request.keywords:
            return
        app.build()
        check_build_success(status, warning)
        get_sphinx_app_doctree(app, regress=True)
>       regress_sphinx_app_output(app)
tests/conftest.py:38: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
app = <SphinxTestApp buildername='html'>, buildername = 'html'
filename = 'index.html', encoding = 'utf-8'
    def read(app, buildername="html", filename="index.html", encoding="utf-8"):
        content = get_sphinx_app_output(app, buildername, filename, encoding)
    
        if buildername == "html":
            soup = BeautifulSoup(content, "html.parser")
    
            # Remove output from ``pygments``, so that test only compares HTML of surrounding tabs
            for div in soup.find_all("div", {"class": "highlight"}):
                div.decompose()
    
            doc_div = soup.findAll("div", {"class": "documentwrapper"})[0]
            doc = doc_div.prettify()
    
        else:
            doc = content
>       file_regression.check(
            doc, extension="." + filename.split(".")[-1], encoding="utf8"
        )
E       AssertionError: FILES DIFFER:
E       /tmp/pytest-of-mockbuild/pytest-0/test_conditional_assets_no_tab1/test_build/test_conditional_assets_no_tabs2_.html
E       /tmp/pytest-of-mockbuild/pytest-0/test_conditional_assets_no_tab1/test_build/test_conditional_assets_no_tabs2_.obtained.html
E       HTML DIFF: /tmp/pytest-of-mockbuild/pytest-0/test_conditional_assets_no_tab1/test_build/test_conditional_assets_no_tabs2_.obtained.diff.html
E       --- 
E       +++ 
E       @@ -15,7 +15,7 @@
E             </li>
E            </ul>
E           </div>
E       -   <div class="section" id="fruits">
E       +   <section id="fruits">
E            <h1>
E             Fruits
E             <a class="headerlink" href="#fruits" title="Permalink to this headline">
E       @@ -50,8 +50,8 @@
E              </p>
E             </div>
E            </div>
E       -   </div>
E       -   <div class="section" id="luminaries">
E       +   </section>
E       +   <section id="luminaries">
E            <h1>
E             Luminaries
E             <a class="headerlink" href="#luminaries" title="Permalink to this headline">
E       @@ -78,8 +78,8 @@
E              </p>
E             </div>
E            </div>
E       -   </div>
E       -   <div class="section" id="code-tabs">
E       +   </section>
E       +   <section id="code-tabs">
E            <h1>
E             Code Tabs
E             <a class="headerlink" href="#code-tabs" title="Permalink to this headline">
E       @@ -178,8 +178,8 @@
E              </div>
E             </div>
E            </div>
E       -   </div>
E       -   <div class="section" id="group-tabs">
E       +   </section>
E       +   <section id="group-tabs">
E            <h1>
E             Group Tabs
E             <a class="headerlink" href="#group-tabs" title="Permalink to this headline">
E       @@ -242,7 +242,7 @@
E              </p>
E             </div>
E            </div>
E       -   </div>
E       +   </section>
E          </div>
E         </div>
E        </div>
tests/conftest.py:93: AssertionError

Expected results:
Tests pass, build is successful, package is installable.

Comment 1 Richard Shaw 2021-09-22 12:33:40 UTC
Interesting, I updated to 3.2.0 which already had pull 132 merged and added the other two patches and was able to built it locally but now I still get the error in my scratch build...

https://koji.fedoraproject.org/koji/taskinfo?taskID=76112822

Not really sure what I can do about this from a packaging perspective. Has it been reported upstream?

Comment 2 Richard Shaw 2021-09-22 12:36:32 UTC
Ok, duh, 2nd cup of coffee kicking in now, docutils is not in rawhide. Adding --enablerepo local shows the error.

Comment 3 Miro Hrončok 2021-09-22 16:52:43 UTC
*** Bug 2006935 has been marked as a duplicate of this bug. ***

Comment 4 Richard Shaw 2021-10-02 11:56:13 UTC
Upstream FINALLY provided a PR that fixed the issue, but it took so long it's causing problems with f35... Working on that now.


Note You need to log in before you can comment on or make changes to this bug.