cschwan / sage-on-gentoo

(Unofficial) Gentoo Overlay for Sage- and Sage-related ebuilds
79 stars 26 forks source link

sphinx-5.2.patch fails (9.8.beta2) #720

Closed strogdon closed 1 year ago

strogdon commented 1 year ago
 * Applying sphinx-5.2.patch ...
patching file sage_docbuild/ext/multidocs.py
Hunk #1 FAILED at 146.
1 out of 1 hunk FAILED -- saving rejects to file sage_docbuild/ext/multidocs.py.rej

I haven't investigated further.

kiwifb commented 1 year ago

That patch should not be there. Sphinx 5.2 has been merged upstream.

kiwifb commented 1 year ago

And it looks like I left it. Which is strange considering that I had no issue on the vbraun branch.

kiwifb commented 1 year ago

Was looking at DEPEND rather than PATCHES :P it is not there. You should not see that.

strogdon commented 1 year ago

I don't understand. Which sphinx should be used - sphinx-5.2.3 was pulled in? And from sage_docbuild it appears the patch is to be applied.

    if has_version -r ">=dev-python/sphinx-5.2.0" ; then
        PATCHES+=( "${FILESDIR}/sphinx-5.2.patch" )
    fi
strogdon commented 1 year ago

I see, it appears (vbraun branch) that the above has_version if block should be removed.

kiwifb commented 1 year ago

OK, I don't know why that part was not merged. Looking for other differences before doing anything.

kiwifb commented 1 year ago

This is so weird. Only half the commit is merged. Fixing.

kiwifb commented 1 year ago

Fixed. sync and check please.

strogdon commented 1 year ago

This works.