URL: https://github.com/SSSD/sssd/pull/5845
Title: #5845: sss-analyze: Fix self imports

stanislavlevin commented:
"""
@pbrezina, you may want to read:
https://docs.python.org/3/library/sysconfig.html#installation-paths
> Each scheme is itself composed of a series of paths and each path has a 
> unique identifier. Python currently uses eight paths:
stdlib: directory containing the standard Python library files that are not 
platform-specific.
platstdlib: directory containing the standard Python library files that are 
platform-specific.
platlib: directory for site-specific, platform-specific files.
purelib: directory for site-specific, non-platform-specific files.
include: directory for non-platform-specific header files.
platinclude: directory for platform-specific header files.
scripts: directory for script files.
data: directory for data files.

`sss_analyze.py` is the combo of module and script. Again, I suggest moving it 
either to sbin or libexec. If you don't want to expose the script to end-users, 
then it can be hidden under libexec. Later, on refactoring sssctl the no longer 
script can be safely removed.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5845#issuecomment-986736988
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to