From 62e9e9a336ab5608405df8114f78c3cbb9dc3a39 Mon Sep 17 00:00:00 2001
From: Caleb Ristvedt <caleb.ristvedt@cune.org>
Date: Wed, 30 Jan 2019 17:00:32 -0600
Subject: [PATCH] gnu: python@2.7: Honor NIX_STORE.
Previously various python packages would fail to work unless the store they
were kept in was /gnu/store. This fixes that.
* gnu/packages/patches/python-2.7-site-prefixes.patch.in: New file that causes
python@2.7 to honor NIX_STORE at runtime or, if it isn't set, to use the
NIX_STORE available when it was built.
* gnu/packages/patches/python-2.7-site-prefixes.patch: Removed.
* gnu/packages/python.scm (python-2.7): generates a patch from
python-2.7-site-prefixes.patch.in at build-time and applies it.
(python-3.7): don't apply that patch.
(python2-minimal): inputs still need to include the patch utility and the
patch.
* gnu/local.mk: adjust patch name since it's been suffixed with ".in".
---
gnu/local.mk | 2 +-
...atch => python-2.7-site-prefixes.patch.in} | 9 ++++--
gnu/packages/python.scm | 29 ++++++++++++++++---
3 files changed, 32 insertions(+), 8 deletions(-)
rename gnu/packages/patches/{python-2.7-site-prefixes.patch => python-2.7-site-prefixes.patch.in} (81%)
@@ -1164,7 +1164,7 @@ dist_patch_DATA = \
%D%/packages/patches/python-2-deterministic-build-info.patch \
%D%/packages/patches/python-2.7-adjust-tests.patch \
%D%/packages/patches/python-2.7-search-paths.patch \
- %D%/packages/patches/python-2.7-site-prefixes.patch \
+ %D%/packages/patches/python-2.7-site-prefixes.patch.in \
%D%/packages/patches/python-2.7-source-date-epoch.patch \
%D%/packages/patches/python-3-deterministic-build-info.patch \
%D%/packages/patches/python-3-search-paths.patch \
similarity index 81%
rename from gnu/packages/patches/python-2.7-site-prefixes.patch
rename to gnu/packages/patches/python-2.7-site-prefixes.patch.in
@@ -5,7 +5,7 @@ site-packages (and .pth files) are searched.
--- Python-2.7.11/Lib/site.py 2016-10-17 23:44:51.930871644 +0200
***************
*** 65,70 ****
---- 65,82 ----
+--- 65,85 ----
# Prefixes for site-packages; add additional prefixes like /usr/local here
PREFIXES = [sys.prefix, sys.exec_prefix]
@@ -16,9 +16,12 @@ site-packages (and .pth files) are searched.
+ # This is necessary if the packages are not merged into a single
+ # `site-packages` directory (like when using `guix environment`) but
+ # listed in PYTHONPATH (like when running `guix build`).
++ guix_store = os.getenv("NIX_STORE")
++ if not guix_store:
++ guix_store = '@storedir@'
+ for p in sys.path:
-+ if p.startswith('/gnu/store/'):
-+ PREFIXES.append(p[:p.find('/', 44)]) # find first pathsep after hash
++ if p.startswith(guix_store):
++ PREFIXES.append(p[:p.find('/', 34 + len(guix_store))]) # find first pathsep after hash
+ del p
+
# Enable per user site-packages directory
@@ -76,6 +76,7 @@
(define-module (gnu packages python)
#:use-module ((guix licenses) #:prefix license:)
#:use-module (gnu packages)
+ #:use-module (gnu packages base)
#:use-module (gnu packages bash)
#:use-module (gnu packages compression)
#:use-module (gnu packages dbm)
@@ -106,7 +107,6 @@
"0x2mvz9dp11wj7p5ccvmk9s0hzjk2fa1m462p395l4r6bfnb3n92"))
(patches (search-patches "python-2.7-search-paths.patch"
"python-2-deterministic-build-info.patch"
- "python-2.7-site-prefixes.patch"
"python-2.7-source-date-epoch.patch"
"python-2.7-adjust-tests.patch"))
(modules '((guix build utils)))
@@ -138,7 +138,10 @@
(assoc-ref %outputs "out") "/lib"))
#:modules ((ice-9 ftw) (ice-9 match)
- (guix build utils) (guix build gnu-build-system))
+ (guix build utils) (guix build gnu-build-system)
+ (guix config))
+ #:imported-modules ,(cons '(guix config)
+ %gnu-build-system-modules)
#:phases
(modify-phases %standard-phases
(add-before
@@ -188,6 +191,19 @@
(utime file circa-1980 circa-1980)
#t))
#t)))
+ (add-after 'unpack 'apply-templated-patch
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((patch-template (assoc-ref inputs "site-patch"))
+ (patch "python-2.7-site-prefixes.patch"))
+ ;; generate actual patch
+ (copy-file patch-template patch)
+ (substitute* patch
+ (("@storedir@")
+ %store-directory))
+ ;; apply it (taken from (guix packages))
+ (invoke (string-append (assoc-ref inputs "patch") "/bin/patch")
+ "--force" "--no-backup-if-mismatch"
+ "-p1" "--input" patch))))
(add-after 'install 'remove-tests
;; Remove 25 MiB of unneeded unit tests. Keep test_support.*
;; because these files are used by some libraries out there.
@@ -244,7 +260,9 @@
("readline" ,readline)
("zlib" ,zlib)
("tcl" ,tcl)
- ("tk" ,tk))) ; for tkinter
+ ("tk" ,tk) ; for tkinter
+ ("site-patch" ,(search-patch "python-2.7-site-prefixes.patch.in"))
+ ("patch" ,patch)))
(native-inputs
`(("pkg-config" ,pkg-config)))
(native-search-paths
@@ -320,6 +338,7 @@ data types.")
(substitute-keyword-arguments (package-arguments python-2)
((#:phases phases)
`(modify-phases ,phases
+ (delete 'apply-templated-patch)
;; Unset SOURCE_DATE_EPOCH while running the test-suite and set it
;; again afterwards. See <https://bugs.python.org/issue34022>.
(add-before 'check 'unset-SOURCE_DATE_EPOCH
@@ -384,7 +403,9 @@ data types.")
;; is invoked upon 'make install'. 'pip' also expects 'ctypes' and thus
;; libffi.
(inputs `(("libffi" ,libffi)
- ("zlib" ,zlib)))))
+ ("zlib" ,zlib)
+ ("site-patch" ,(search-patch "python-2.7-site-prefixes.patch.in"))
+ ("patch" ,patch)))))
(define-public python-minimal
(package/inherit python
--
2.20.0