Bug 1629894 - Don't prepend PATH with /usr/libexec/python3-sphinx/
Summary: Don't prepend PATH with /usr/libexec/python3-sphinx/
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-sphinx
Version: 29
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
Assignee: Miro Hrončok
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: SPHINX2
TreeView+ depends on / blocked
 
Reported: 2018-09-17 15:11 UTC by sedrubal
Modified: 2019-03-17 12:37 UTC (History)
5 users (show)

Fixed In Version: python-sphinx-2.0.0~b1-1.fc31
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-17 12:37:19 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description sedrubal 2018-09-17 15:11:52 UTC
Description of problem:
The rpm package for sphinx adds a script to /etc/profile.d/ that loads the shell module python-sphinx. This module instruct shells to prepend /usr/libexec/python3-sphinx/ to PATH. I think, this is not how you should do it.

1. /usr/libexec/ is for "internal binaries that are not intended to be executed directly by users or shell scripts". I think sphinx-{build,apidoc,autogen,quickstart} are scripts, that are intended to be executed by users.
2. If every program would do this, PATH would be very bloated.
3. Due to the prepend, it is not possible to install another version of sphinx in a virtual environment, because /usr/libexec/python3-sphinx will be preferred before the venv bin dir.

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

python3-sphinx-1.7.6-1.fc29.noarch

Expected results:

Many other python packages put their binaries under /usr/bin/ and append a `-3` to the file name. Then they create a symlink to the binaries without the -3. Why didn't you do it the same?

Comment 1 Miro Hrončok 2019-03-17 12:37:19 UTC
This "feature" was removed from rawhide. Due to backwards compatibility concerns it won't be removed form older Fedoras, sorry.

See also https://fedoraproject.org/wiki/Changes/Sphinx2


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