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

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

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

Re: [PATCH] gnu: Add slurm.

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

Re: [PATCH] gnu: Add laby.

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

Re: [PATCH] gnu: Add laby.

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. >> *

Re: [PATCH] gnu: Add laby.

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

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

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

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

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

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 ---

[PATCH 3/3] gnu: Add vcsh

* 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 ---

Re: [PATCH] gnu: Add slurm.

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

Massive graph

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.

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

Re: [PATCH] gnu: Add slurm.

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

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

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

Re: [PATCH] gnu: Add slurm.

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 >

[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-system.am | 1 + gnu/packages/parallel.scm

[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-system.am | 1 + gnu/packages/parallel.scm

Re: [PATCH] gnu: Add slurm.

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

[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-system.am | 1 + gnu/packages/parallel.scm

[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-system.am | 1 + gnu/packages/parallel.scm

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

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>"

[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-system.am | 1 + gnu/packages/parallel.scm

Re: [PATCH] gnu: Add slurm.

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.)

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

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

guix system init co-existing with other distros

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

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

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(-) [...] >

[PATCH 0/3] emacs: Few improvements.

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.

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

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

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

* 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

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

* 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(-)

Documentation patches

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

Re: [PATCH] gnu: Add slurm.

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 > >

Re: [PATCH] gnu: Add slurm.

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

Re: [PATCH] gnu: Add slurm.

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

Re: [PATCH] gnu: Add slurm.

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

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

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 >

Re: [PATCH] gnu: Add laby.

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. > *

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

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

Re: [PATCH] gnu: Add tclxml.

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 slurm.

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

Re: [PATCH] gnu: Add laby.

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

Re: [PATCH] gnu: Add laby.

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

Re: [PATCH] gnu: Add laby.

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

Re: [PATCH] gnu: Add erlang.

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 > > +

Re: [PATCH 0/5] python2-variants

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

Re: gnu: Add python-wtforms.

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

Re: Massive graph

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.

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 > >>

[PATCH] gnu: Add libosinfo.

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

Re: Massive graph

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

Re: [PATCH] gnu: Add pianobar.

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

Re: Massive graph

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/. > > > > > >

Re: [PATCH] gnu: Add pianobar.

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,

Re: [PATCH] gnu: Add pianobar.

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 =

Re: gnu: Add python-wtforms.

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

Re: [PATCH] gnu: Add slurm.

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

Re: [PATCH] gnu: Add slurm.

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

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

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

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

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>" >

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

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

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

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) > +

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

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. > --- >

Re: gnu: Add python-wtforms.

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

Re: Massive graph

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/.

Re: Massive graph

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: >

Re: [PATCH] gnu: Add pianobar.

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

Inherit inputs when unnecessary in these python2 packages?

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"

Re: [PATCH] gnu: add lispf4

Efraim Flashner writes: > On Tue, 09 Feb 2016 02:31:51 +0100 > Nils Gillmann wrote: > >> Leo Famulari writes: >> >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> >> Thanks! >>

Re: [PATCH] gnu: Add baobab.

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

Re: Inherit inputs when unnecessary in these python2 packages?

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 > >

Re: [PATCH] gnu: add lispf4

Efraim Flashner writes: > On Tue, 09 Feb 2016 02:31:51 +0100 > Nils Gillmann wrote: > >> Leo Famulari writes: >> >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> [...] >> >> Thanks! >>

[PATCH] gnu: Add baobab.

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: