~ruther/guix-local

874188c0ab2955412519dc917402fefc6dd723fc — Maxim Cournoyer 5 years ago ac44173
aux-files: sitecustomize: Cleanup and add explanatory comments.

Fixes <https://issues.guix.gnu.org/50105>.

* gnu/packages/aux-files/python/sitecustomize.py: Add a comment explaining the
general idea, and use sys.prefix instead of sys.executable.

(major_minor): Use the unpacking operator (*) to provide the arguments.
(site_packages_prefix): Use os.path.join to form the path.
(python_site): Likewise.  Use sys.prefix instead of sys.executable.
(all_sites_raw): Split on os.path.pathsep.
(sys.path): Directly splice the result in the list.

Suggested-by: Hartmut Goebel <h.goebel@crazy-compilers.com>
Reported-by: Mathieu Othacehe <othacehe@gnu.org>
Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
1 files changed, 21 insertions(+), 7 deletions(-)

M gnu/packages/aux-files/python/sitecustomize.py
M gnu/packages/aux-files/python/sitecustomize.py => gnu/packages/aux-files/python/sitecustomize.py +21 -7
@@ 20,13 20,26 @@
import os
import sys

python_root = os.path.realpath(sys.executable).split('/bin/')[0]
major_minor = '{}.{}'.format(sys.version_info[0], sys.version_info[1])
site_packages_prefix = 'lib/python' + major_minor + '/site-packages'
python_site = python_root + '/' + site_packages_prefix
# Commentary:
#
# Site-specific customization for Guix.
#
# The program below honors the GUIX_PYTHONPATH environment variable to
# discover Python packages.  File names appearing in this variable that match
# a predefined versioned installation prefix are added to the sys.path.  To be
# considered, a Python package must be installed under the
# 'lib/pythonX.Y/site-packages' directory, where X and Y are the major and
# minor version numbers of the Python interpreter.
#
# Code:

major_minor = '{}.{}'.format(*sys.version_info)
site_packages_prefix = os.path.join(
    'lib', 'python' + major_minor, 'site-packages')
python_site = os.path.join(sys.prefix, site_packages_prefix)

try:
    all_sites_raw = os.environ['GUIX_PYTHONPATH'].split(':')
    all_sites_raw = os.environ['GUIX_PYTHONPATH'].split(os.path.pathsep)
except KeyError:
    all_sites_raw = []
# Normalize paths, otherwise a trailing slash would cause it to not match.


@@ 35,7 48,8 @@ matching_sites = [p for p in all_sites_norm
                  if p.endswith(site_packages_prefix)]

# Insert sites matching the current version into sys.path, right before
# Python's own site.
# Python's own site.  This way, the user can override the libraries provided
# by Python itself.
sys_path_absolute = [os.path.realpath(p) for p in sys.path]
index = sys_path_absolute.index(python_site)
sys.path = sys.path[:index] + matching_sites + sys.path[index:]
sys.path[index:index] = matching_sites