Re: [PATCH] gnu: Add slurm-llnl.

2016-02-12 Thread Pjotr Prins
On Fri, Feb 12, 2016 at 01:30:10PM +0100, Andreas Enge wrote: > On Fri, Feb 12, 2016 at 01:20:42PM +0100, Pjotr Prins wrote: > > Now I remember. The python3 package does not have python in the path. > > So I patch it with python3. > > Alternatively, I would suggest to use python-wrapper; it is exa

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-12 Thread Pjotr Prins
Hopefully taken all comments into account. For better or for worse, at least this works. Pj. >From 5941b6b2624ee18444c2fa9f0805136cf6924a88 Mon Sep 17 00:00:00 2001 From: Pjotr Prins Date: Fri, 12 Feb 2016 13:26:56 +0100 Subject: [PATCH] gnu: Add slurm-wlm. * gnu/packages/parallel.scm (slurm-w

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-12 Thread Andreas Enge
On Fri, Feb 12, 2016 at 01:20:42PM +0100, Pjotr Prins wrote: > Now I remember. The python3 package does not have python in the path. > So I patch it with python3. Alternatively, I would suggest to use python-wrapper; it is exactly python-3 with symlinks python->python3 etc. added. Andreas

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-12 Thread Pjotr Prins
On Thu, Feb 11, 2016 at 09:15:34AM +0100, Pjotr Prins wrote: > > > + (build-system gnu-build-system) > > > + (arguments > > > +`(#:configure-flags '("--enable-pam") > > > + #:phases > > > + (modify-phases %standard-phases > > > + (add-before > > > +'configure 'rewrit

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Efraim Flashner
On Wed, 10 Feb 2016 18:36:48 +0100 Pjotr Prins wrote: > * gnu/packages/parallel.scm (slurm-llnl): New variable. > --- > gnu/packages/parallel.scm | 51 > --- > 1 file changed, 48 insertions(+), 3 deletions(-) > > diff --git a/gnu/packages/parallel.sc

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Eric Bavier
On Thu, 11 Feb 2016 12:11:27 +0100 Pjotr Prins wrote: > On Thu, Feb 11, 2016 at 11:08:01AM +0100, Andreas Enge wrote: > > > > I think we should rather use "slurm-wlm" if we are trying to > > > > dissambiguate with the "Slurm the Realtime network interface monitor". > > > The website https://compu

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Pjotr Prins
On Thu, Feb 11, 2016 at 03:21:21PM +0100, Andreas Enge wrote: > On Thu, Feb 11, 2016 at 12:11:27PM +0100, Pjotr Prins wrote: > > There is already a slurm package. > > Did I overlook anything? No, I also checked. The point is that https://packages.debian.org/sid/slurm is a network tool. So, it is

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Andreas Enge
On Thu, Feb 11, 2016 at 12:11:27PM +0100, Pjotr Prins wrote: > There is already a slurm package. Apparently not in current master: $ guix package -A slurm does not show anything, not even $ grep slurm gnu/packages/*.scm Did I overlook anything? Andreas

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Pjotr Prins
On Thu, Feb 11, 2016 at 11:08:01AM +0100, Andreas Enge wrote: > > > I think we should rather use "slurm-wlm" if we are trying to > > > dissambiguate with the "Slurm the Realtime network interface monitor". > > The website https://computing.llnl.gov/linux/slurm/ suggests > > otherwise. Also archlinu

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Andreas Enge
On Thu, Feb 11, 2016 at 09:15:34AM +0100, Pjotr Prins wrote: > I am under the impression one should fix one package at a time. What > is the policy anyway? With or without namespace? Without the namespace > there is a naming conflict between the openssl license and the openssl > package (in tls).

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Ricardo Wurmus
Pjotr Prins writes: > On Wed, Feb 10, 2016 at 10:33:30PM -0600, Eric Bavier wrote: >> On Wed, 10 Feb 2016 18:36:48 +0100 >> Pjotr Prins wrote: >> >> > * gnu/packages/parallel.scm (slurm-llnl): New variable. >> > --- >> > gnu/packages/parallel.scm | 51 >> > +++

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Pjotr Prins
On Wed, Feb 10, 2016 at 10:33:30PM -0600, Eric Bavier wrote: > On Wed, 10 Feb 2016 18:36:48 +0100 > Pjotr Prins wrote: > > > * gnu/packages/parallel.scm (slurm-llnl): New variable. > > --- > > gnu/packages/parallel.scm | 51 > > --- > > 1 file changed

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-11 Thread Pjotr Prins
Thanks for the review. This apparently a problematic package. Let's try fix a few things at a time. On Wed, Feb 10, 2016 at 10:04:47PM -0600, Eric Bavier wrote: > >#:use-module (guix download) > > + #:use-module (guix licenses) > > + #:use-module ((guix licenses) #:prefix license:) > >#:

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-10 Thread Eric Bavier
On Wed, 10 Feb 2016 18:36:48 +0100 Pjotr Prins wrote: > * gnu/packages/parallel.scm (slurm-llnl): New variable. > --- > gnu/packages/parallel.scm | 51 > --- > 1 file changed, 48 insertions(+), 3 deletions(-) [...] > + (license license:openssl))) T

Re: [PATCH] gnu: Add slurm-llnl.

2016-02-10 Thread Eric Bavier
On Wed, 10 Feb 2016 18:36:48 +0100 Pjotr Prins wrote: > * gnu/packages/parallel.scm (slurm-llnl): New variable. > --- > gnu/packages/parallel.scm | 51 > --- > 1 file changed, 48 insertions(+), 3 deletions(-) > > diff --git a/gnu/packages/parallel.sc

[PATCH] gnu: Add slurm-llnl.

2016-02-10 Thread Pjotr Prins
* gnu/packages/parallel.scm (slurm-llnl): New variable. --- gnu/packages/parallel.scm | 51 --- 1 file changed, 48 insertions(+), 3 deletions(-) diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm index 8f63bda..03ee460 100644 --- a/gnu/p