FS#70805 - [python-sphinx] [fish] man pages installed to wrong directories

Attached to Project: Community Packages
Opened by Evangelos Foutras (foutrelis) - Wednesday, 12 May 2021, 01:39 GMT
Last edited by Evangelos Foutras (foutrelis) - Saturday, 15 May 2021, 14:11 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Johannes Löthberg (demize)
Levente Polyak (anthraxx)
Daniel M. Capella (polyzen)
Filipe Laíns (FFY00)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

Poked sphinx upstream about this here: https://github.com/sphinx-doc/sphinx/issues/9217

Hopefully they'll disable `man_make_section_directory` again to restore the old behavior.

This also affects llvm and probably several other packages. For fish in particular, these man pages are gone from usr/share/man:

-usr/share/man/man1/fish.1.gz
-usr/share/man/man1/fish_indent.1.gz
-usr/share/man/man1/fish_key_reader.1.gz

and are now in usr/share/fish/man/man1/1 along with the rest of built-in cmd man pages for fish, that used to reside in usr/share/fish/man/man1.

Additional info:
* package version(s): fish 3.2.2-1, python-sphinx 4.0.x
This task depends upon

Closed by  Evangelos Foutras (foutrelis)
Saturday, 15 May 2021, 14:11 GMT
Reason for closing:  Fixed
Additional comments about closing:  fish 3.2.2-2 rebuilt with fixed sphinx
Comment by loqs (loqs) - Thursday, 13 May 2021, 13:13 GMT
Not sure if this is the same issue but it has similar symptoms.
mpd fails during package with python-sphinx 4.0.1-2, downgrading to python-sphinx 3.5.2-1 and the build succeeds.
   mpd-0.22.6-4-x86_64-package.l... (5.1 KiB)
Comment by Evangelos Foutras (foutrelis) - Thursday, 13 May 2021, 13:36 GMT
Appears to be the same issue, yes. It tries to copy "doc/mpd.1" but sphinx 4 likely generates "doc/1/mpd.1".
Comment by Johannes Löthberg (demize) - Saturday, 15 May 2021, 10:57 GMT
I've reverted that commit in 4.0.1-3.

It's a bit weird that they made this change when the new behavior doesn't even achieve what it was meant to achieve, so it just needlessly breaks things.

Loading...