all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH]: fix python-matplotlib install-doc phase
@ 2015-02-18 10:52 Ricardo Wurmus
  2015-02-18 18:14 ` Andreas Enge
  0 siblings, 1 reply; 2+ messages in thread
From: Ricardo Wurmus @ 2015-02-18 10:52 UTC (permalink / raw)
  To: Guix-devel

[-- Attachment #1: Type: text/plain, Size: 491 bytes --]

Hi Guix,

attached is a patch to fix the install-doc phase of python-matplotlib.
Since UTF locales are no longer available by default the install-doc
phase fails as it depends on setting the locale to en_US.UTF-8.

The attached patch installs the locale before it is set, thereby fixing
the phase.

We likely won't be able to see the effect of this fix on hydra because
numpy fails to build, which is an input to matplotlib, but at least
matplotlib can now be compiled locally.

~~ Ricardo


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-matplotlib-install-UTF-8-locale.patch --]
[-- Type: text/x-patch, Size: 1309 bytes --]

From 31f9b8800de32cdbd1fd3d2266f5fbb41548dccc Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Wed, 18 Feb 2015 11:47:40 +0100
Subject: [PATCH] gnu: matplotlib: install UTF-8 locale.

* gnu/packages/python.scm (python-matplotlib): Install en_US.UTF-8 locale in
  install-doc phase to avoid encoding error.
---
 gnu/packages/python.scm | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 6fe524e..7dc7435 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -2259,7 +2259,11 @@ backend = GTK3Agg~%")))))
                   (info (string-append data "/info"))
                   (html (string-append doc "/html")))
              (with-directory-excursion "doc"
-               ;; Without setting this variable we get an encoding error.
+               ;; Install and set UTF-8 locale to avoid an encoding error.
+               (setenv "LOCPATH" (getcwd))
+               (system* "localedef" "--no-archive"
+                        "--prefix" (getcwd) "-i" "en_US"
+                        "-f" "UTF-8" "./en_US.UTF-8")
                (setenv "LANG" "en_US.UTF-8")
                ;; Produce pdf in 'A4' format.
                (substitute* (find-files "." "conf\\.py")
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH]: fix python-matplotlib install-doc phase
  2015-02-18 10:52 [PATCH]: fix python-matplotlib install-doc phase Ricardo Wurmus
@ 2015-02-18 18:14 ` Andreas Enge
  0 siblings, 0 replies; 2+ messages in thread
From: Andreas Enge @ 2015-02-18 18:14 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: Guix-devel

On Wed, Feb 18, 2015 at 11:52:41AM +0100, Ricardo Wurmus wrote:
> attached is a patch to fix the install-doc phase of python-matplotlib.
> Since UTF locales are no longer available by default the install-doc
> phase fails as it depends on setting the locale to en_US.UTF-8.

Temporarily this sounds good. In the long run, I still think we should
use a UTF-8 locale for building.

Andreas

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-02-18 18:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-18 10:52 [PATCH]: fix python-matplotlib install-doc phase Ricardo Wurmus
2015-02-18 18:14 ` Andreas Enge

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.