On Thu, May 19, 2016 at 10:58:08AM +0200, Ricardo Wurmus wrote:
> From: Ricardo Wurmus
>
> * gnu/packages/parallel.scm (slurm-drmaa): New variable.
> ---
> gnu/packages/parallel.scm | 29 +
> 1 file changed, 29 insertions(+)
>
> diff --git a/gnu/packages/parallel.scm
From: Ricardo Wurmus
* gnu/packages/parallel.scm (slurm-drmaa): New variable.
---
gnu/packages/parallel.scm | 29 +
1 file changed, 29 insertions(+)
diff --git a/gnu/packages/parallel.scm b/gnu/packages/parallel.scm
index 828bd26..b0984a8 100644
--- a/gnu/packages/pa
On Tue, Feb 16, 2016 at 08:21:07AM +0100, Pjotr Prins wrote:
> > A number of configure flags were missing - without them, the inputs
> > would not be found in their non-standard location.
> Hmmm. It compiled, the tools ran...
They probably concerned optional inputs, that maybe you did not need
in
On Mon, Feb 15, 2016 at 10:51:00PM +0100, Andreas Enge wrote:
> On Mon, Feb 15, 2016 at 01:09:01PM +0100, Pjotr Prins wrote:
> > New version cleaned up (using python-wrapper) and patch created
> > with latest git and en_US.utf8 Guix style...
>
> Well, it turned out it was still in iso-latin-1, and
On Mon, Feb 15, 2016 at 01:09:01PM +0100, Pjotr Prins wrote:
> New version cleaned up (using python-wrapper) and patch created
> with latest git and en_US.utf8 Guix style...
Well, it turned out it was still in iso-latin-1, and patching failed again...
Anyway, I applied the modifications and expand
..
Ah yes, lint is happy too. I am proud of this one. Hope you are.
Pj.
--
>From 72513ea3f730323dc8977d97c9d4ddd691f6d48a Mon Sep 17 00:00:00 2001
From: Pjotr Prins
Date: Mon, 15 Feb 2016 10:14:44 +
Subject: [PATCH] gnu: Add slurm.
* gnu/packages/parallel.scm (slurm): New variab
On Sun, Feb 14, 2016 at 03:22:56PM +0100, Andreas Enge wrote:
> On Sat, Feb 13, 2016 at 03:40:55PM -0500, Leo Famulari wrote:
> > 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
On Sat, Feb 13, 2016 at 03:40:55PM -0500, Leo Famulari wrote:
> 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.
The last line of your
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 a
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
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
From·ef998204cde384d35b605509be09b25170c28fb8·Mon·Sep·17·00:00:00·2001
2
Message-Id:·<788a22631663d1e4060ebe87b4272c590f138a25.1455395436.git.leo@famulari.name>
2
From:·Pjotr·Prins·<pjotr.public01@thebird.nl>
3
From:·Leo·Famulari·<leo@
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 explain
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-co
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
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
* 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
* 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
* 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
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
* 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
* 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
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
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
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 package
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 exe
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 char
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/para
Hi Piotr,
"git am" fails on current master with this patch, claiming an error at
line 39:
+ #:use-module (gnu packages tcl)
+ #:use-module (gnu packages tls))
etc.
Did we not agree in the end that the package should simply be called slurm?
Could you please send a patch that applies to current
Removed a few extraneous comments and changed subject.
>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-wlm): New variable.
---
gnu/packa
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
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 (sl
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
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
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
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
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
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
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
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).
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
>> > +++
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
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:)
> >#:
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
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
* 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
on IRC.
> From 53832a06a61801e9996252506aacad468d54f071 Mon Sep 17 00:00:00 2001
> From: swedebugia
> Date: Mon, 28 Dec 2015 15:04:21 +0100
> Subject: [PATCH] gnu: Add slurm
>
> ---
> gnu/packages/networking.scm | 21 +
> 1 file changed, 21 insertio
Hello,
Welcome, and thanks for the patch.
> If okay I would like to avoid providing my legal name.
Using a pseudonym is fine.
On Mon, 28 Dec 2015 23:00:14 +0100
swedebu...@riseup.net wrote:
> gnu/packages/networking.scm | 21 +
> 1 file changed, 21 insertions(+)
Please se
My first patch ever to a software project. How did I do?
If okay I would like to avoid providing my legal name.From 53832a06a61801e9996252506aacad468d54f071 Mon Sep 17 00:00:00 2001
From: swedebugia
Date: Mon, 28 Dec 2015 15:04:21 +0100
Subject: [PATCH] gnu: Add slurm
---
gnu/packages
49 matches
Mail list logo
