Re: [PATCH] Add and use sqlite-legacy-for-python

[Andreas Enge]

Hello,


On Fri, Feb 12, 2016 at 04:13:52PM -0800, Christopher Allan Webber wrote:
> +(define-public sqlite-legacy-for-python
> +(version "3.8.11.1")

in line with other cases where we package several versions, I would simply
call this sqlite-3.8.

Maybe it could be a simple "define" instead of "define-public", assuming
it would effectively be used only as in input inside the python module?
(Then it would need to be placed in the same module, of course.)

Andreas

Re: [PATCH] gnu: Add tclxml.

[Jan Nieuwenhuizen]

Leo Famulari writes:

> Thanks for the patch! Pushed as 511539ae811

Thanks!

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH] gnu: Add laby.

[Jan Nieuwenhuizen]

Andreas Enge writes:

>> +   (native-search-paths
>> +(list (search-path-specification
>> +   (variable "OCAMLPATH")
>> +   (files (list (string-append "lib/ocaml"))
>
> I have doubts about this search path. Normally, a search path is registered
> with the program that needs it; for instance, gcc declares a LIBRARY_PATH,
> and then whenever a program is installed that provides libraries, these are
> added to the environment variable.

> What do you think? If yes, this should be split into two patches.

Ah yes, that makes sense.

> So I suppose the search path declaration should really be in the ocaml
> package, and if lablgtk provides files matching the pattern, they will be
> added and available for laby.

Nice!  New patch set attached.

Greetings,
Jan

>From 73199e0de318a6b9c80060fe21f8d0667cb34df7 Mon Sep 17 00:00:00 2001
From: Jan Nieuwenhuizen 
Date: Sat, 13 Feb 2016 08:42:34 +0100
Subject: [PATCH 1/3] gnu: Update ocaml: set search path.

* gnu/packages/ocaml.scm (ocaml): Set OCAMLPATH for libraries like lablgtk.
---
 gnu/packages/ocaml.scm | 4 
 1 file changed, 4 insertions(+)

diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index 3baccf4..a584006 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -62,6 +62,10 @@
(base32
 "1qwwvy8nzd87hk8rd9sm667nppakiapnx4ypdwcrlnav2dz6kil3"
 (build-system gnu-build-system)
+(native-search-paths
+ (list (search-path-specification
+(variable "OCAMLPATH")
+(files (list (string-append "lib/ocaml"))
 (native-inputs
  `(("perl" ,perl)
("pkg-config" ,pkg-config)))
-- 
2.1.4

>From 9ff710f10bce9bf21b7b4b2f0703511c44343371 Mon Sep 17 00:00:00 2001
From: Jan Nieuwenhuizen 
Date: Thu, 11 Feb 2016 18:52:15 +0100
Subject: [PATCH 2/3] gnu: Update lablgtk: also build cmxa libraries.

* gnu/packages/ocaml.scm (lablgtk): Use "opt" to build *.cmxa files.
---
 gnu/packages/ocaml.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index a584006..8fc5c8a 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -477,6 +477,8 @@ provers.")
 (arguments
  `(#:tests? #f ; no check target
 
+   ;; opt: also install cmxa files
+   #:make-flags (list "all" "opt")
;; Occasionally we would get "Error: Unbound module GtkThread" when
;; compiling 'gtkThInit.ml', with 'make -j'.  So build sequentially.
#:parallel-build? #f
-- 
2.1.4

>From a1b9a3a164a2c743066c6fdf59c5970980579ff4 Mon Sep 17 00:00:00 2001
From: Jan Nieuwenhuizen 
Date: Sun, 7 Feb 2016 12:57:40 +0100
Subject: [PATCH 3/3] gnu: Add laby.

* gnu/packages/patches/laby-make-install.patch: New file.
* gnu/packages/patches/laby-make-png.patch: New file.
* gnu-system.am (dist_patch_DATA): Add them.
* gnu/packages/games.scm (laby): New variable.
---
 gnu-system.am|  2 ++
 gnu/packages/games.scm   | 45 
 gnu/packages/patches/laby-make-install.patch | 25 
 gnu/packages/patches/laby-make-png.patch | 27 +
 4 files changed, 99 insertions(+)
 create mode 100644 gnu/packages/patches/laby-make-install.patch
 create mode 100644 gnu/packages/patches/laby-make-png.patch

diff --git a/gnu-system.am b/gnu-system.am
index eba7c4e..19f1244 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -540,6 +540,8 @@ dist_patch_DATA =		\
   gnu/packages/patches/jasper-CVE-2016-1867.patch		\
   gnu/packages/patches/jbig2dec-ignore-testtest.patch		\
   gnu/packages/patches/kmod-module-directory.patch		\
+  gnu/packages/patches/laby-make-png.patch			\
+  gnu/packages/patches/laby-make-install.patch			\
   gnu/packages/patches/ldc-disable-tests.patch			\
   gnu/packages/patches/lftp-dont-save-unknown-host-fingerprint.patch \
   gnu/packages/patches/liba52-enable-pic.patch			\
diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
index 7eb0e7a..aa74665 100644
--- a/gnu/packages/games.scm
+++ b/gnu/packages/games.scm
@@ -14,6 +14,7 @@
 ;;; Copyright © 2015, 2016 Alex Kost 
 ;;; Copyright © 2015 Paul van der Walt 
 ;;; Copyright © 2015 Taylan Ulrich Bayırlı/Kammer 
+;;; Copyright © 2016 Jan Nieuwenhuizen 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -56,9 +57,11 @@
   #:use-module (gnu packages libcanberra)
   #:use-module (gnu packages libunwind)
   #:use-module (gnu packages haskell)
+  #:use-module (gnu packages imagemagick)
   #:use-module (gnu packages mp3)
   #:use-module (gnu packages image)
   #:use-module (gnu packages ncurses)
+  #:use-module (gnu packages ocaml)
   #:use-module (gnu packages python)
   #:use-module (gnu packages readline)
   #:use-module (gnu packages xorg)
@@ -1891,3 +1894,45 @@ and a game metadata scraper.")
 (description "The Emilia Pinball Project is a pinball simulator.  There
 are only two levels to play with, but they are very addictive.")
 (license licen

Re: [PATCH] gnu: Add laby.

[Andreas Enge]

On Fri, Feb 12, 2016 at 09:43:47PM +0100, Jan Nieuwenhuizen wrote:
> Ah yes it is inpired by the Nix package, it is store-specific.  Ocaml
> wants its "core" libraries all in the same prefix, store won't allow
> that.

Okay.

> When I hit this, I wondered if we have a way to import/convert packages
> and or what Guix's policy is on that.

There is a "guix import nix" command; I do not think it will go so far as
to also take care of patches, but it will create a template for a scheme
object from nixpkgs.

"git am" complained about whitespace issues with tabs, but I suppose these
are okay in make files :-)

> +(inputs
> + `(("camlp4" ,camlp4)
> +   ("m4" ,m4)
> +   ("ocaml" ,ocaml)))

I think all of these should be native-inputs; the first two are not referenced
in the result ("guix gc --references /gnu/store/..."), and ocaml is definitely
needed during compile time. I moved them and pushed the patch.

Thank you very much for your patience!

Andreas

Re: [PATCH] gnu: Add laby.

[Andreas Enge]

On Sat, Feb 13, 2016 at 09:11:17AM +0100, Jan Nieuwenhuizen wrote:
> Nice!  New patch set attached.

Very good, I pushed the first two. Concerning the commit message, it should
read "gnu: ocaml: ..." in case a package is modified.

> Subject: [PATCH 3/3] gnu: Add laby.
> * gnu/packages/patches/laby-make-install.patch: New file.
> * gnu/packages/patches/laby-make-png.patch: New file.
> * gnu-system.am (dist_patch_DATA): Add them.
> * gnu/packages/games.scm (laby): New variable.

Here I would not use a patch. The policy is not totally clear (and not
written up), but if an additional phase for installation is needed, I
would do it in a custom phase on the scheme side instead of patching the
makefile. See, for instance, the tinyxml package in xml.scm.

Maybe it would also warrant an upstream bug report?

Similarly for the make-png patch. Is this an upstream bug?

Andreas

PS: The game starts and shows the maze, it looks quite nice!
PPS: You do not happen to use unison? I think your previous patches would
 allow to build it with gtk support, and unlike me, you seem to know
 what you are doing...

Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

Another package name (this time we go down to slurm), another patch.
I also put the modules in alphabetic order when adding the license
name space. Hope that doesn't need a separate patch.

Pj.
>From ef998204cde384d35b605509be09b25170c28fb8 Mon Sep 17 00:00:00 2001
From: Pjotr Prins 
Date: Sat, 13 Feb 2016 09:45:30 +0100
Subject: [PATCH] gnu: Add slurm.

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu/packages/parallel.scm  | 67 --
 ...urm-wlm-configure-remove-nonfree-contribs.patch | 43 ++
 2 files changed, 105 insertions(+), 5 deletions(-)
 create mode 100644 gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch

diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append "https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") ".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch "slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch b/gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a4be2882cea4befee03591a7436 Mon Sep 17 00:00:00 2001
+From: Pjotr Prins 
+Date: Fri, 12 Feb 2016 12:43:33 +0100
+Subject: [PATCH] Remove contribs
+
+---
+ configure.ac | 20 
+ 1 file changed, 20 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index fedf354..e010732 100644
+--- a/configure.ac
 b/configure.ac
+@@ -438,26 +438,6 @@ dnl

Re: [PATCH] Update pandas, numpy, pysnptools, fastlmm

[Andreas Enge]

Hello,

it looks like the python2-numpy update has broken vigra:
   http://hydra.gnu.org/build/1008323/nixlog/1/tail-reload
I could confirm it on my own machine.

This is quite annoying, as vigra is an input to libreoffice.

Could one of you have a look?

Andreas

Re: Small texlive variant

[Andreas Enge]

Thanks for the comments!

On Thu, Feb 11, 2016 at 11:45:01AM +0100, Ludovic Courtès wrote:
> This comment is too indented and redundant with the code below, I think.

Okay, I kept a shorter and clearer comment.

> Make it an inner ‘define’, to reduce indentation.

Done.

> Also the docstring should be below the formal parameter list, and should
> be a sentence.

I took inspiration from other files; is it okay/required to start the
second line of a real docstring in the first column? I am attaching the patch
again mainly for that reason.

> > + (lambda (subdir exclude)
> I’d make exclude a keyword parameter, for clarity.

Well, I think it would make sense if the procedure were exported globally;
here it is really one with two parameters, so I am not following your advice.

I also made the texlive-texmf-minimal variable private. As mentioned quickly
on irc yesterday, I would like to do the same with texlive-texmf, to avoid
mistakes (someone mentioned they had installed texlive-texmf and texlive
into the same profile). As long as texlive-bin remains public, one can still
download texlive-bin (which takes a while to compile) from hydra and unpack
the data locally; the step from texlive-texmf to texlive is trivial and
consists only of adding symlinks.

What do you think?

Andreas

>From 481a18d13e977ec259b6ab3dfc4bd9369fba5abd Mon Sep 17 00:00:00 2001
From: Andreas Enge 
Date: Fri, 15 Jan 2016 16:00:47 +0100
Subject: [PATCH] gnu: Add texlive-minimal.

* gnu/packages/texlive.scm (texlive-texmf-minimal, texlive-minimal):
  New variables.
---
 gnu/packages/texlive.scm | 78 ++--
 1 file changed, 76 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/texlive.scm b/gnu/packages/texlive.scm
index 0b2dec4..a9673a0 100644
--- a/gnu/packages/texlive.scm
+++ b/gnu/packages/texlive.scm
@@ -24,6 +24,7 @@
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system trivial)
+  #:use-module (guix utils)
   #:use-module (gnu packages)
   #:use-module (gnu packages bash)
   #:use-module (gnu packages compression)
@@ -45,7 +46,9 @@
   #:use-module (gnu packages xorg)
   #:use-module (gnu packages xdisorg)
   #:use-module (gnu packages zip)
-  #:autoload   (gnu packages texinfo) (texinfo))
+  #:autoload   (gnu packages texinfo) (texinfo)
+  #:use-module (ice-9 ftw)
+  #:use-module (srfi srfi-1))
 
 (define texlive-extra-src
   (origin
@@ -226,7 +229,7 @@ This package contains the complete tree of texmf-dist 
data.")
(license (license:fsf-free "http://tug.org/texlive/copying.html";))
(home-page "http://www.tug.org/texlive/";)))
 
-(define-public texlive
+(define texlive
   (package
(name "texlive")
(version "2015")
@@ -284,6 +287,77 @@ This package contains the complete TeX Live distribution.")
(license (license:fsf-free "http://tug.org/texlive/copying.html";))
(home-page "http://www.tug.org/texlive/";)))
 
+
+;; texlive-texmf-minimal is a pruned, small version of the texlive tree,
+;; in particular dropping documentation and fonts.
+(define-public texlive-texmf-minimal
+  (package (inherit texlive-texmf)
+   (name "texlive-texmf-minimal")
+   (arguments
+(substitute-keyword-arguments
+ (package-arguments texlive-texmf)
+ ((#:modules modules)
+  `((ice-9 ftw)
+(srfi srfi-1)
+,@modules))
+ ((#:phases phases)
+  `(modify-phases ,phases
+ (add-after 'unpack 'prune
+   (lambda _
+ (define (delete subdir exclude)
+   "Delete all files and directories in SUBDIR except for those
+given in the list EXCLUDE."
+   (with-directory-excursion subdir
+ (for-each delete-file-recursively
+   (lset-difference equal?
+(scandir ".")
+(append '("." "..")
+exclude)
+ (with-directory-excursion "texmf-dist"
+   (for-each delete-file-recursively
+ '("doc" "source" "tex4ht"))
+   ;; Delete all subdirectories of "fonts", except for "tfm" and
+   ;; any directories named "cm".
+   (delete "fonts" '("afm" "map" "pk" "source" "tfm" "type1"))
+   (delete "fonts/afm" '("public"))
+   (delete "fonts/afm/public" '("amsfonts"))
+   (delete "fonts/afm/public/amsfonts" '("cm"))
+   (delete "fonts/map" '("dvips"))
+   (delete "fonts/map/dvips" '("cm"))
+   (delete "fonts/source" '("public"))
+   (delete "fonts/source/public" '("cm"))
+   (delete "fonts/tfm" '("public"))
+   (delete "fonts/type1" '("public"))
+   (delete "fonts/type1/public" '("amsfonts"))
+   (delete "fonts/type1/public/amsfonts" '("cm")))
+ #t))
+   (description
+ 

Re: [PATCH] gnu: Add slurm.

[Andreas Enge]

On Sat, Feb 13, 2016 at 09:59:40AM +0100, Pjotr Prins wrote:
> Another package name (this time we go down to slurm), another patch.

Actually, the name of the patch file: slurm-wml-patch should also
be changed... And it needs to be registered in gnu-system.am.

Some lines are way beyond 80 characters; this should be detected by
"guix lint".

> I also put the modules in alphabetic order when adding the license
> name space. Hope that doesn't need a separate patch.

I think that is fine.

However, your patch still does not apply on top of master:
Applying: gnu: Add slurm.
.git/rebase-apply/patch:87: trailing whitespace.
 (substitute* "./doc/html/shtml2html.py" 
.git/rebase-apply/patch:119: space before tab in indent.
 config.xml
.git/rebase-apply/patch:120: space before tab in indent.
 auxdir/Makefile
.git/rebase-apply/patch:141: space before tab in indent.
 doc/Makefile
.git/rebase-apply/patch:142: space before tab in indent.
 doc/man/Makefile
error: patch failed: gnu/packages/parallel.scm:2
error: gnu/packages/parallel.scm: patch does not apply
Patch failed at 0001 gnu: Add slurm.

I do not think that the whitespace is a real issue; did you rebase before
sending the patch? It is quite curious since the parallel.scm file has not
been changed yet this year.

Andreas

Re: [PATCH] gnu: Add laby.

[Jan Nieuwenhuizen]

Andreas Enge writes:

> "git am" complained about whitespace issues with tabs, but I suppose these
> are okay in make files :-)

:-)

>> +(inputs
>> + `(("camlp4" ,camlp4)
>> +   ("m4" ,m4)
>> +   ("ocaml" ,ocaml)))
>
> I think all of these should be native-inputs; the first two are not referenced
> in the result ("guix gc --references /gnu/store/..."), and ocaml is definitely
> needed during compile time. I moved them and pushed the patch.

Thanks!  guix gc --reference is useful next to lint.
Greetings,
Jan

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH] gnu: Add laby.

[Jan Nieuwenhuizen]

Andreas Enge writes:

> Very good, I pushed the first two. Concerning the commit message, it should
> read "gnu: ocaml: ..." in case a package is modified.

Ah okay...and Yay, thanks!

>> Subject: [PATCH 3/3] gnu: Add laby.
>> * gnu/packages/patches/laby-make-install.patch: New file.
>> * gnu/packages/patches/laby-make-png.patch: New file.
>> * gnu-system.am (dist_patch_DATA): Add them.
>> * gnu/packages/games.scm (laby): New variable.
>
> Here I would not use a patch. The policy is not totally clear (and not
> written up), but if an additional phase for installation is needed, I
> would do it in a custom phase on the scheme side instead of patching the
> makefile. See, for instance, the tinyxml package in xml.scm.

Yes, I see.  Good to know, thanks.

> Maybe it would also warrant an upstream bug report?

Yes, I have sent the patches upstream.

> Similarly for the make-png patch. Is this an upstream bug?

I think it's a problem with Guix's ocaml or gtk not rendering svg
images; so I wanted to first `see if png works' and then figure out
what's going on...So this could be temporary, depending on the
availability of svg rendering, not sure what to do here.

So...please advise.

Greetings,
Jan

> PS: The game starts and shows the maze, it looks quite nice!

:-)

Laby looks for programming languages installed and offer only those that
are available.  So while it does not depend on guile, gcc, ocaml etc...,
those packages make it more functional.

> PPS: You do not happen to use unison? I think your previous patches would
>  allow to build it with gtk support, and unlike me, you seem to know
>  what you are doing...

No, I have used SparkleShare until it became obvious there won't be a
headless version and I reverted to plain git.  I'll have a look.  Does
it use git for a backend?

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH] gnu: Add laby.

[Andreas Enge]

On Sat, Feb 13, 2016 at 03:09:59PM +0100, Jan Nieuwenhuizen wrote:
> I think it's a problem with Guix's ocaml or gtk not rendering svg
> images; so I wanted to first `see if png works' and then figure out
> what's going on...So this could be temporary, depending on the
> availability of svg rendering, not sure what to do here.
> So...please advise.

Since there is no terrible hurry, I would suggest to examine the svg rendering
issue first; if you do not find a solution in reasonable time, we can still
push the png patch.

> > PPS: You do not happen to use unison?
> No, I have used SparkleShare until it became obvious there won't be a
> headless version and I reverted to plain git.  I'll have a look.  Does
> it use git for a backend?

No, it just synchronises and does not allow to roll back. On the other hand,
it also works well with binary files (for instance, for synchronising films
from one machine to another, I would not want them to take space forever
in a git repository). It is essentially a two-way rsync.

Andreas

[PATCH] gnu: Add vcsh and perl-shell-command

[Christopher Baines]

Add vcsh (Version control system for $HOME), and perl-shell-command which is a
necessary input for running the tests.

[PATCH 2/3] gnu: Don't use guix build utils

[Christopher Baines]

As this provides a which procedure which conflicts with the which package
provided by gnu packages base.

* gnu/packages/version-control.scm (version-control): Don't use utils
---
 gnu/packages/version-control.scm | 1 -
 1 file changed, 1 deletion(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index e0626cb..721a284 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -37,7 +37,6 @@
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system python)
   #:use-module (guix build-system trivial)
-  #:use-module (guix build utils)
   #:use-module (gnu packages apr)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages asciidoc)
-- 
2.7.0

[PATCH 1/3] gnu: Add perl-shell-command

[Christopher Baines]

This is required for the tests for vcsh.

* gnu/packages/perl.scm (perl-shell-command): New variable.
---
 gnu/packages/perl.scm | 24 
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 0e63aa9..5146828 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6277,4 +6277,28 @@ really be high enough to warrant the use of a keyword, 
and the size so small
 such that being individual extensions would be wasteful.")
 (license (package-license perl
 
+(define-public perl-shell-command
+  (package
+(name "perl-shell-command")
+(version "0.06")
+(source
+  (origin
+(method url-fetch)
+(uri (string-append
+   "mirror://cpan/authors/id/F/FL/FLORA/Shell-Command-"
+   version
+   ".tar.gz"))
+(sha256
+  (base32
+"1lgc2rb3b5a4lxvbq0cbg08qk0n2i88srxbsz93bwi3razpxxr7k"
+(build-system perl-build-system)
+(home-page
+  "http://search.cpan.org/dist/Shell-Command";)
+(synopsis
+  "Cross-platform functions emulating common shell commands")
+(description
+  "Thin wrapper around ExtUtils::Command.  See ExtUtils::Command for a
+ description of available commands.")
+(license (package-license perl
+
 ;;; END: Core module overrides
-- 
2.7.0

[PATCH 3/3] gnu: Add vcsh

[Christopher Baines]

* gnu/packages/version-control.scm (vcsh): New variable.
---
 gnu/packages/version-control.scm | 38 ++
 1 file changed, 38 insertions(+)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 721a284..fd2056d 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -1058,3 +1058,41 @@ Mercurial, Bazaar, Darcs, CVS, Fossil, and Veracity.")
  "This package allows you to use your hubic account as a \"special
 repository\" with git-annex.")
 (license gpl3+)))
+
+(define-public vcsh
+  (package
+(name "vcsh")
+(version "1.20151229")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append
+ "https://github.com/RichiH/vcsh/archive/v";
+ version ".tar.gz"))
+   (file-name (string-append name "-" version ".tar.gz"))
+   (sha256
+(base32 "1ym3swkh738c3vciffvlr96vqzhwmzkb8ajqzap8f0j9n039a1mf"
+(build-system gnu-build-system)
+(arguments
+ `(#:phases (modify-phases %standard-phases
+  (delete 'configure)
+  (delete 'check)
+  (delete 'build))
+   #:make-flags (list (string-append "PREFIX=" %output
+(native-inputs
+ `(("which" ,which)))
+(inputs
+ `(("perl-shell-command" ,perl-shell-command)
+   ("perl-test-most" ,perl-test-most)))
+(propagated-inputs
+ `(("git" ,git)))
+(home-page "https://github.com/RichiH/vcsh";)
+(synopsis "Version control system for $HOME")
+(description
+ "vcsh provides the @code{vcsh} command, which allows you to maintain
+several Git repositories in one single directory.  This can be useful when
+using version control for files in your home directory, as it allows for
+the separation of the files in to different repositories, for example you
+may keep your bash configuration in a separate Git repository from your
+emacs configuration.")
+(license gpl2+)))
-- 
2.7.0

Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

On Sat, Feb 13, 2016 at 11:28:25AM +0100, Andreas Enge wrote:
> Some lines are way beyond 80 characters; this should be detected by
> "guix lint".

guix lint has not been working for me:

./pre-inst-env guix lint slurm
warning: failed to install locale: Invalid argument
filtered-port: failed to execute ' -dc ': No such file or directory
Backtrace:
In unknown file:
   ?: 19 [apply-smob/1 #]
In ice-9/boot-9.scm:
  63: 18 [call-with-prompt prompt0 ...]
In ice-9/eval.scm:
 432: 17 [eval # #]
In ice-9/boot-9.scm:
(...)
In guix/utils.scm:
 255: 5 [call-with-decompressed-port gzip ...]
In guix/cve.scm:
 143: 4 [# #]
 133: 3 [xml->vulnerabilities #]
In ice-9/boot-9.scm:
 105: 2 [# 
parser-error ...]
  65: 1 [abort-to-prompt catch7 parser-error ...]
In unknown file:
   ?: 0 [scm-error misc-error #f "~A ~S" ("decompressed-port failure" (6495)) 
#f]

ERROR: In procedure scm-error:
ERROR: decompressed-port failure (6495)

any idea what this could be?

> However, your patch still does not apply on top of master:
> Applying: gnu: Add slurm.
> .git/rebase-apply/patch:87: trailing whitespace.
>  (substitute* "./doc/html/shtml2html.py" 
> .git/rebase-apply/patch:119: space before tab in indent.
>  config.xml
> .git/rebase-apply/patch:120: space before tab in indent.
>  auxdir/Makefile
> .git/rebase-apply/patch:141: space before tab in indent.
>  doc/Makefile
> .git/rebase-apply/patch:142: space before tab in indent.
>  doc/man/Makefile
> error: patch failed: gnu/packages/parallel.scm:2
> error: gnu/packages/parallel.scm: patch does not apply
> Patch failed at 0001 gnu: Add slurm.
> 
> I do not think that the whitespace is a real issue; did you rebase before
> sending the patch? It is quite curious since the parallel.scm file has not
> been changed yet this year.

I am using a recent repo of guix. I did do a rebase and the patch applies
on my tree (as did the previous one). Is there no more information?

Pj.

-- 

Massive graph

[Pjotr Prins]

I am packaging the software that makes up http://genenetwork.org/.

  http://biobeat.org/gn2.svg

or

  http://biobeat.org/gn2.eps

Amazing. And there is more to come.

Pj.
- 

Re: [PATCH] gnu: Add slurm.

[Andreas Enge]

On Sat, Feb 13, 2016 at 05:42:57PM +0100, Pjotr Prins wrote:
> On Sat, Feb 13, 2016 at 11:28:25AM +0100, Andreas Enge wrote:
> > Some lines are way beyond 80 characters; this should be detected by
> > "guix lint".
> guix lint has not been working for me:

Strange, it works for me with other packages.

> I am using a recent repo of guix. I did do a rebase and the patch applies
> on my tree (as did the previous one). Is there no more information?

No, that is all (except a few more lines explaining what to do now in a git
context). Could these two issues be related?

It would be good if someone else could try to apply the patch.

Andreas

Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

On Sat, Feb 13, 2016 at 05:42:57PM +0100, Pjotr Prins wrote:
> > However, your patch still does not apply on top of master:
> I am using a recent repo of guix. I did do a rebase and the patch applies
> on my tree (as did the previous one). Is there no more information?

git pull origin master
>From git://git.savannah.gnu.org/guix
 * branchmaster -> FETCH_HEAD
Updating a747bab..a42d99f
Fast-forward
 gnu/packages/ebook.scm | 4 ++--
 gnu/packages/maths.scm | 4 ++--
 gnu/packages/ocaml.scm | 6 ++
 3 files changed, 10 insertions(+), 4 deletions(-)
bergamo:~/izip/git/opensource/scheme/guix$ git log
commit a42d99f2843d3880cad2f20777b4bd1f969c38dd
Author: Andreas Enge 
Date:   Sat Feb 13 14:34:50 2016 +0100

gnu: calibre: Update to 2.51.0.

* gnu/packages/ebook.scm (calibre): Update to 2.51.0.

(i.e. latest)

cat 0001-gnu-Add-slurm.patch |patch -p1  # same patch, different name
patching file gnu/packages/parallel.scm
patching file 
gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch

Works fine. Maybe because I used an attachment previously?

I'll send it again after renaming the patch.

-- 

Gee Gee: ever typed: git package -l, guix rebase -i HEAD~8?

[Jan Nieuwenhuizen]

Hi,

Git is *awesome*.  Guile is *awesome*.  Guix is *awesome*.  Combined,
they're...totallyconfusing my limbic system, especially late at night.

If you also find yourself typing `that g* command' and guessing wrong
(git: "Did you mean this?"  bundle) like I did ever so often last night,
then Gee Gee might be for you.

Have a look at/get it from

https://github.com/janneke/gee-gee

and let me know what you think.

Greetings,
Jan

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

OK, patch coming up.

On Sat, Feb 13, 2016 at 11:28:25AM +0100, Andreas Enge wrote:
> On Sat, Feb 13, 2016 at 09:59:40AM +0100, Pjotr Prins wrote:
> > Another package name (this time we go down to slurm), another patch.
> 
> Actually, the name of the patch file: slurm-wml-patch should also
> be changed... And it needs to be registered in gnu-system.am.
> 
> Some lines are way beyond 80 characters; this should be detected by
> "guix lint".

Only one line that I can tell

  (patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))

> > I also put the modules in alphabetic order when adding the license
> > name space. Hope that doesn't need a separate patch.
> 
> I think that is fine.
> 
> However, your patch still does not apply on top of master:
> Applying: gnu: Add slurm.
> .git/rebase-apply/patch:87: trailing whitespace.
>  (substitute* "./doc/html/shtml2html.py" 
> .git/rebase-apply/patch:119: space before tab in indent.
>  config.xml
> .git/rebase-apply/patch:120: space before tab in indent.
>  auxdir/Makefile
> .git/rebase-apply/patch:141: space before tab in indent.
>  doc/Makefile
> .git/rebase-apply/patch:142: space before tab in indent.
>  doc/man/Makefile
> error: patch failed: gnu/packages/parallel.scm:2
> error: gnu/packages/parallel.scm: patch does not apply
> Patch failed at 0001 gnu: Add slurm.
> 
> I do not think that the whitespace is a real issue; did you rebase before
> sending the patch? It is quite curious since the parallel.scm file has not
> been changed yet this year.

I double checked. There is no tabs or trailing whitespace in the
package.

Anyway try again. If it fails I give up. I have had enough of slurm
and for now of sending patches back and forth.

Pj.

[PATCH] gnu: Add slurm.

[Pjotr Prins]

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New 
file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu-system.am  |  1 +
 gnu/packages/parallel.scm  | 67 --
 .../slurm-configure-remove-nonfree-contribs.patch  | 43 ++
 3 files changed, 106 insertions(+), 5 deletions(-)
 create mode 100644 
gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch

diff --git a/gnu-system.am b/gnu-system.am
index fd9795e..ac4b241 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -702,6 +702,7 @@ dist_patch_DATA =   
\
   gnu/packages/patches/slim-session.patch  \
   gnu/packages/patches/slim-config.patch   \
   gnu/packages/patches/slim-sigusr1.patch  \
+  gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch\
   gnu/packages/patches/soprano-find-clucene.patch  \
   gnu/packages/patches/sudo-CVE-2015-5602.patch\
   gnu/packages/patches/superlu-dist-scotchmetis.patch  \
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append 
"https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") 
".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch 
b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a

[PATCH] gnu: Add slurm.

[Pjotr Prins]

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New 
file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu-system.am  |  1 +
 gnu/packages/parallel.scm  | 67 --
 .../slurm-configure-remove-nonfree-contribs.patch  | 43 ++
 3 files changed, 106 insertions(+), 5 deletions(-)
 create mode 100644 
gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch

diff --git a/gnu-system.am b/gnu-system.am
index fd9795e..ac4b241 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -702,6 +702,7 @@ dist_patch_DATA =   
\
   gnu/packages/patches/slim-session.patch  \
   gnu/packages/patches/slim-config.patch   \
   gnu/packages/patches/slim-sigusr1.patch  \
+  gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch\
   gnu/packages/patches/soprano-find-clucene.patch  \
   gnu/packages/patches/sudo-CVE-2015-5602.patch\
   gnu/packages/patches/superlu-dist-scotchmetis.patch  \
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append 
"https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") 
".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch 
b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a

Re: [PATCH] gnu: Add slurm.

[Andreas Enge]

On Sat, Feb 13, 2016 at 06:01:19PM +0100, Pjotr Prins wrote:
> cat 0001-gnu-Add-slurm.patch |patch -p1  # same patch, different name
> patching file gnu/packages/parallel.scm
> patching file 
> gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
> Works fine. Maybe because I used an attachment previously?

No, that should not be a problem (after I removed the first line ">From..."
of the attachment). I still get:

patching file gnu/packages/parallel.scm
Hunk #1 FAILED at 2.
1 out of 3 hunks FAILED -- saving rejects to file gnu/packages/parallel.scm.rej
patching file 
gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch

The thing that does not work is adding your copyright line. Very strange;
we seem to have the same white-space in the file.

Andreas

[PATCH] gnu: Add slurm.

[Pjotr Prins]

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New 
file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu-system.am  |  1 +
 gnu/packages/parallel.scm  | 67 --
 .../slurm-configure-remove-nonfree-contribs.patch  | 43 ++
 3 files changed, 106 insertions(+), 5 deletions(-)
 create mode 100644 
gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch

diff --git a/gnu-system.am b/gnu-system.am
index fd9795e..ac4b241 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -702,6 +702,7 @@ dist_patch_DATA =   
\
   gnu/packages/patches/slim-session.patch  \
   gnu/packages/patches/slim-config.patch   \
   gnu/packages/patches/slim-sigusr1.patch  \
+  gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch\
   gnu/packages/patches/soprano-find-clucene.patch  \
   gnu/packages/patches/sudo-CVE-2015-5602.patch\
   gnu/packages/patches/superlu-dist-scotchmetis.patch  \
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append 
"https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") 
".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch 
b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a

[PATCH] gnu: Add slurm.

[Pjotr Prins]

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New 
file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu-system.am  |  1 +
 gnu/packages/parallel.scm  | 67 --
 .../slurm-configure-remove-nonfree-contribs.patch  | 43 ++
 3 files changed, 106 insertions(+), 5 deletions(-)
 create mode 100644 
gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch

diff --git a/gnu-system.am b/gnu-system.am
index fd9795e..ac4b241 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -702,6 +702,7 @@ dist_patch_DATA =   
\
   gnu/packages/patches/slim-session.patch  \
   gnu/packages/patches/slim-config.patch   \
   gnu/packages/patches/slim-sigusr1.patch  \
+  gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch\
   gnu/packages/patches/soprano-find-clucene.patch  \
   gnu/packages/patches/sudo-CVE-2015-5602.patch\
   gnu/packages/patches/superlu-dist-scotchmetis.patch  \
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append 
"https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") 
".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch 
b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a

How to keep the In-Reply-To field in a patch with formail?

[Pjotr Prins]

Anyone know how to reating the In-Reply-To field when using formail?

  cat mail.patch|formail -s mail -t -f -I "In-Reply-To: 
<20160213102825.GA19194@debian>"  
  Ignoring header field "Date: Sat, 13 Feb 2016 18:07:31 +0100"
  Ignoring header field "In-Reply-To: <20160213102825.GA19194@debian>"  

How you you keep patches on the same thread?

Pj.

[PATCH] gnu: Add slurm.

[Pjotr Prins]

* gnu/packages/parallel.scm (slurm): New variable.
* gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch: New 
file.
* gnu/packages/parallel.scm (parallel): Introduce license name space.
---
 gnu-system.am  |  1 +
 gnu/packages/parallel.scm  | 67 --
 .../slurm-configure-remove-nonfree-contribs.patch  | 43 ++
 3 files changed, 106 insertions(+), 5 deletions(-)
 create mode 100644 
gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch

diff --git a/gnu-system.am b/gnu-system.am
index fd9795e..ac4b241 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -702,6 +702,7 @@ dist_patch_DATA =   
\
   gnu/packages/patches/slim-session.patch  \
   gnu/packages/patches/slim-config.patch   \
   gnu/packages/patches/slim-sigusr1.patch  \
+  gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch\
   gnu/packages/patches/soprano-find-clucene.patch  \
   gnu/packages/patches/sudo-CVE-2015-5602.patch\
   gnu/packages/patches/superlu-dist-scotchmetis.patch  \
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 8f63bda..a52e554 100644
--- a/gnu/packages/parallel.scm
+++ b/gnu/packages/parallel.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013,2014 Eric Bavier 
 ;;; Copyright © 2015 Mark H Weaver 
 ;;; Copyright © 2015 Efraim Flashner 
+;;; Copyright © 2016 Pjotr Prins 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -19,12 +20,18 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages parallel)
-  #:use-module (guix packages)
-  #:use-module (guix licenses)
-  #:use-module (guix download)
   #:use-module (guix build-system gnu)
+  #:use-module (guix download)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix packages)
   #:use-module (gnu packages)
-  #:use-module (gnu packages perl))
+  #:use-module (gnu packages admin)
+  #:use-module (gnu packages autotools)
+  #:use-module (gnu packages linux)
+  #:use-module (gnu packages perl)
+  #:use-module (gnu packages python)
+  #:use-module (gnu packages tcl)
+  #:use-module (gnu packages tls))
 
 (define-public parallel
   (package
@@ -46,4 +53,54 @@
  "GNU Parallel is a tool for executing shell jobs in parallel using one
 or more computers.  Jobs can consist of single commands or of scripts
 and they are executed on lists of files, hosts, users or other items.")
-(license gpl3+)))
+(license license:gpl3+)))
+
+(define-public slurm
+  (package
+   (name "slurm")
+   (version "15.08.7.1")
+   (source (origin
+(method url-fetch)
+(uri (string-append 
"https://github.com/SchedMD/slurm/archive/slurm-";
+(string-join (string-split version #\.) "-") 
".tar.gz"))
+(file-name (string-append name "-" version ".tar.gz"))
+(sha256
+ (base32
+  "1rmi35l4img00dr4vic8cv8s7b6n1yx1mkq2s7kjf5hvqdh6s2ki"))
+(patches (list (search-patch 
"slurm-wlm-configure-remove-nonfree-contribs.patch")))
+(modules '((guix build utils)))
+(snippet
+ '(begin
+(delete-file-recursively "contribs")
+#t
+   (inputs `(("openssl" ,openssl)
+ ("munge" ,munge)
+ ("perl" ,perl)
+ ("expect" ,expect)
+ ("python" ,python)
+ ("linux-pam" , linux-pam)))
+   (native-inputs
+`(("autoconf" ,autoconf)))
+   (build-system gnu-build-system)
+   (arguments
+`(#:configure-flags '("--enable-pam")
+  #:phases
+  (modify-phases %standard-phases
+   (add-before
+'configure 'autogen
+(lambda _ (zero? (system* "autoconf" ; configure.ac was patched
+   (add-before
+'autogen 'rewrite-usr-bin
+(lambda* (#:key inputs #:allow-other-keys)
+ ;; replace python call with python3
+ (substitute* "./doc/html/shtml2html.py" 
+  (("#!/usr/bin/env python")
+   (string-append "#!" (which "python3"
+ (substitute* "src/common/env.c"
+  (("/usr/bin/env") (which "env"
+   (home-page "http://slurm.schedmd.com/";)
+   (synopsis "Workload manager for cluster computing")
+   (description
+"Fault-tolerant and highly scalable cluster management and job
+scheduling system for large and small clusters.")
+   (license license:gpl2+)))
diff --git a/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch 
b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
new file mode 100644
index 000..b63d5bb
--- /dev/null
+++ b/gnu/packages/patches/slurm-configure-remove-nonfree-contribs.patch
@@ -0,0 +1,43 @@
+From 53eda9102b969a

Re: [PATCH] gnu: Add slurm.

[Andreas Enge]

I still have the same problem with your patch. If you do not mind, could
you just send me your final parallel.scm and
slurm-wlm-configure-remove-nonfree-contribs.patch ?

When I will have a bit of time I will then go over it again.
(I still think we should use python-wrapper, for instance.)

Thanks!

Andreas

Ooops: Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

Sorry everyone for the extra E-mail load. 

On Sat, Feb 13, 2016 at 06:30:58PM +0100, Andreas Enge wrote:
> I still have the same problem with your patch. If you do not mind, could
> you just send me your final parallel.scm and
> slurm-wlm-configure-remove-nonfree-contribs.patch ?

Can someone else try the patch and confirm I must be losing my mind?

> When I will have a bit of time I will then go over it again.
> (I still think we should use python-wrapper, for instance.)

OK. 

Pj.
-- 

guix system init co-existing with other distros

[Jan Nieuwenhuizen]

Hi,

The people at Fosdem inspired me to move to GuixSD.  I have tried
GuixSD with ratpoison, xfce and guile-wm (oh, how I'd love to get
another sawfish-like windowmanager!) but now moved from Ubuntu
to Debian-hosted with guix.  For now.

I am using this snippet in (operatating-system) to multi-boot

  (bootloader
   (grub-configuration
(device "/dev/sda")
;; Here is my hack (grub-configuration appends bzImage to `linux')
;; to add a custom entry for debian on /dev/sda3 to the Grub menu
;; On Debian, do:
;;   sudo mkdir -p /boot/latest
;;   sudo ln -s $(ls -t /boot/vmlinuz*|head -1) /boot/latest/bzImage
;;   sudo ln -s $(ls -t /boot/initrd*|head -1) /boot/latest/initrd
(menu-entries
 (list (menu-entry
(label "debian")
(linux "(hd0,msdos3)/boot/latest")
(linux-arguments
 '("root=UUID=73ede06b-19e0-4d09-bc1e-23fe6c76d77a ro"))
(initrd "(hd0,msdos3)/boot/latest/initrd"))

how are you all doing this?  Can we/do we want to give almost-GuixSD
users a better experience?

Greetings,
Jan

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH 2/3] gnu: iproute: Update to 4.4.0.

[Alex Kost]

Efraim Flashner (2016-02-11 14:12 +0300) wrote:

> On Thu, 11 Feb 2016 14:03:18 +0300
> Alex Kost  wrote:
>
>> * gnu/packages/linux.scm (iproute): Update to 4.4.0.
>> ---
>>  gnu/packages/linux.scm | 16 
>>  1 file changed, 8 insertions(+), 8 deletions(-)
[...]
> LGTM!

Thanks, I've pushed this patch, sorry for stepping on your feet with
this update :-)

-- 
Alex

[PATCH 0/3] emacs: Few improvements.

[Alex Kost]

A couple of fixes/improvements discovered in bug reports opened by myglc2.

Patches:

[PATCH 1/3] emacs: Add 'M-x guix-installed-{user/system}-packages'.
  As proposed at .

[PATCH 2/3] emacs: Set 'guix-buffer-item' before displaying entries.
  This is a required change for the next patch.

[PATCH 3/3] emacs: Do not allow to modify system profiles.

[PATCH 3/3] emacs: Do not allow to modify system profiles.

[Alex Kost]

Fixes .
Reported by myglc2 .

Remove possibilities to install/delete packages to/from a system profile
both for "Package List" and "Package Info" buffers.

* emacs/guix-profiles.el (guix-system-profile-regexp): New variable.
(guix-system-profile?): New procedure.
* emacs/guix-ui-package.el (guix-package-info-insert-output): Do not
display "Install"/"Delete" button for a system profile.
(guix-package-assert-non-system-profile): New procedure.
(guix-package-execute-actions): Use it.
---
 emacs/guix-profiles.el   |  8 
 emacs/guix-ui-package.el | 27 ++-
 2 files changed, 30 insertions(+), 5 deletions(-)

diff --git a/emacs/guix-profiles.el b/emacs/guix-profiles.el
index 43ad1d4..12cf46d 100644
--- a/emacs/guix-profiles.el
+++ b/emacs/guix-profiles.el
@@ -40,6 +40,14 @@
 (defvar guix-current-profile guix-default-profile
   "Current profile.")
 
+(defvar guix-system-profile-regexp
+  (concat "\\`" (regexp-quote guix-system-profile))
+  "Regexp matching system profiles.")
+
+(defun guix-system-profile? (profile)
+  "Return non-nil, if PROFILE is a system one."
+  (string-match-p guix-system-profile-regexp profile))
+
 (defun guix-profile-prompt (&optional default)
   "Prompt for profile and return it.
 Use DEFAULT as a start directory.  If it is nil, use
diff --git a/emacs/guix-ui-package.el b/emacs/guix-ui-package.el
index 78d2c40..c92108b 100644
--- a/emacs/guix-ui-package.el
+++ b/emacs/guix-ui-package.el
@@ -454,17 +454,22 @@ current OUTPUT is installed (if there is such output in
  (string= (guix-entry-value entry 'output)
   output))
installed))
- (action-type (if installed-entry 'delete 'install)))
+ (action-type (if installed-entry 'delete 'install))
+ (profile (guix-ui-current-profile)))
 (guix-info-insert-indent)
 (guix-format-insert output
 (if installed-entry
 'guix-package-info-installed-outputs
   'guix-package-info-uninstalled-outputs)
 guix-package-info-output-format)
-(guix-package-info-insert-action-button action-type entry output)
-(when obsolete
-  (guix-info-insert-indent)
-  (guix-package-info-insert-action-button 'upgrade entry output))
+;; Do not allow to install/delete anything to/from a system profile,
+;; so add action buttons only for non-system profiles.
+(when (and profile
+   (not (guix-system-profile? profile)))
+  (guix-package-info-insert-action-button action-type entry output)
+  (when obsolete
+(guix-info-insert-indent)
+(guix-package-info-insert-action-button 'upgrade entry output)))
 (insert "\n")
 (when installed-entry
   (guix-info-insert-entry installed-entry 'installed-output 2
@@ -723,10 +728,22 @@ take an entry as argument."
 'upgrade nil
 (guix-package-installed-outputs entry)
 
+(defun guix-package-assert-non-system-profile ()
+  "Verify that the current profile is not a system one.
+The current profile is the one used by the current buffer."
+  (let ((profile (guix-ui-current-profile)))
+(and profile
+ (guix-system-profile? profile)
+ (user-error "Packages cannot be installed or removed to/from \
+profile '%s'.
+Use 'guix system reconfigure' shell command to modify a system profile."
+ profile
+
 (defun guix-package-execute-actions (fun)
   "Perform actions on the marked packages.
 Use FUN to define actions suitable for `guix-process-package-actions'.
 FUN should take action-type as argument."
+  (guix-package-assert-non-system-profile)
   (let ((actions (delq nil
(mapcar fun '(install delete upgrade)
 (if actions
-- 
2.6.3

[PATCH 2/3] emacs: Set 'guix-buffer-item' before displaying entries.

[Alex Kost]

* emacs/guix-buffer.el (guix-buffer-set): Set 'guix-buffer-item' early,
  so that it can be used during displaying entries.  For example, this
  allows to use a value of the current guix profile when package entries
  are inserted in a "List" or "Info" buffer.
---
 emacs/guix-buffer.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/emacs/guix-buffer.el b/emacs/guix-buffer.el
index af76e63..873540b 100644
--- a/emacs/guix-buffer.el
+++ b/emacs/guix-buffer.el
@@ -241,8 +241,8 @@ HISTORY should be one of the following:
   `replace' - replace the current history item."
   (guix-buffer-with-item buffer-item
 (when %entries
-  (guix-buffer-show-entries %entries %buffer-type %entry-type)
   (setq guix-buffer-item buffer-item)
+  (guix-buffer-show-entries %entries %buffer-type %entry-type)
   (when history
 (funcall (cl-ecase history
(add #'guix-history-add)
-- 
2.6.3

[PATCH 1/3] emacs: Add 'M-x guix-installed-{user/system}-packages'.

[Alex Kost]

* emacs/guix-ui-package.el (guix-installed-user-packages)
(guix-installed-system-packages): New commands.
* doc/emacs.texi (Emacs Commands): Document them.
---
 doc/emacs.texi   |  8 +++-
 emacs/guix-ui-package.el | 13 +
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/doc/emacs.texi b/doc/emacs.texi
index dbe59f0..27f5365 100644
--- a/doc/emacs.texi
+++ b/doc/emacs.texi
@@ -150,7 +150,13 @@ Commands for displaying packages:
 Display all/newest available packages.
 
 @item M-x guix-installed-packages
-Display all installed packages.
+@itemx M-x guix-installed-user-packages
+@itemx M-x guix-installed-system-packages
+Display installed packages.  As described above, @kbd{M-x
+guix-installed-packages} uses an arbitrary profile that you can specify,
+while the other commands display packages installed in 2 special
+profiles: @file{~/.guix-profile} and @file{/run/current-system/profile}
+(only on GuixSD).
 
 @item M-x guix-obsolete-packages
 Display obsolete packages (the packages that are installed in a profile
diff --git a/emacs/guix-ui-package.el b/emacs/guix-ui-package.el
index ff10a1e..78d2c40 100644
--- a/emacs/guix-ui-package.el
+++ b/emacs/guix-ui-package.el
@@ -982,6 +982,19 @@ Interactively with prefix, prompt for PROFILE."
   (guix-package-get-display profile 'installed))
 
 ;;;###autoload
+(defun guix-installed-user-packages ()
+  "Display information about Guix packages installed in a user profile."
+  (interactive)
+  (guix-installed-packages guix-user-profile))
+
+;;;###autoload
+(defun guix-installed-system-packages ()
+  "Display information about Guix packages installed in a system profile."
+  (interactive)
+  (guix-installed-packages
+   (guix-packages-profile guix-system-profile nil t)))
+
+;;;###autoload
 (defun guix-obsolete-packages (&optional profile)
   "Display information about obsolete Guix packages.
 If PROFILE is nil, use `guix-current-profile'.
-- 
2.6.3

Documentation patches

[Andreas Enge]

Attached are four more patches that go beyond typos and so on, so I would
like to hear your opinion. For the last one, I am not sure if the info is
correct, but since the importer treats JSON files, I thought that assuming
a dependency on Guile-JSON had a good chance of being correct.

Andreas

>From a58b5438300c0e5cd64aec4959fa34bcf5c0f205 Mon Sep 17 00:00:00 2001
From: Andreas Enge 
Date: Sat, 13 Feb 2016 20:04:10 +0100
Subject: [PATCH 1/4] doc: Move paragraphs in description of guix archive.

* doc/guix.texi ("invoking guix archive"): Move paragraph describing basic
  invocation from the end to the beginning of the section.
---
 doc/guix.texi | 46 --
 1 file changed, 24 insertions(+), 22 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index bac7389..0ff5cfc 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -2004,8 +2004,30 @@ useful to Guix developers.
 The @command{guix archive} command allows users to @dfn{export} files
 from the store into a single archive, and to later @dfn{import} them.
 In particular, it allows store files to be transferred from one machine
-to the store on another machine.  For example, to transfer the @code{emacs}
-package to a machine connected over SSH, one would run:
+to the store on another machine.
+
+To export store files as an archive to standard output, run:
+
+@example
+guix archive --export @var{options} @var{specifications}...
+@end example
+
+@var{specifications} may be either store file names or package
+specifications, as for @command{guix package} (@pxref{Invoking guix
+package}).  For instance, the following command creates an archive
+containing the @code{gui} output of the @code{git} package and the main
+output of @code{emacs}:
+
+@example
+guix archive --export git:gui /gnu/store/...-emacs-24.3 > great.nar
+@end example
+
+If the specified packages are not built yet, @command{guix archive}
+automatically builds them.  The build process may be controlled with the
+common build options (@pxref{Common Build Options}).
+
+To transfer the @code{emacs} package to a machine connected over SSH,
+one would run:
 
 @example
 guix archive --export -r emacs | ssh the-machine guix archive --import
@@ -2127,26 +2149,6 @@ archive contents coming from possibly untrusted 
substitute servers.
 
 @end table
 
-To export store files as an archive to standard output, run:
-
-@example
-guix archive --export @var{options} @var{specifications}...
-@end example
-
-@var{specifications} may be either store file names or package
-specifications, as for @command{guix package} (@pxref{Invoking guix
-package}).  For instance, the following command creates an archive
-containing the @code{gui} output of the @code{git} package and the main
-output of @code{emacs}:
-
-@example
-guix archive --export git:gui /gnu/store/...-emacs-24.3 > great.nar
-@end example
-
-If the specified packages are not built yet, @command{guix archive}
-automatically builds them.  The build process may be controlled with the
-common build options (@pxref{Common Build Options}).
-
 @c *
 @include emacs.texi
 
-- 
2.6.3

>From 4ff40a70eb7ed35f153e001f5935956286dfbdbf Mon Sep 17 00:00:00 2001
From: Andreas Enge 
Date: Sat, 13 Feb 2016 20:07:08 +0100
Subject: [PATCH 2/4] doc: Use modify-phases syntax in example.

* doc/guix.texi ("build systems"): Use modify-phases syntax instead of
  alist-delete in example.
---
 doc/guix.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 0ff5cfc..42b3fba 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -2651,7 +2651,7 @@ The list of phases used for a particular package can be 
changed with the
 @code{#:phases} parameter.  For instance, passing:
 
 @example
-#:phases (alist-delete 'configure %standard-phases)
+#:phases (modify-phases %standard-phases (delete 'configure))
 @end example
 
 means that all the phases described above will be used, except the
-- 
2.6.3

>From b46aab8149d30708e58b10e2428eb4a41f63f61e Mon Sep 17 00:00:00 2001
From: Andreas Enge 
Date: Sat, 13 Feb 2016 20:08:26 +0100
Subject: [PATCH 3/4] doc: Drop documentation of deprecated procedures.

* doc/guix.texi ("build systems"): Drop documentation of
  BUILD-EXPRESSION->DERIVATION.
---
 doc/guix.texi | 53 -
 1 file changed, 53 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 42b3fba..979288c 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -3022,59 +3022,6 @@ best course of action for that is to write the build 
code as a
 ``G-expression'', and to pass it to @code{gexp->derivation}.  For more
 information, @pxref{G-Expressions}.
 
-Once upon a time, @code{gexp->derivation} did not exist and constructing
-derivations with build code written in Scheme was achieved with
-@code{build-expression->derivation}, documented below.  This procedure
-is now deprecated in favor of the much

Re: [PATCH] gnu: Add slurm.

[Leo Famulari]

On Sat, Feb 13, 2016 at 06:20:01PM +0100, Andreas Enge wrote:
> On Sat, Feb 13, 2016 at 06:01:19PM +0100, Pjotr Prins wrote:
> > cat 0001-gnu-Add-slurm.patch |patch -p1  # same patch, different name
> > patching file gnu/packages/parallel.scm
> > patching file 
> > gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
> > Works fine. Maybe because I used an attachment previously?
> 
> No, that should not be a problem (after I removed the first line ">From..."
> of the attachment). I still get:
> 
> patching file gnu/packages/parallel.scm
> Hunk #1 FAILED at 2.
> 1 out of 3 hunks FAILED -- saving rejects to file 
> gnu/packages/parallel.scm.rej
> patching file 
> gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
> 
> The thing that does not work is adding your copyright line. Very strange;
> we seem to have the same white-space in the file.

I had the same issue with some of Jan's patches yesterday.

I used diffoscope to compare the non-working patch and a "should-be"
identical patch that I generated locally. The only differences were in
the patch headers and the encoding of the file (UTF-8 vs ISO-8859-1).

I couldn't figure out why these differences would matter and moved on,
but it's disturbing that others are hitting the same issue!

Re: [PATCH] gnu: Add slurm.

[Andreas Enge]

On Sat, Feb 13, 2016 at 03:13:37PM -0500, Leo Famulari wrote:
> I used diffoscope to compare the non-working patch and a "should-be"
> identical patch that I generated locally. The only differences were in
> the patch headers and the encoding of the file (UTF-8 vs ISO-8859-1).

I think that explains it: The non-working part of the patch was the copyright
line, which contains the copyright symbol!

So let us all move to utf-8!

Andreas

Re: [PATCH] gnu: Add slurm.

[Leo Famulari]

On Sat, Feb 13, 2016 at 03:13:37PM -0500, Leo Famulari wrote:
> On Sat, Feb 13, 2016 at 06:20:01PM +0100, Andreas Enge wrote:
> > On Sat, Feb 13, 2016 at 06:01:19PM +0100, Pjotr Prins wrote:
> > > cat 0001-gnu-Add-slurm.patch |patch -p1  # same patch, different name
> > > patching file gnu/packages/parallel.scm
> > > patching file 
> > > gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
> > > Works fine. Maybe because I used an attachment previously?
> > 
> > No, that should not be a problem (after I removed the first line ">From..."
> > of the attachment). I still get:
> > 
> > patching file gnu/packages/parallel.scm
> > Hunk #1 FAILED at 2.
> > 1 out of 3 hunks FAILED -- saving rejects to file 
> > gnu/packages/parallel.scm.rej
> > patching file 
> > gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
> > 
> > The thing that does not work is adding your copyright line. Very strange;
> > we seem to have the same white-space in the file.
> 
> I had the same issue with some of Jan's patches yesterday.
> 
> I used diffoscope to compare the non-working patch and a "should-be"
> identical patch that I generated locally. The only differences were in
> the patch headers and the encoding of the file (UTF-8 vs ISO-8859-1).
> 
> I couldn't figure out why these differences would matter and moved on,
> but it's disturbing that others are hitting the same issue!

I investigated some more. I used the patch that is upthread of what I am
replying to.

I applied the patch with `patch -p1 < ...`. I could have used `git am`
but I'm not doing it again ;)

Everything applied except for Pjotr's attribution in parallel.scm, so I
manually copied that into the file.

Then, I added all the changes to the index, committed, and used `git
format-patch` to create a patch on master. I'll show you:

$ git add -u
$ git add gnu/packages/patches/slurm-wlm-configure-remove-nonfree-contribs.patch
$ git commit -m "patch-test"
$ git format-patch -n --thread=shallow master
$ git reset --hard HEAD^ # roll back one commit
$ git am 0001-patch-test.patch
$ echo $?
0
$ diffoscope --html report.html 0001-patch-test.patch 0001-gnu-Add-slurm.patch

I've attached report.html and 0001-patch-test.patch for your inspection.

I wonder if the version of git is a factor? 2.1.4 vs 2.6.3. Jan was also
using 2.1.4, and his patches also failed *only* on the attribution line.
Title: /gnu/store/l0a7x49wsxhvgnc33zazfiml7dfi150s-diffoscope-34/bin/.diffoscope-real --html report.html 0001-patch-test.patch 0001-gnu-Add-slurm.patch







0001-patch-test.patch vs.
0001-gnu-Add-slurm.patch
 ¶








Offset 1, 13 lines modified
Offset 1, 15 lines modified


1 

From·​788a22631663d1e4060eb​e87b4272c590f138a25·​Mon·​Sep·​17·​00:​00:​00·​2001

1 

From·​ef998204cde384d35b605​509be09b25170c28fb8·​Mon·​Sep·​17·​00:​00:​00·​2001




2 

Message-​Id:​·​<788a22631663d1e4060e​be87b4272c590f138a25.​1455395436.​git.​leo@famulari.​name>

2 

From:​·​Pjotr·​Prins·​




3 

From:​·​Leo·​Famulari·​

3 

Date:​·​Sat,​·​13·​Feb·​2016·​09:​45:​30·​+0100




4 

Date:​·​Sat,​·​13·​Feb·​2016·​15:​30:​30·​-​0500

4 

Subject:​·​[PATCH]·​gnu:​·​Add·​slurm.​




5 

Subject:​·​[PATCH·​1/​1]·​patch·​test

5 






6 



6 






 
7 

*·​gnu/​packages/​parallel.​scm·​(slurm)​:​·​New·​variable.​




 
8 

*·​gnu/​packages/​patches/​slurm-​wlm-​configure-​remove-​nonfree-​contribs.​patch:​·​New·​file.​




 
9 

*·​gnu/​packages/​parallel.​scm·​(parallel)​:​·​Introduce·​license·​name·​space.​




7 

-​-​-​

10 

-​-​-​




8 

·​gnu/​packages/​parallel.​scm·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​|·​67·​-​-​

11 

·​gnu/​packages/​parallel.​scm·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​·​|·​67·​-​-​




9 

·​.​.​.​urm-​wlm-​configure-​remove-​nonfree-​contribs.​patch·​|·​43·​++

12 

·​.​.​.​urm-​wlm-​configure-​remove-​nonfree-​contribs.​patch·​|·​43·​++




10 

·​2·​files·​changed,​·​105·​insertions(+)​,​·​5·​deletions(-​)​

13 

·​2·​files·​changed,​·​105·​insertions(+)​,​·​5·​deletions(-​)​




11 

·​create·​mode·​100644·​gnu/​packages/​patches/​slurm-​wlm-​configure-​remove-​nonfree-​contribs.​patch

14 

·​create·​mode·​100644·​gnu/​packages/​patches/​slurm-​wlm-​configure-​remove-​nonfree-​contribs.​patch




12 



15 






13 

diff·​-​-​git·​a/​gnu/​packages/​parallel.​scm·​b/​gnu/​packages/​parallel.​scm

16 

diff·​-​-​git·​a/​gnu/​packages/​parallel.​scm·​b/​gnu/​packages/​parallel.​scm



Offset 147, 9 lines modified
Offset 149, 9 lines modified


147 

+·​»   ​»   ​·​doc/​Makefile

149 

+·​»   ​»   ​·​doc/​Makefile




148 

+·​»   ​»   ​·​doc/​man/​Makefile

150 

+·​»   ​»   ​·​doc/​man/​Makefile




149 

+·​»   ​»   ​·​doc/​man/​man1/​Makefile

151 

+·​»   ​»   ​·​doc/​man/​man1/​Makefile




150 

+-​-​·​

152 

+-​-​·​




151 

+2.​1.​4

153 

+2.​1.​4

Re: [PATCH] gnu: Add slurm.

[Leo Famulari]

On Sat, Feb 13, 2016 at 09:34:22PM +0100, Andreas Enge wrote:
> On Sat, Feb 13, 2016 at 03:13:37PM -0500, Leo Famulari wrote:
> > I used diffoscope to compare the non-working patch and a "should-be"
> > identical patch that I generated locally. The only differences were in
> > the patch headers and the encoding of the file (UTF-8 vs ISO-8859-1).
> 
> I think that explains it: The non-working part of the patch was the copyright
> line, which contains the copyright symbol!
> 
> So let us all move to utf-8!

If that is it, then I believe git should "do the right thing" when
generating patches. I've noticed that it progressively sets the encoding
based on the minimum required to encode the characters in the patch.

For example, if all characters can be encoded in ASCII, it uses ASCII,
otherwise I think it uses UTF-8.

Re: Gee Gee: ever typed: git package -l, guix rebase -i HEAD~8?

[Leo Famulari]

On Sat, Feb 13, 2016 at 06:10:35PM +0100, Jan Nieuwenhuizen wrote:
> Hi,
> 
> Git is *awesome*.  Guile is *awesome*.  Guix is *awesome*.  Combined,
> they're...totallyconfusing my limbic system, especially late at night.
> 
> If you also find yourself typing `that g* command' and guessing wrong
> (git: "Did you mean this?"  bundle) like I did ever so often last night,
> then Gee Gee might be for you.
> 
> Have a look at/get it from
> 
> https://github.com/janneke/gee-gee
> 
> and let me know what you think.

Very nice idea!

I already have a set of shell functions that shorten my commonly used
git invocations to 2 or 3 characters, but I still screw this up
sometimes ;)

> 
> Greetings,
> Jan
> 
> -- 
> Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
> Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  
> 

Re: [PATCH] gnu: Add laby.

[Jan Nieuwenhuizen]

Andreas Enge writes:

> Since there is no terrible hurry, I would suggest to examine the svg rendering
> issue first; if you do not find a solution in reasonable time, we can still
> push the png patch.

Sure.

> No, it just synchronises and does not allow to roll back. On the other hand,
> it also works well with binary files (for instance, for synchronising films
> from one machine to another, I would not want them to take space forever
> in a git repository). It is essentially a two-way rsync.

I had a try...it needs a 2GB texlive download to build...  I'm currently
stuck at

File 
"/tmp/nix-build-unison-2.48.3.drv-0/unison-2.48.3-checkout/src/uigtk.ml", line 
67, characters 10-23:
Error: Unbound module Gdk
Makefile.OCaml:434: recipe for target 'uigtk.cmx' failed
make[1]: *** [uigtk.cmx] Error 2

trying the patch below...

Greetings, Jan

$ git diff
diff --git a/gnu/packages/ocaml.scm b/gnu/packages/ocaml.scm
index 4b5ac61..78e7ea2 100644
--- a/gnu/packages/ocaml.scm
+++ b/gnu/packages/ocaml.scm
@@ -565,6 +565,8 @@ libpanel, librsvg and quartz.")
 (build-system gnu-build-system)
 (outputs '("out"
"doc"))  ; 1.9 MiB of documentation
+(inputs `(("gtk" ,gtk+-2)
+  ("lablgtk" ,lablgtk)))
 (native-inputs
  `(("ocaml" ,ocaml)
;; For documentation
@@ -579,6 +581,7 @@ libpanel, librsvg and quartz.")
#:tests? #f ; Tests require writing to $HOME.
; If some $HOME is provided, they fail with the message
; "Fatal error: Skipping some tests -- remove me!"
+   #:make-flags '("UISTYLE=gtk" "OCAMLOPT=ocamlopt.opt")
#:phases
  (modify-phases %standard-phases
(delete 'configure)
21:57:32 janneke@drakenvlieg:~/src/guix

-- 
Jan Nieuwenhuizen  | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar®  http://AvatarAcademy.nl  

Re: [PATCH] gnu: Add erlang.

[Efraim Flashner]

On Fri, 12 Feb 2016 19:44:46 -0500
Leo Famulari  wrote:

> On Sat, Feb 06, 2016 at 12:45:39PM -0800, Steve Sprang wrote:
> > It looks like downloading the tarball from erlang.org works now.
> > Here's an improved patch.  
> > +  (lambda* _
> > +(let ((escripts
> > +   (append
> > +(find-files "." "\\.escript")
> > +(find-files "lib/stdlib/test/escript_SUITE_data/")
> > +'("erts/lib_src/utils/make_atomics_api"
> > +  "erts/preloaded/src/add_abstract_code"
> > +  "lib/diameter/bin/diameterc"
> > +  "lib/reltool/examples/display_args"
> > +  "lib/reltool/examples/mnesia_core_dump_viewer"
> > +  "lib/snmp/src/compile/snmpc.src"
> > +  "make/verify_runtime_dependencies"
> > +  "make/emd2exml.in"
> > +  (substitute* escripts
> > +(("/usr/bin/env") (which "env"))
> > + (add-before
> > + 'configure 'set-erl-top
> > +   (lambda* _
> > + (setenv "ERL_TOP" (getcwd)  
> 
> I _think_ that 'lambda _' would be appropriate here. I need a real
> Schemer to weigh in ;)
> 

I'm definately not a "real schemer," but since the lambda only leads into the
let (ie. one item), I'm pretty sure 'lambda _' should be sufficient.

-- 
Efraim Flashner  אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted


pgppWTumf01Im.pgp
Description: OpenPGP digital signature

Re: [PATCH 0/5] python2-variants

[Efraim Flashner]

On Fri, 12 Feb 2016 18:56:25 -0500
Leo Famulari  wrote:

> 
> [1] python2-tempest-lib continues to fail to build from source, as
> before.
> 

I was hoping that this would magically fix it. :)

-- 
Efraim Flashner  אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted


pgpXcyyGfmq61.pgp
Description: OpenPGP digital signature

Re: [PATCH] gnu: Add slurm.

[Pjotr Prins]

On Sat, Feb 13, 2016 at 03:38:46PM -0500, Leo Famulari wrote:
> I wonder if the version of git is a factor? 2.1.4 vs 2.6.3. Jan was also
> using 2.1.4, and his patches also failed *only* on the attribution line.

Thank you Leo and Andreas for restoring my sanity! 

I'll upgrade git. I was using the newer version in my build
environment but generating the patch in an older version. Also locales
I need to settle on. I tend to have en_GB on Debian, but in a guix
shell it may act strange if I don't specify locales.

And I won't add attribution again ;). I.e., the combination of being
an old git, having wrong locales and injecting self-attribution, was
lethal. 

Pj.

Re: [PATCH] gnu: Add slurm.

[Leo Famulari]

On Sat, Feb 13, 2016 at 10:32:01PM +0100, Pjotr Prins wrote:
> On Sat, Feb 13, 2016 at 03:38:46PM -0500, Leo Famulari wrote:
> > I wonder if the version of git is a factor? 2.1.4 vs 2.6.3. Jan was also
> > using 2.1.4, and his patches also failed *only* on the attribution line.
> 
> Thank you Leo and Andreas for restoring my sanity! 
> 
> I'll upgrade git. I was using the newer version in my build
> environment but generating the patch in an older version. Also locales
> I need to settle on. I tend to have en_GB on Debian, but in a guix
> shell it may act strange if I don't specify locales.
> 
> And I won't add attribution again ;). I.e., the combination of being
> an old git, having wrong locales and injecting self-attribution, was
> lethal. 

Does this mean the old version of git doesn't generate patches with the
correct locale?

I wonder... where is everyone getting git 2.1.4 from? It's 2 years old
;)

> 
> Pj.

Re: gnu: Add python-wtforms.

[Christopher Allan Webber]

Christopher Allan Webber writes:

> Thompson, David writes:
>
>> This looks good, Chris!  I can tell by looking at the diff that it's
>> based on the wip-mediagoblin.  If it cleanly applies on master then I
>> say "push"!
>>
>> - Dave
>
> Ha, yeah.  I'm trying to get through as many MediaGoblin deps as I can.
>
> Leo Famulari writes:
>
>> On Fri, Feb 12, 2016 at 11:43:56AM -0800, Christopher Allan Webber wrote:
>>> From 8de2a4095987efcf1d9b8f1b5d71ba6fc57e12e8 Mon Sep 17 00:00:00 2001
>>> From: Christopher Allan Webber 
>>> Date: Thu, 11 Feb 2016 16:16:45 -0800
>>> Subject: [PATCH 2/3] gnu: Add python-wtforms
>>> 
>>> * gnu/packages/python.scm (python-wtforms, python2-wtforms): New variables.
>>
>> It looks good to me, but I wonder if the python-3 variant really needs
>> setuptools? If so, okay. If not, can you use the new python2-variant
>> system [0] to provide setuptools for python2-wtforms?
>>
>> [0] Introduced in 48b311b1b3ba
>
> Okay, I'll give that a try, and will make an adjustment if appropriate,
> and regardless, push after figuring that out.
>
> Thanks to both of you!

Well, I think it's significant enough of a change where I should get
some validation that everything looks right before I push.  New version,
ahoy!  Is it ok?

>From 0b3543265a1597206f28ed343d40812c021575d4 Mon Sep 17 00:00:00 2001
From: Christopher Allan Webber 
Date: Sat, 13 Feb 2016 13:42:51 -0800
Subject: [PATCH] gnu: Add python-wtforms

* gnu/packages/python.scm (python-wtforms, python2-wtforms): New variables.
---
 gnu/packages/python.scm | 29 +
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 3758eed..756bb0d 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -7509,3 +7509,32 @@ input.  (Note that this is mostly a legacy library; you may wish to look at
 python-xdo for newer bindings.)")
 (license bsd-3)))
 
+(define-public python-wtforms
+  (package
+(name "python-wtforms")
+(version "2.1")
+(source
+ (origin
+   (method url-fetch)
+   (uri (pypi-uri "WTForms" version ".zip"))
+   (sha256
+(base32
+ "0vyl26y9cg409cfyj8rhqxazsdnd0jipgjw06civhrd53yyi1pzz"
+(build-system python-build-system)
+(native-inputs
+ `(("unzip" ,unzip)))
+(home-page "http://wtforms.simplecodes.com/";)
+(synopsis
+ "Form validation and rendering library for Python web development")
+(description
+ "WTForms is a flexible forms validation and rendering library
+for Python web development.  It is very similar to the web form API
+available in Django, but is a standalone package.")
+(license bsd-3)
+(properties `((python2-variant . ,(delay python2-wtforms))
+
+(define-public python2-wtforms
+  (package
+(inherit python-wtforms)
+(name "python2-wtforms")
+(inputs `(("python2-setuptools" ,python2-setuptools)
-- 
2.6.3

Re: How to keep the In-Reply-To field in a patch with formail?

[Ricardo Wurmus]

Pjotr Prins  writes:

> Anyone know how to reating the In-Reply-To field when using formail?
>
>   cat mail.patch|formail -s mail -t -f -I "In-Reply-To: 
> <20160213102825.GA19194@debian>"  
>   Ignoring header field "Date: Sat, 13 Feb 2016 18:07:31 +0100"
>   Ignoring header field "In-Reply-To: <20160213102825.GA19194@debian>"  
>
> How you you keep patches on the same thread?

I haven’t used “formail” before, but when generating emails with an
external programme I usually don’t allow it to send the email directly.
Instead I load it up in Emacs message-mode and manually add an
In-Reply-To header.

Not sure if this helps you.

~~ Ricardo

Re: Massive graph

[Ricardo Wurmus]

Pjotr Prins  writes:

> I am packaging the software that makes up http://genenetwork.org/.
>
>   http://biobeat.org/gn2.svg

Woah!  Nice how this gets close to the complexity of gene networks,
e.g. this one: http://www.john.ranola.org/Files/RadNetFull.pdf

~~ Ricardo

Re: gnu: Add python-wtforms.

[Leo Famulari]

On Sat, Feb 13, 2016 at 01:55:01PM -0800, Christopher Allan Webber wrote:
> Christopher Allan Webber writes:

[...]

> >>> * gnu/packages/python.scm (python-wtforms, python2-wtforms): New 
> >>> variables.
> >>
> >> It looks good to me, but I wonder if the python-3 variant really needs
> >> setuptools? If so, okay. If not, can you use the new python2-variant
> >> system [0] to provide setuptools for python2-wtforms?
> >>
> >> [0] Introduced in 48b311b1b3ba
> >
> > Okay, I'll give that a try, and will make an adjustment if appropriate,
> > and regardless, push after figuring that out.
> >
> > Thanks to both of you!
> 
> Well, I think it's significant enough of a change where I should get
> some validation that everything looks right before I push.  New version,
> ahoy!  Is it ok?

I believe that you still must call 'package-with-python2' in the
python-2 variant, and the argument to that should be
(strip-python2-variant python-foo).

I've attached an example commit that seems to work, from one of my WIP
branches.
>From c5e63b1f8788e44657a5f508554dc8c21d00c3b4 Mon Sep 17 00:00:00 2001
From: Leo Famulari 
Date: Thu, 28 Jan 2016 00:42:33 -0500
Subject: [PATCH] gnu: Add python2-atomicwrites.

* gnu/packages/python.scm (python2-atomicwrites): New variable.
---
 gnu/packages/python.scm | 9 -
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 5b988c1..3f9886d 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -6751,7 +6751,14 @@ WebSocket usage in Python programs.")
 (description "Library for atomic file writes using platform dependent tools
 for atomic filesystem operations.")
 (home-page "https://github.com/untitaker/python-atomicwrites";)
-(license license:expat)))
+(license license:expat)
+(properties `((python2-variant . ,(delay python2-atomicwrites))
+
+(define-public python2-atomicwrites
+  (package (inherit (package-with-python2
+ (strip-python2-variant python-atomicwrites)))
+(native-inputs
+ `(("python2-setuptools" ,python2-setuptools)
 
 (define-public python-requests-toolbelt
   (package
-- 
2.6.3

Re: How to keep the In-Reply-To field in a patch with formail?

[Leo Famulari]

On Sat, Feb 13, 2016 at 11:10:40PM +0100, Ricardo Wurmus wrote:
> 
> Pjotr Prins  writes:
> 
> > Anyone know how to reating the In-Reply-To field when using formail?
> >
> >   cat mail.patch|formail -s mail -t -f -I "In-Reply-To: 
> > <20160213102825.GA19194@debian>"  
> >   Ignoring header field "Date: Sat, 13 Feb 2016 18:07:31 +0100"
> >   Ignoring header field "In-Reply-To: <20160213102825.GA19194@debian>"  
> >
> > How you you keep patches on the same thread?
> 
> I haven’t used “formail” before, but when generating emails with an
> external programme I usually don’t allow it to send the email directly.
> Instead I load it up in Emacs message-mode and manually add an
> In-Reply-To header.

Another option for organizing patches is to use the '--thread=' argument
to git-format-patch and git-send-email. I use '--thread=shallow'.

[PATCH] gnu: Add libosinfo.

[rennes]

Hi,
i attached libosinfo patch required for GNOME Boxes.

Considerations:

 a) In the source i used 
"https://fedorahosted.org/releases/l/i/libosinfo"; instead 
"mirror://gnome/sources/".
 b) In native inputs, i used "vala" instead "glib:bin"; i follow the 
README.


Thanks.
From 073a183499bd764b0b0efc246748638c6e4d3aeb Mon Sep 17 00:00:00 2001
From: Rene Saavedra 
Date: Sat, 13 Feb 2016 16:23:10 -0600
Subject: [PATCH] gnu: Add libosinfo.

* gnu/packages/gnome.scm (libosinfo): New variable.

---
 gnu/packages/gnome.scm | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 49e6197..2d7ca89 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -93,6 +93,7 @@
   #:use-module (gnu packages tls)
   #:use-module (gnu packages web)
   #:use-module (gnu packages webkit)
+  #:use-module (gnu packages wget)
   #:use-module (gnu packages xorg)
   #:use-module (gnu packages xdisorg)
   #:use-module (gnu packages freedesktop)
@@ -4648,3 +4649,37 @@ as SASL, TLS and VeNCrypt.  Additionally it supports encoding extensions.")
 design and behaviour, giving the user a simple way to navigate and manage its
 files.")
 (license license:gpl2+)))
+
+(define-public libosinfo
+  (package
+(name "libosinfo")
+(version "0.3.0")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append "https://fedorahosted.org/releases/l/i/libosinfo/";
+name "-" version ".tar.gz"))
+   (sha256
+   (base32
+"1g7g5hc4lhi4y0j3mbcj19hawlqkflni1zk4aggrx49fg5l392jk"
+(build-system glib-or-gtk-build-system)
+(native-inputs
+ `(("check" ,check)
+   ("intltool" ,intltool)
+   ("libsoup" ,libsoup)
+   ("pkg-config" ,pkg-config)
+   ("vala" ,vala)
+   ("wget" ,wget)))
+(inputs
+ `(("libxslt" ,libxslt)))
+(home-page "https://libosinfo.org";)
+(synopsis "Library for managing information about operating systems")
+(description
+ "libosinfo is a GObject based library API for managing information about
+operating systems, hypervisors and the (virtual) hardware devices they can
+support.  It includes a database containing device metadata and provides APIs
+to match/identify optimal devices for deploying an operating system on a
+hypervisor.  Via the magic of GObject Introspection, the API is available in all
+common programming languages with demos for javascript (GJS/Seed) and python
+(PyGObject).  Also provided are Vala bindings.")
+(license license:lgpl2.1+)))
-- 
2.6.3

Re: Massive graph

[Pjotr Prins]

On Sat, Feb 13, 2016 at 11:15:14PM +0100, Ricardo Wurmus wrote:
> 
> Pjotr Prins  writes:
> 
> > I am packaging the software that makes up http://genenetwork.org/.
> >
> >   http://biobeat.org/gn2.svg
> 
> Woah!  Nice how this gets close to the complexity of gene networks,
> e.g. this one: http://www.john.ranola.org/Files/RadNetFull.pdf

We could do a hair-ball of the full Guix repository at some point :)

Pj.
-- 

Re: [PATCH] gnu: Add pianobar.

[Al McElrath]

I tracked this down a bit further. There is nothing wrong with Pandora's
cert. If I specify in pianobar's config file to use:

ca_bundle = /home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt

It works. Note if I 'env | grep SSL' I get:

SSL_CERT_DIR=/home/al/.guix-profile/etc/ssl/certs
SSL_CERT_FILE=/home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt

I was under the impression libcurl (which pianobar uses) would honor
these variables, but it appears it does not. Or there's something wrong
with my guix setup which is running on Arch.

I can patch pianobar to use SSL_CERT_FILE by default, but it seems like
there is a larger issue.


Leo Famulari  writes:

> On Thu, Feb 11, 2016 at 10:00:17PM -0800, Al McElrath wrote:
>> 
>> Thanks for testing and pointing this out. Real radio stations require
>> too much manual effort! That said...
>> 
>> You should be getting a different error without a login (pandora.com
>> requires a login): "(i) Login... Error: Wrong email address or
>> password." There is an outstanding issue with pianobar regarding
>> Pandora's certificate described here:
>> 
>> https://github.com/PromyLOPh/pianobar/issues/560
>> 
>> There is a config file option "ca_file" which I use with a pem file I
>> downloaded. There is also a "tls_fingerprint" option described in the
>> bug commentary. This broke recently when Pandora changed their
>> certificate. It's not specifically a Guix bug.
>
> To clarify, is my error the result of a configuration problem and a
> missing .pem file?
>
> If so, is that .pem file something we should distribute with our
> package?
>
>> 
>> I realize this means pianobar simply doesn't work out of the box. I'm
>> open to suggestions on how to make this better.
>> 
>> 
>> Leo Famulari  writes:
>> 
>> > On Thu, Feb 11, 2016 at 01:36:13PM -0800, Al McElrath wrote:
>> >>
>> >> Revised patch is attached. I updated/expanded the description and fixed
>> >> the license. Thanks!
>> >>
>> >
>> >> From 91d5c428428eed605c981a39022fe90e6f98be6d Mon Sep 17 00:00:00 2001
>> >> From: Al McElrath 
>> >> Date: Fri, 11 Dec 2015 21:22:33 -0800
>> >> Subject: [PATCH] gnu: Add pianobar.
>> >>
>> >> * gnu/packages/music.scm (pianobar): New variable.
>> >
>> > One nice thing about "real" radio stations is that you don't need an
>> > account to listen to them ;) Anyways, I tested it as far as I could:
>> >
>> > $ pianobar
>> > Welcome to pianobar (2015.11.22)! Press ? for a list of commands.
>> > [?] Email:
>> > [?] Password:
>> > (i) Login... Network error: Peer certificate cannot be authenticated with 
>> > given CA certificates
>> >
>> > That error seems to describe a problem unrelated to me not having an
>> > account. Can you see what happens when you leave the Email and Password
>> > fields blank? Of course, the message might be wrong, and it works if you
>> > have an account.
>> >
>> > If so, then I think it's okay (but buggy). If not, then it must be
>> > finding the certificate store in the environment and we should figure
>> > out how to provide it reliably.
>> >
>> >> ---
>> >>  gnu/packages/music.scm | 38 ++
>> >>  1 file changed, 38 insertions(+)
>> >>
>> >> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
>> >> index a60ba4b..1382b0a 100644
>> >> --- a/gnu/packages/music.scm
>> >> +++ b/gnu/packages/music.scm
>> >> @@ -2,6 +2,7 @@
>> >>  ;;; Copyright © 2014 Eric Bavier 
>> >>  ;;; Copyright © 2015, 2016 Ricardo Wurmus 
>> >>  ;;; Copyright © 2015 Paul van der Walt 
>> >> +;;; Copyright © 2016 Al McElrath 
>> >>  ;;;
>> >>  ;;; This file is part of GNU Guix.
>> >>  ;;;
>> >> @@ -39,6 +40,7 @@
>> >>#:use-module (gnu packages code)
>> >>#:use-module (gnu packages check)
>> >>#:use-module (gnu packages compression)
>> >> +  #:use-module (gnu packages curl)
>> >>#:use-module (gnu packages docbook)
>> >>#:use-module (gnu packages doxygen)
>> >>#:use-module (gnu packages flex)
>> >> @@ -46,6 +48,7 @@
>> >>#:use-module (gnu packages fonts)
>> >>#:use-module (gnu packages fontutils)
>> >>#:use-module (gnu packages gcc)
>> >> +  #:use-module (gnu packages gnupg)
>> >>#:use-module (gnu packages gettext)
>> >>#:use-module (gnu packages ghostscript)
>> >>#:use-module (gnu packages gl)
>> >> @@ -1078,3 +1081,38 @@ computer's keyboard.")
>> >>  JACK for audio and ALSA sequencer for MIDI as multimedia infrastructures 
>> >> and
>> >>  follows a traditional multi-track tape recorder control paradigm.")
>> >>  (license license:gpl2+)))
>> >> +
>> >> +(define-public pianobar
>> >> +  (package
>> >> +(name "pianobar")
>> >> +(version "2015.11.22")
>> >> +(source (origin
>> >> +  (method url-fetch)
>> >> +  (uri (string-append "https://github.com/PromyLOPh/";
>> >> +  name "/archive/" version ".tar.gz"))
>> >> +  (file-name (string-append name "-" version ".tar.gz"))
>> >> +  (sha256
>> >> +   (ba

Re: [PATCH 2/3] gnu: Don't use guix build utils

[Leo Famulari]

On Sat, Feb 13, 2016 at 04:07:54PM +, Christopher Baines wrote:
> As this provides a which procedure which conflicts with the which package
> provided by gnu packages base.
> 
> * gnu/packages/version-control.scm (version-control): Don't use utils

I can build all the packages in version-control.scm with this change,
but I don't fully understand its implications so I'll wait for another
reviewer to weigh in.

If it's okay, I will squash this commit into "Add vcsh", since that
commit is broken without this one.

> ---
>  gnu/packages/version-control.scm | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/gnu/packages/version-control.scm 
> b/gnu/packages/version-control.scm
> index e0626cb..721a284 100644
> --- a/gnu/packages/version-control.scm
> +++ b/gnu/packages/version-control.scm
> @@ -37,7 +37,6 @@
>#:use-module (guix build-system gnu)
>#:use-module (guix build-system python)
>#:use-module (guix build-system trivial)
> -  #:use-module (guix build utils)
>#:use-module (gnu packages apr)
>#:use-module (gnu packages autotools)
>#:use-module (gnu packages asciidoc)
> -- 
> 2.7.0
> 
> 

Re: [PATCH 3/3] gnu: Add vcsh

[Leo Famulari]

On Sat, Feb 13, 2016 at 04:07:55PM +, Christopher Baines wrote:
> * gnu/packages/version-control.scm (vcsh): New variable.

[...]

> +(arguments
> + `(#:phases (modify-phases %standard-phases
> +  (delete 'configure)
> +  (delete 'check)
> +  (delete 'build))

Can you add comments explaining why the check and build phases are
deleted?

For the former, it can be as simple as "no test suite".

I assume the build phase is replaced by the install phase; the comment
can say something like that if it's the case.

> +   #:make-flags (list (string-append "PREFIX=" %output
> +(native-inputs
> + `(("which" ,which)))
> +(inputs
> + `(("perl-shell-command" ,perl-shell-command)
> +   ("perl-test-most" ,perl-test-most)))
> +(propagated-inputs
> + `(("git" ,git)))
> +(home-page "https://github.com/RichiH/vcsh";)
> +(synopsis "Version control system for $HOME")
> +(description
> + "vcsh provides the @code{vcsh} command, which allows you to maintain
> +several Git repositories in one single directory.  This can be useful when
> +using version control for files in your home directory, as it allows for
> +the separation of the files in to different repositories, for example you
> +may keep your bash configuration in a separate Git repository from your
> +emacs configuration.")
> +(license gpl2+)))
> -- 
> 2.7.0
> 
> 

Re: [PATCH 1/3] gnu: Add perl-shell-command

[Leo Famulari]

On Sat, Feb 13, 2016 at 04:07:53PM +, Christopher Baines wrote:
> This is required for the tests for vcsh.
> 
> * gnu/packages/perl.scm (perl-shell-command): New variable.

LGTM. If I am the one to push, I'll make some minor formatting changes
and I'll add attribution for you.

> ---
>  gnu/packages/perl.scm | 24 
>  1 file changed, 24 insertions(+)
> 
> diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
> index 0e63aa9..5146828 100644
> --- a/gnu/packages/perl.scm
> +++ b/gnu/packages/perl.scm
> @@ -6277,4 +6277,28 @@ really be high enough to warrant the use of a keyword, 
> and the size so small
>  such that being individual extensions would be wasteful.")
>  (license (package-license perl
>  
> +(define-public perl-shell-command
> +  (package
> +(name "perl-shell-command")
> +(version "0.06")
> +(source
> +  (origin
> +(method url-fetch)
> +(uri (string-append
> +   "mirror://cpan/authors/id/F/FL/FLORA/Shell-Command-"
> +   version
> +   ".tar.gz"))
> +(sha256
> +  (base32
> +"1lgc2rb3b5a4lxvbq0cbg08qk0n2i88srxbsz93bwi3razpxxr7k"
> +(build-system perl-build-system)
> +(home-page
> +  "http://search.cpan.org/dist/Shell-Command";)
> +(synopsis
> +  "Cross-platform functions emulating common shell commands")
> +(description
> +  "Thin wrapper around ExtUtils::Command.  See ExtUtils::Command for a
> + description of available commands.")
> +(license (package-license perl
> +
>  ;;; END: Core module overrides
> -- 
> 2.7.0
> 
> 

Re: Massive graph

[Leo Famulari]

On Sat, Feb 13, 2016 at 11:30:11PM +0100, Pjotr Prins wrote:
> On Sat, Feb 13, 2016 at 11:15:14PM +0100, Ricardo Wurmus wrote:
> > 
> > Pjotr Prins  writes:
> > 
> > > I am packaging the software that makes up http://genenetwork.org/.
> > >
> > >   http://biobeat.org/gn2.svg
> > 
> > Woah!  Nice how this gets close to the complexity of gene networks,
> > e.g. this one: http://www.john.ranola.org/Files/RadNetFull.pdf
> 
> We could do a hair-ball of the full Guix repository at some point :)

I've killed `dot` after several minutes of trying to render a really
large graph in PNG format. I wonder if it's up to the task!

> 
> Pj.
> -- 
> 

Re: [PATCH] gnu: Add pianobar.

[Leo Famulari]

On Sat, Feb 13, 2016 at 02:37:05PM -0800, Al McElrath wrote:
> 
> I tracked this down a bit further. There is nothing wrong with Pandora's
> cert. If I specify in pianobar's config file to use:
> 
> ca_bundle = /home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt

Can you share your config, having replaced any personal info with dummy
content?

> 
> It works. Note if I 'env | grep SSL' I get:
> 
> SSL_CERT_DIR=/home/al/.guix-profile/etc/ssl/certs
> SSL_CERT_FILE=/home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt
> 
> I was under the impression libcurl (which pianobar uses) would honor
> these variables, but it appears it does not. Or there's something wrong
> with my guix setup which is running on Arch.
> 
> I can patch pianobar to use SSL_CERT_FILE by default, but it seems like
> there is a larger issue.
> 
> 
> Leo Famulari  writes:
> 
> > On Thu, Feb 11, 2016 at 10:00:17PM -0800, Al McElrath wrote:
> >> 
> >> Thanks for testing and pointing this out. Real radio stations require
> >> too much manual effort! That said...
> >> 
> >> You should be getting a different error without a login (pandora.com
> >> requires a login): "(i) Login... Error: Wrong email address or
> >> password." There is an outstanding issue with pianobar regarding
> >> Pandora's certificate described here:
> >> 
> >> https://github.com/PromyLOPh/pianobar/issues/560
> >> 
> >> There is a config file option "ca_file" which I use with a pem file I
> >> downloaded. There is also a "tls_fingerprint" option described in the
> >> bug commentary. This broke recently when Pandora changed their
> >> certificate. It's not specifically a Guix bug.
> >
> > To clarify, is my error the result of a configuration problem and a
> > missing .pem file?
> >
> > If so, is that .pem file something we should distribute with our
> > package?
> >
> >> 
> >> I realize this means pianobar simply doesn't work out of the box. I'm
> >> open to suggestions on how to make this better.
> >> 
> >> 
> >> Leo Famulari  writes:
> >> 
> >> > On Thu, Feb 11, 2016 at 01:36:13PM -0800, Al McElrath wrote:
> >> >>
> >> >> Revised patch is attached. I updated/expanded the description and fixed
> >> >> the license. Thanks!
> >> >>
> >> >
> >> >> From 91d5c428428eed605c981a39022fe90e6f98be6d Mon Sep 17 00:00:00 2001
> >> >> From: Al McElrath 
> >> >> Date: Fri, 11 Dec 2015 21:22:33 -0800
> >> >> Subject: [PATCH] gnu: Add pianobar.
> >> >>
> >> >> * gnu/packages/music.scm (pianobar): New variable.
> >> >
> >> > One nice thing about "real" radio stations is that you don't need an
> >> > account to listen to them ;) Anyways, I tested it as far as I could:
> >> >
> >> > $ pianobar
> >> > Welcome to pianobar (2015.11.22)! Press ? for a list of commands.
> >> > [?] Email:
> >> > [?] Password:
> >> > (i) Login... Network error: Peer certificate cannot be authenticated 
> >> > with given CA certificates
> >> >
> >> > That error seems to describe a problem unrelated to me not having an
> >> > account. Can you see what happens when you leave the Email and Password
> >> > fields blank? Of course, the message might be wrong, and it works if you
> >> > have an account.
> >> >
> >> > If so, then I think it's okay (but buggy). If not, then it must be
> >> > finding the certificate store in the environment and we should figure
> >> > out how to provide it reliably.
> >> >
> >> >> ---
> >> >>  gnu/packages/music.scm | 38 ++
> >> >>  1 file changed, 38 insertions(+)
> >> >>
> >> >> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> >> >> index a60ba4b..1382b0a 100644
> >> >> --- a/gnu/packages/music.scm
> >> >> +++ b/gnu/packages/music.scm
> >> >> @@ -2,6 +2,7 @@
> >> >>  ;;; Copyright © 2014 Eric Bavier 
> >> >>  ;;; Copyright © 2015, 2016 Ricardo Wurmus 
> >> >>  ;;; Copyright © 2015 Paul van der Walt 
> >> >> +;;; Copyright © 2016 Al McElrath 
> >> >>  ;;;
> >> >>  ;;; This file is part of GNU Guix.
> >> >>  ;;;
> >> >> @@ -39,6 +40,7 @@
> >> >>#:use-module (gnu packages code)
> >> >>#:use-module (gnu packages check)
> >> >>#:use-module (gnu packages compression)
> >> >> +  #:use-module (gnu packages curl)
> >> >>#:use-module (gnu packages docbook)
> >> >>#:use-module (gnu packages doxygen)
> >> >>#:use-module (gnu packages flex)
> >> >> @@ -46,6 +48,7 @@
> >> >>#:use-module (gnu packages fonts)
> >> >>#:use-module (gnu packages fontutils)
> >> >>#:use-module (gnu packages gcc)
> >> >> +  #:use-module (gnu packages gnupg)
> >> >>#:use-module (gnu packages gettext)
> >> >>#:use-module (gnu packages ghostscript)
> >> >>#:use-module (gnu packages gl)
> >> >> @@ -1078,3 +1081,38 @@ computer's keyboard.")
> >> >>  JACK for audio and ALSA sequencer for MIDI as multimedia 
> >> >> infrastructures and
> >> >>  follows a traditional multi-track tape recorder control paradigm.")
> >> >>  (license license:gpl2+)))
> >> >> +
> >> >> +(define-public pianobar
> >> >> +  (package
> >> >> +(name "pi

Re: [PATCH] gnu: Add pianobar.

[Leo Famulari]

On Sat, Feb 13, 2016 at 05:49:35PM -0500, Leo Famulari wrote:
> On Sat, Feb 13, 2016 at 02:37:05PM -0800, Al McElrath wrote:
> > 
> > I tracked this down a bit further. There is nothing wrong with Pandora's
> > cert. If I specify in pianobar's config file to use:
> > 
> > ca_bundle = /home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt
> 
> Can you share your config, having replaced any personal info with dummy
> content?

Never mind! I made a config with just the ca_bundle line pointing to my
Debian-provided certificates, and I got the expected error about the
login failing due to bad credentials.

In my opinion, if the software provided by the package fails due to lack
of configuration, that's okay.

> 
> > 
> > It works. Note if I 'env | grep SSL' I get:
> > 
> > SSL_CERT_DIR=/home/al/.guix-profile/etc/ssl/certs
> > SSL_CERT_FILE=/home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt
> > 
> > I was under the impression libcurl (which pianobar uses) would honor
> > these variables, but it appears it does not. Or there's something wrong
> > with my guix setup which is running on Arch.
> > 
> > I can patch pianobar to use SSL_CERT_FILE by default, but it seems like
> > there is a larger issue.
> > 
> > 
> > Leo Famulari  writes:
> > 
> > > On Thu, Feb 11, 2016 at 10:00:17PM -0800, Al McElrath wrote:
> > >> 
> > >> Thanks for testing and pointing this out. Real radio stations require
> > >> too much manual effort! That said...
> > >> 
> > >> You should be getting a different error without a login (pandora.com
> > >> requires a login): "(i) Login... Error: Wrong email address or
> > >> password." There is an outstanding issue with pianobar regarding
> > >> Pandora's certificate described here:
> > >> 
> > >> https://github.com/PromyLOPh/pianobar/issues/560
> > >> 
> > >> There is a config file option "ca_file" which I use with a pem file I
> > >> downloaded. There is also a "tls_fingerprint" option described in the
> > >> bug commentary. This broke recently when Pandora changed their
> > >> certificate. It's not specifically a Guix bug.
> > >
> > > To clarify, is my error the result of a configuration problem and a
> > > missing .pem file?
> > >
> > > If so, is that .pem file something we should distribute with our
> > > package?
> > >
> > >> 
> > >> I realize this means pianobar simply doesn't work out of the box. I'm
> > >> open to suggestions on how to make this better.
> > >> 
> > >> 
> > >> Leo Famulari  writes:
> > >> 
> > >> > On Thu, Feb 11, 2016 at 01:36:13PM -0800, Al McElrath wrote:
> > >> >>
> > >> >> Revised patch is attached. I updated/expanded the description and 
> > >> >> fixed
> > >> >> the license. Thanks!
> > >> >>
> > >> >
> > >> >> From 91d5c428428eed605c981a39022fe90e6f98be6d Mon Sep 17 00:00:00 2001
> > >> >> From: Al McElrath 
> > >> >> Date: Fri, 11 Dec 2015 21:22:33 -0800
> > >> >> Subject: [PATCH] gnu: Add pianobar.
> > >> >>
> > >> >> * gnu/packages/music.scm (pianobar): New variable.
> > >> >
> > >> > One nice thing about "real" radio stations is that you don't need an
> > >> > account to listen to them ;) Anyways, I tested it as far as I could:
> > >> >
> > >> > $ pianobar
> > >> > Welcome to pianobar (2015.11.22)! Press ? for a list of commands.
> > >> > [?] Email:
> > >> > [?] Password:
> > >> > (i) Login... Network error: Peer certificate cannot be authenticated 
> > >> > with given CA certificates
> > >> >
> > >> > That error seems to describe a problem unrelated to me not having an
> > >> > account. Can you see what happens when you leave the Email and Password
> > >> > fields blank? Of course, the message might be wrong, and it works if 
> > >> > you
> > >> > have an account.
> > >> >
> > >> > If so, then I think it's okay (but buggy). If not, then it must be
> > >> > finding the certificate store in the environment and we should figure
> > >> > out how to provide it reliably.
> > >> >
> > >> >> ---
> > >> >>  gnu/packages/music.scm | 38 ++
> > >> >>  1 file changed, 38 insertions(+)
> > >> >>
> > >> >> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> > >> >> index a60ba4b..1382b0a 100644
> > >> >> --- a/gnu/packages/music.scm
> > >> >> +++ b/gnu/packages/music.scm
> > >> >> @@ -2,6 +2,7 @@
> > >> >>  ;;; Copyright © 2014 Eric Bavier 
> > >> >>  ;;; Copyright © 2015, 2016 Ricardo Wurmus 
> > >> >>  ;;; Copyright © 2015 Paul van der Walt 
> > >> >> +;;; Copyright © 2016 Al McElrath 
> > >> >>  ;;;
> > >> >>  ;;; This file is part of GNU Guix.
> > >> >>  ;;;
> > >> >> @@ -39,6 +40,7 @@
> > >> >>#:use-module (gnu packages code)
> > >> >>#:use-module (gnu packages check)
> > >> >>#:use-module (gnu packages compression)
> > >> >> +  #:use-module (gnu packages curl)
> > >> >>#:use-module (gnu packages docbook)
> > >> >>#:use-module (gnu packages doxygen)
> > >> >>#:use-module (gnu packages flex)
> > >> >> @@ -46,6 +48,7 @@
> > >> >>#:use-module (gnu packages fonts)
> > >> >>#:use-module (gnu

Re: gnu: Add python-wtforms.

[Christopher Allan Webber]

Leo Famulari writes:

> On Sat, Feb 13, 2016 at 01:55:01PM -0800, Christopher Allan Webber wrote:
>> Christopher Allan Webber writes:
>
> [...]
>
>> >>> * gnu/packages/python.scm (python-wtforms, python2-wtforms): New 
>> >>> variables.
>> >>
>> >> It looks good to me, but I wonder if the python-3 variant really needs
>> >> setuptools? If so, okay. If not, can you use the new python2-variant
>> >> system [0] to provide setuptools for python2-wtforms?
>> >>
>> >> [0] Introduced in 48b311b1b3ba
>> >
>> > Okay, I'll give that a try, and will make an adjustment if appropriate,
>> > and regardless, push after figuring that out.
>> >
>> > Thanks to both of you!
>> 
>> Well, I think it's significant enough of a change where I should get
>> some validation that everything looks right before I push.  New version,
>> ahoy!  Is it ok?
>
> I believe that you still must call 'package-with-python2' in the
> python-2 variant, and the argument to that should be
> (strip-python2-variant python-foo).
>
> I've attached an example commit that seems to work, from one of my WIP
> branches.

Ah, you're right!  New patch!

 - Chris

>From 6d3d18425409a738b6e341d9f1640f048a12300a Mon Sep 17 00:00:00 2001
From: Christopher Allan Webber 
Date: Sat, 13 Feb 2016 13:42:51 -0800
Subject: [PATCH] gnu: Add python-wtforms

* gnu/packages/python.scm (python-wtforms, python2-wtforms): New variables.
---
 gnu/packages/python.scm | 30 ++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 3758eed..c9aabc8 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -7509,3 +7509,33 @@ input.  (Note that this is mostly a legacy library; you may wish to look at
 python-xdo for newer bindings.)")
 (license bsd-3)))
 
+(define-public python-wtforms
+  (package
+(name "python-wtforms")
+(version "2.1")
+(source
+ (origin
+   (method url-fetch)
+   (uri (pypi-uri "WTForms" version ".zip"))
+   (sha256
+(base32
+ "0vyl26y9cg409cfyj8rhqxazsdnd0jipgjw06civhrd53yyi1pzz"
+(build-system python-build-system)
+(native-inputs
+ `(("unzip" ,unzip)))
+(home-page "http://wtforms.simplecodes.com/";)
+(synopsis
+ "Form validation and rendering library for Python web development")
+(description
+ "WTForms is a flexible forms validation and rendering library
+for Python web development.  It is very similar to the web form API
+available in Django, but is a standalone package.")
+(license bsd-3)
+(properties `((python2-variant . ,(delay python2-wtforms))
+
+(define-public python2-wtforms
+  (package
+(inherit (package-with-python2
+  (strip-python2-variant python-wtforms)))
+(name "python2-wtforms")
+(inputs `(("python2-setuptools" ,python2-setuptools)
-- 
2.6.3

Re: gnu: Add python-wtforms.

[Christopher Allan Webber]

Christopher Allan Webber writes:

> Leo Famulari writes:
>
>> On Sat, Feb 13, 2016 at 01:55:01PM -0800, Christopher Allan Webber wrote:
>>> Christopher Allan Webber writes:
>>
>> [...]
>>
>>> >>> * gnu/packages/python.scm (python-wtforms, python2-wtforms): New 
>>> >>> variables.
>>> >>
>>> >> It looks good to me, but I wonder if the python-3 variant really needs
>>> >> setuptools? If so, okay. If not, can you use the new python2-variant
>>> >> system [0] to provide setuptools for python2-wtforms?
>>> >>
>>> >> [0] Introduced in 48b311b1b3ba
>>> >
>>> > Okay, I'll give that a try, and will make an adjustment if appropriate,
>>> > and regardless, push after figuring that out.
>>> >
>>> > Thanks to both of you!
>>> 
>>> Well, I think it's significant enough of a change where I should get
>>> some validation that everything looks right before I push.  New version,
>>> ahoy!  Is it ok?
>>
>> I believe that you still must call 'package-with-python2' in the
>> python-2 variant, and the argument to that should be
>> (strip-python2-variant python-foo).
>>
>> I've attached an example commit that seems to work, from one of my WIP
>> branches.
>
> Ah, you're right!  New patch!
>
>  - Chris

Leo also pointed out that the (name) redefinition might not be needed,
and indeed it wasn't!

And with that, pushed.  Whew!

Re: Massive graph

[Ben Woodcroft]

On 14/02/16 08:45, Leo Famulari wrote:

On Sat, Feb 13, 2016 at 11:30:11PM +0100, Pjotr Prins wrote:

On Sat, Feb 13, 2016 at 11:15:14PM +0100, Ricardo Wurmus wrote:

Pjotr Prins  writes:


I am packaging the software that makes up http://genenetwork.org/.

   http://biobeat.org/gn2.svg

Woah!  Nice how this gets close to the complexity of gene networks,
e.g. this one: http://www.john.ranola.org/Files/RadNetFull.pdf

We could do a hair-ball of the full Guix repository at some point :)

I've killed `dot` after several minutes of trying to render a really
large graph in PNG format. I wonder if it's up to the task!
From memory of probably outdated versions, svg output is much more 
efficient, maybe give that a go.


ben

Re: Massive graph

[Leo Famulari]

On Sun, Feb 14, 2016 at 09:13:28AM +1000, Ben Woodcroft wrote:
> 
> 
> On 14/02/16 08:45, Leo Famulari wrote:
> >On Sat, Feb 13, 2016 at 11:30:11PM +0100, Pjotr Prins wrote:
> >>On Sat, Feb 13, 2016 at 11:15:14PM +0100, Ricardo Wurmus wrote:
> >>>Pjotr Prins  writes:
> >>>
> I am packaging the software that makes up http://genenetwork.org/.
> 
>    http://biobeat.org/gn2.svg
> >>>Woah!  Nice how this gets close to the complexity of gene networks,
> >>>e.g. this one: http://www.john.ranola.org/Files/RadNetFull.pdf
> >>We could do a hair-ball of the full Guix repository at some point :)
> >I've killed `dot` after several minutes of trying to render a really
> >large graph in PNG format. I wonder if it's up to the task!
> From memory of probably outdated versions, svg output is much more
> efficient, maybe give that a go.

I noticed it was a lot easier on my web browser! I will use it in the
future.

> 
> ben

Re: [PATCH] gnu: Add pianobar.

[Leo Famulari]

On Sat, Feb 13, 2016 at 05:56:16PM -0500, Leo Famulari wrote:
> On Sat, Feb 13, 2016 at 05:49:35PM -0500, Leo Famulari wrote:
> > On Sat, Feb 13, 2016 at 02:37:05PM -0800, Al McElrath wrote:
> > > 
> > > I tracked this down a bit further. There is nothing wrong with Pandora's
> > > cert. If I specify in pianobar's config file to use:
> > > 
> > > ca_bundle = /home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt
> > 
> > Can you share your config, having replaced any personal info with dummy
> > content?
> 
> Never mind! I made a config with just the ca_bundle line pointing to my
> Debian-provided certificates, and I got the expected error about the
> login failing due to bad credentials.
> 
> In my opinion, if the software provided by the package fails due to lack
> of configuration, that's okay.
> 
> > 
> > > 
> > > It works. Note if I 'env | grep SSL' I get:
> > > 
> > > SSL_CERT_DIR=/home/al/.guix-profile/etc/ssl/certs
> > > SSL_CERT_FILE=/home/al/.guix-profile/etc/ssl/certs/ca-certificates.crt
> > > 
> > > I was under the impression libcurl (which pianobar uses) would honor
> > > these variables, but it appears it does not. Or there's something wrong
> > > with my guix setup which is running on Arch.
> > > 
> > > I can patch pianobar to use SSL_CERT_FILE by default, but it seems like
> > > there is a larger issue.

I think it must be some issue with the way that pianobar uses curl.
The curl provided by Guix has no trouble finding my certificates.

Pushed as aa0edd3f5. Thanks for the patch!

> > > 
> > > 
> > > Leo Famulari  writes:
> > > 
> > > > On Thu, Feb 11, 2016 at 10:00:17PM -0800, Al McElrath wrote:
> > > >> 
> > > >> Thanks for testing and pointing this out. Real radio stations require
> > > >> too much manual effort! That said...
> > > >> 
> > > >> You should be getting a different error without a login (pandora.com
> > > >> requires a login): "(i) Login... Error: Wrong email address or
> > > >> password." There is an outstanding issue with pianobar regarding
> > > >> Pandora's certificate described here:
> > > >> 
> > > >> https://github.com/PromyLOPh/pianobar/issues/560
> > > >> 
> > > >> There is a config file option "ca_file" which I use with a pem file I
> > > >> downloaded. There is also a "tls_fingerprint" option described in the
> > > >> bug commentary. This broke recently when Pandora changed their
> > > >> certificate. It's not specifically a Guix bug.
> > > >
> > > > To clarify, is my error the result of a configuration problem and a
> > > > missing .pem file?
> > > >
> > > > If so, is that .pem file something we should distribute with our
> > > > package?
> > > >
> > > >> 
> > > >> I realize this means pianobar simply doesn't work out of the box. I'm
> > > >> open to suggestions on how to make this better.
> > > >> 
> > > >> 
> > > >> Leo Famulari  writes:
> > > >> 
> > > >> > On Thu, Feb 11, 2016 at 01:36:13PM -0800, Al McElrath wrote:
> > > >> >>
> > > >> >> Revised patch is attached. I updated/expanded the description and 
> > > >> >> fixed
> > > >> >> the license. Thanks!
> > > >> >>
> > > >> >
> > > >> >> From 91d5c428428eed605c981a39022fe90e6f98be6d Mon Sep 17 00:00:00 
> > > >> >> 2001
> > > >> >> From: Al McElrath 
> > > >> >> Date: Fri, 11 Dec 2015 21:22:33 -0800
> > > >> >> Subject: [PATCH] gnu: Add pianobar.
> > > >> >>
> > > >> >> * gnu/packages/music.scm (pianobar): New variable.
> > > >> >
> > > >> > One nice thing about "real" radio stations is that you don't need an
> > > >> > account to listen to them ;) Anyways, I tested it as far as I could:
> > > >> >
> > > >> > $ pianobar
> > > >> > Welcome to pianobar (2015.11.22)! Press ? for a list of commands.
> > > >> > [?] Email:
> > > >> > [?] Password:
> > > >> > (i) Login... Network error: Peer certificate cannot be authenticated 
> > > >> > with given CA certificates
> > > >> >
> > > >> > That error seems to describe a problem unrelated to me not having an
> > > >> > account. Can you see what happens when you leave the Email and 
> > > >> > Password
> > > >> > fields blank? Of course, the message might be wrong, and it works if 
> > > >> > you
> > > >> > have an account.
> > > >> >
> > > >> > If so, then I think it's okay (but buggy). If not, then it must be
> > > >> > finding the certificate store in the environment and we should figure
> > > >> > out how to provide it reliably.
> > > >> >
> > > >> >> ---
> > > >> >>  gnu/packages/music.scm | 38 ++
> > > >> >>  1 file changed, 38 insertions(+)
> > > >> >>
> > > >> >> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> > > >> >> index a60ba4b..1382b0a 100644
> > > >> >> --- a/gnu/packages/music.scm
> > > >> >> +++ b/gnu/packages/music.scm
> > > >> >> @@ -2,6 +2,7 @@
> > > >> >>  ;;; Copyright © 2014 Eric Bavier 
> > > >> >>  ;;; Copyright © 2015, 2016 Ricardo Wurmus 
> > > >> >>  ;;; Copyright © 2015 Paul van der Walt 
> > > >> >> +;;; Copyright © 2016 Al McElrath 
> > > >> >>  ;;;
> > > >> >>  ;;; This file is part of

[PATCH] gnu: Add baobab.

[Jochem Raat]

Hi,

Here's my first package, feedback appreciated. This is my first time
sending a patch via email, so please tell me if I did it wrong.

>From e1746af39470ff85937e1cf7f8d62b7c56b90cef Mon Sep 17 00:00:00 2001
From: Jochem Raat 
Date: Sat, 13 Feb 2016 22:03:39 +0100
Subject: [PATCH] gnu: Add baobab.

* gnu/packages/gnome.scm (baobab): New variable.
---
 gnu/packages/gnome.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 49e6197..170e8ee 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -13,6 +13,7 @@
 ;;; Copyright © 2015 David Thompson 
 ;;; Copyright © 2015, 2016 Efraim Flashner 
 ;;; Copyright © 2016 Rene Saavedra 
+;;; Copyright © 2016 Jochem Raat 
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -4648,3 +4649,35 @@ as SASL, TLS and VeNCrypt.  Additionally it supports 
encoding extensions.")
 design and behaviour, giving the user a simple way to navigate and manage its
 files.")
 (license license:gpl2+)))
+
+(define-public baobab
+  (package
+(name "baobab")
+(version "3.18.1")
+(source (origin
+  (method url-fetch)
+  (uri (string-append
+"mirror://gnome/sources/" name "/"
+(version-major+minor version) "/"
+name "-" version ".tar.xz"))
+  (sha256
+   (base32
+"1da4bdkw5bnxansl1xr4lb03d6f4h0a0qaba8i3p3rwhcd191b62"
+(build-system glib-or-gtk-build-system)
+(native-inputs
+ `(("intltool" ,intltool)
+   ("pkg-config" ,pkg-config)
+   ("itstool" ,itstool)
+   ("xmllint" ,libxml2)
+   ("glib" ,glib "bin")
+   ("vala" ,vala)))
+(inputs
+ `(("gtk+" ,gtk+)))
+(synopsis "Disk usage analyzer for GNOME")
+(description
+ "baobab (Disk Usage Analyzer) is a graphical application to analyse disk
+usage in any Gnome environment.  It can easily scan device volumes or a
+specific user-requested directory branch (local or remote).  Once the scan is
+complete it provides a graphical representation of each selected folder.")
+(home-page "https://wiki.gnome.org/Apps/Baobab";)
+(license license:gpl2+)))
-- 
2.6.3

Inherit inputs when unnecessary in these python2 packages?

[Christopher Allan Webber]

I'm doing a lot of packaging for Python packages right now.  A lot of
the code looks like this:

> (define-public python-execnet
>   (package
> (name "python-execnet")
> (version "1.4.1")
> (source (origin
>  (method url-fetch)
>  (uri (pypi-uri "execnet" version))
>  (sha256
>   (base32
>"1rpk1vyclhg911p3hql0m0nrpq7q7mysxnaaw6vs29cpa6kx8vgn"
> (build-system python-build-system)
> (native-inputs
>  `(("python-setuptools-scm" ,python-setuptools-scm)))
> (propagated-inputs
>  `(("python-apipkg" ,python-apipkg)))
> (synopsis "Rapid multi-Python deployment")
> (description "Execnet provides a share-nothing model with
> channel-send/receive communication for distributing execution across many
> Python interpreters across version, platform and network barriers.  It has a
> minimal and fast API targetting the following uses:
> @enumerate
> @item distribute tasks to (many) local or remote CPUs
> @item write and deploy hybrid multi-process applications
> @item write scripts to administer multiple environments
> @end enumerate")
> (home-page "http://codespeak.net/execnet/";)
> (license license:expat)
> (properties `((python2-variant . ,(delay python2-execnet))
> 
> (define-public python2-execnet
>   (package
> (inherit (package-with-python2
>   (strip-python2-variant python-execnet)))
> (inputs
>  `(("python2-setuptools" ,python2-setuptools)

As you can see, there is no inputs on python-execnet, so it's not
necessary to include the inputs.  However, if (inputs) were added in the
future, this could lead to a developer mistakenly forgetting to change
the python2 variant.

What's the better approach?

 - Chris

Re: [PATCH] gnu: Add baobab.

[Thompson, David]

On Sat, Feb 13, 2016 at 5:14 PM, Jochem Raat  wrote:
> Hi,
>
> Here's my first package, feedback appreciated. This is my first time
> sending a patch via email, so please tell me if I did it wrong.

Welcome aboard!  The patch looks great!  It built successfully, ran
successfully, and 'guix lint' was happy.  I just made a minor change
to the description field and pushed it to master.  Thank you!

- Dave

Re: Inherit inputs when unnecessary in these python2 packages?

[Leo Famulari]

On Sat, Feb 13, 2016 at 06:10:17PM -0800, Christopher Allan Webber wrote:
> I'm doing a lot of packaging for Python packages right now.  A lot of
> the code looks like this:

[...]

> > (define-public python2-execnet
> >   (package
> > (inherit (package-with-python2
> >   (strip-python2-variant python-execnet)))
> > (inputs
> >  `(("python2-setuptools" ,python2-setuptools)
> 
> As you can see, there is no inputs on python-execnet, so it's not
> necessary to include the inputs.  However, if (inputs) were added in the
> future, this could lead to a developer mistakenly forgetting to change
> the python2 variant.
> 
> What's the better approach?

Take a look at python2-wheel, as in 8ad4ae204f.

Re: [PATCH] gnu: add lispf4

[Nils Gillmann]

Efraim Flashner  writes:

> On Tue, 09 Feb 2016 02:31:51 +0100
> Nils Gillmann  wrote:
>
>> Leo Famulari  writes:
>> 
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>> 
>> Thanks!
>> 
>> But now as I did guix pull and installed it from there, and do
>> not run it inside guix environment, I encounter an error I did
>> not see coming as it worked in the environment.
>> 
>> 
>> niasterisk@khazad-dum ~$ guix package -i lispf4
>> The following package will be installed:
>>lispf40.0.0-1-174d876 
>> /gnu/store/7rkw1rhn4ijap9b936waspki03260lpb-lispf4-0.0.0-1-174d876
>> 
>> 56 packages in profile
>> niasterisk@khazad-dum ~$ lispf4
>> Can't open 'SYSATOMS'
>> niasterisk@khazad-dum ~$ ls -al 
>> /gnu/store/7rkw1rhn4ijap9b936waspki03260lpb-lispf4-0.0.0-1-174d876/bin/
>> total 200
>> dr-xr-xr-x 2 root guixbuild  4096 Jan  1  1970 ./
>> dr-xr-xr-x 3 root guixbuild  4096 Jan  1  1970 ../
>> -r--r--r-- 3 root guixbuild 94972 Jan  1  1970 BASIC.IMG
>> -r-xr-xr-x 7 root guixbuild 92088 Jan  1  1970 lispf4
>> -r--r--r-- 7 root guixbuild  2105 Jan  1  1970 SYSATOMS
>> 
>> I checked ~/.guix-profile/bin/ and it's there, all three files.
>> For what I know, SYSATOMS includes allowed commands, that's it.
>> My guix checkout I develop with is in my home folder, permissions
>> for my daily user and all checked in guix environment.
>> 
>> Any Ideas?
>
> If you replace the install phase sometimes you have to change the permissions
> yourself. Try searching for #o555.

Okay, I just tried #o555 and #o644 in individual builds for both
files. Both builds produced the same mistake which currently
keeps lispf4 from running ("can't open 'SYSATOMS'").

Should I try other permission levels like 755 now until I find
one which works for guix? 644 was the one which worked in $HOME
in debian, that's my only build and succesful run experience with
lispf4.

what do you think?
-- 
ng

Re: [PATCH] gnu: add lispf4

[Nils Gillmann]

Efraim Flashner  writes:

> On Tue, 09 Feb 2016 02:31:51 +0100
> Nils Gillmann  wrote:
>
>> Leo Famulari  writes:
>> 
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>>  [...]  
>> 
>> Thanks!
>> 
>> But now as I did guix pull and installed it from there, and do
>> not run it inside guix environment, I encounter an error I did
>> not see coming as it worked in the environment.
>> 
>> 
>> niasterisk@khazad-dum ~$ guix package -i lispf4
>> The following package will be installed:
>>lispf40.0.0-1-174d876 
>> /gnu/store/7rkw1rhn4ijap9b936waspki03260lpb-lispf4-0.0.0-1-174d876
>> 
>> 56 packages in profile
>> niasterisk@khazad-dum ~$ lispf4
>> Can't open 'SYSATOMS'
>> niasterisk@khazad-dum ~$ ls -al 
>> /gnu/store/7rkw1rhn4ijap9b936waspki03260lpb-lispf4-0.0.0-1-174d876/bin/
>> total 200
>> dr-xr-xr-x 2 root guixbuild  4096 Jan  1  1970 ./
>> dr-xr-xr-x 3 root guixbuild  4096 Jan  1  1970 ../
>> -r--r--r-- 3 root guixbuild 94972 Jan  1  1970 BASIC.IMG
>> -r-xr-xr-x 7 root guixbuild 92088 Jan  1  1970 lispf4
>> -r--r--r-- 7 root guixbuild  2105 Jan  1  1970 SYSATOMS
>> 
>> I checked ~/.guix-profile/bin/ and it's there, all three files.
>> For what I know, SYSATOMS includes allowed commands, that's it.
>> My guix checkout I develop with is in my home folder, permissions
>> for my daily user and all checked in guix environment.
>> 
>> Any Ideas?
>
> If you replace the install phase sometimes you have to change the permissions
> yourself. Try searching for #o555.

Tried 755 for both, still the same.. Has anyone experienced such
an error before?

-- 
ng