Re: [PATCH] gnu: Add tcllib.

2016-02-12 Thread Ludovic Courtès
Leo Famulari skribis: > Does anyone object to having a slash in the license variable name? I > don't know whether or not its "safe" [0]. There’s no problem with that, pretty much anything can be used in Scheme identifiers. Ludo’.

Re: [PATCH] gnu: Add tcllib.

2016-02-12 Thread Ludovic Courtès
Hi! I agree with Pjotr: we need to pay special attention to the way we review patches coming from newcomers. If a first patch triggers loads of whitespace issues, that can be discouraging. So I think it’s a good idea to take extra time to fix trivial things in patches from newcomers and just

Re: PulseAudio

2016-02-12 Thread Ludovic Courtès
Leo Famulari skribis: > All kidding aside, I'm happy to push this with PulseAudio support, *if* > a GuixSD user can confirm that it works. I don't have sound on my GuixSD > installation. I use it all the time and it works, except for IceCat & co. as Chris notes, which try to

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

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

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

2016-02-12 Thread Pjotr Prins
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

Re: [PATCH 3/3] system: Add iproute to '%base-packages'.

2016-02-12 Thread Alex Kost
Efraim Flashner (2016-02-11 14:15 +0300) wrote: > On Thu, 11 Feb 2016 14:03:19 +0300 > Alex Kost wrote: > >> * gnu/system.scm (%base-packages): Add iproute. >> --- >> gnu/system.scm | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/gnu/system.scm

Re: GSoC ideas

2016-02-12 Thread Alex Sassmannshausen
Andreas Enge writes: > On Thu, Feb 11, 2016 at 01:46:30PM +0200, Manolis Ragkousis wrote: >> As I am still a student I would like to suggest continuing the project >> of porting Guix to GNU/Hurd in this GSoC as well, with the objective of >> having a working GuixSD/Hurd by the end of the summer.

Re: PulseAudio

2016-02-12 Thread Alex Sassmannshausen
Hello, Ricardo Wurmus writes: > Ludovic Courtès writes: > >> Leo Famulari skribis: >> >>> All kidding aside, I'm happy to push this with PulseAudio support, *if* >>> a GuixSD user can confirm that it works. I don't have sound on my GuixSD >>> installation. >>

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

regarding commit 3ccdd4305ef76e7c5c10554a9d1c338c3362078f

2016-02-12 Thread Nils Gillmann
I believe to keep it consistent (sorry for being inconsistent in the beginning), my email address should be a valid one (change to ) and my name change from Ni* to Nils Gillmann in the patch commited to gnunet.scm. I wonder, is this too small a change

Re: [PATCH 2/2] install: Add btrfs-progs to the image.

2016-02-12 Thread Ludovic Courtès
Leo Famulari skribis: > On Wed, Feb 10, 2016 at 05:01:47AM +0100, tobias.geerinckx.r...@gmail.com > wrote: >> From: Tobias Geerinckx-Rice >> >> * gnu/system/install.scm (installation-os): Add BTRFS-PROGS. > > This adds only a few megabyes

Re: PulseAudio

2016-02-12 Thread Ricardo Wurmus
Ludovic Courtès writes: > Leo Famulari skribis: > >> All kidding aside, I'm happy to push this with PulseAudio support, *if* >> a GuixSD user can confirm that it works. I don't have sound on my GuixSD >> installation. > > I use it all the time and it works,

Re: [PATCH] gnu: Add tcllib.

2016-02-12 Thread Jan Nieuwenhuizen
Leo Famulari writes: >> Can you check if it works for you? I don't really know how to test this >> package. > > My apologies is that patch series does not apply. I have attached > another. Works for me; however, 0002-gnu-tcl-Use-tcl-tk-license.patch somehow includes a bit from my tclxml patch

Re: Adding btrfs-progs

2016-02-12 Thread Tobias Geerinckx-Rice
On 12 February 2016 at 03:41, Leo Famulari wrote: > Thanks for the patch! Pushed as fc686f9a19. \o/ > On Wed, Feb 10, 2016 at 05:01:45AM +0100, tobias.geerinckx.r...@gmail.com > wrote: >> [1]: , >> although this

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Andreas Enge
Hello, On Thu, Feb 11, 2016 at 11:21:30PM +0100, Jan Nieuwenhuizen wrote: > Ah... And also, ocaml-findlib was missing and lablgtk did not compile > cmxa files, which laby needs. So, three patches to get a running Laby. > + (native-search-paths > +(list (search-path-specification > +

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Andreas Enge
Hello again, I notice that this: > Subject: [PATCH 2/3] gnu: Add ocaml-findlib. is independent, so I also had a look. In fact, it contains the copyright line, so that is already settled! At the top of the patch file, could you add a quick explanation where it comes from and what it is needed

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Jan Nieuwenhuizen
Andreas Enge writes: >> + (system* >> +"./configure" >> +"-bindir" (string-append out "/bin") >> +"-config" (string-append out "/etc/ocamfind.conf") >> +"-mandir" (string-append

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

2016-02-12 Thread Andreas Enge
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

gnu: Add python-wtforms.

2016-02-12 Thread Christopher Allan Webber
>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. ---

Re: [PATCH] gnu: Add gnome.

2016-02-12 Thread Thompson, David
On Fri, Feb 12, 2016 at 11:33 PM, 宋文武 wrote: > * gnu/packages/gnome.scm (gnome): New variable. > --- > gnu/packages/gnome.scm | 41 + > 1 file changed, 41 insertions(+) This is awesome! I'm reconfiguring my system now to see how it all

[PATCH] gnu: Add gnome.

2016-02-12 Thread 宋文武
* gnu/packages/gnome.scm (gnome): New variable. --- gnu/packages/gnome.scm | 41 + 1 file changed, 41 insertions(+) diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm index 49e6197..dbf8ffe 100644 --- a/gnu/packages/gnome.scm +++

Re: [PATCH] gnu: guile-next: Set correct native search paths.

2016-02-12 Thread Jan Nieuwenhuizen
Thompson, David writes: > I've started building my own projects with Guile 2.1.2 and I noticed > that the search paths needed adjustment in order to find modules. Ah...great I've been wondering about this; assuming everyone would be running latest guile. Do we want a guile-next-lib, I am using

[PATCH] gnu: guile-next: Set correct native search paths.

2016-02-12 Thread Thompson, David
I've started building my own projects with Guile 2.1.2 and I noticed that the search paths needed adjustment in order to find modules. - Dave From 7b26acb50b60a68b1c64ff38a663212e03b0bb7d Mon Sep 17 00:00:00 2001 From: David Thompson Date: Sat, 13 Feb 2016 00:08:22

Re: [PATCH] gnu: Add laby.

2016-02-12 Thread Jan Nieuwenhuizen
Andreas Enge writes: > At the top of the patch file, could you add a quick explanation where it comes > from and what it is needed for? Is it Guix specific, or an upstream > bug? Ah yes it is inpired by the Nix package, it is store-specific. Ocaml wants its "core" libraries all in the same

Re: [PATCH] segfaulting gnunet-gtk.

2016-02-12 Thread Ricardo Wurmus
Ludovic Courtès writes: >> From 29708ddfff95b06ae2e7076d1e2fd13d93b782b9 Mon Sep 17 00:00:00 2001 >> From: Ni* Gillmann >> Date: Wed, 13 Jan 2016 18:36:31 +0100 >> Subject: [PATCH 2/2] gnu: Add gnunet-gtk. >> >> * gnu/packages/gnunet.scm (gnunet-gtk): New

Re: gnu: Add python-wtforms.

2016-02-12 Thread Leo Famulari
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 > > *

Re: [PATCH] gnu: Add tcllib.

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 05:33:29PM +0100, Jan Nieuwenhuizen wrote: > Leo Famulari writes: > > >> Can you check if it works for you? I don't really know how to test this > >> package. > > > > My apologies is that patch series does not apply. I have attached > > another. > > Works for me;

Re: [PATCH] gnu: Add tcllib.

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 05:23:22PM -0500, Leo Famulari wrote: > On Fri, Feb 12, 2016 at 05:33:29PM +0100, Jan Nieuwenhuizen wrote: > > Leo Famulari writes: > > > > >> Can you check if it works for you? I don't really know how to test this > > >> package. > > > > > > My apologies is that patch

Re: regarding commit 3ccdd4305ef76e7c5c10554a9d1c338c3362078f

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 02:31:20PM +0100, Nils Gillmann wrote: > I believe to keep it consistent (sorry for being inconsistent in > the beginning), my email address should be a valid one > (change to ) and my > name change from Ni* to Nils Gillmann in

Re: gnu: Add python-wtforms.

2016-02-12 Thread Thompson, David
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

Re: Adding btrfs-progs

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 05:03:16PM +0100, Tobias Geerinckx-Rice wrote: > On 12 February 2016 at 03:41, Leo Famulari wrote: > > Thanks for the patch! Pushed as fc686f9a19. > > \o/ > > > On Wed, Feb 10, 2016 at 05:01:45AM +0100, tobias.geerinckx.r...@gmail.com > > wrote: > >>

Re: [PATCH 2/2] install: Add btrfs-progs to the image.

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 10:07:35AM +0100, Ludovic Courtès wrote: > Leo Famulari skribis: > > > On Wed, Feb 10, 2016 at 05:01:47AM +0100, tobias.geerinckx.r...@gmail.com > > wrote: > >> From: Tobias Geerinckx-Rice > >> > >> *

Re: [PATCH] gnu: Add pianobar.

2016-02-12 Thread Leo Famulari
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

Re: [PATCH 0/5] python2-variants

2016-02-12 Thread Leo Famulari
On Wed, Feb 10, 2016 at 10:21:17PM +0100, Ludovic Courtès wrote: > Leo Famulari skribis: > > > These patches apply the new python2-variant system introduced in > > 1be83341 to python2-{cryptography, jsonschema, wheel, requests}, and > > then split acme into python-acme and

Re: [PATCH] gnu: Add tclxml.

2016-02-12 Thread Leo Famulari
On Sat, Feb 13, 2016 at 01:07:27AM +0100, Jan Nieuwenhuizen wrote: > Jan Nieuwenhuizen writes: > > Hi, > > Rebased against master, cleanups. > > Greetings, > Jan > > From bf4e4975f7470d0f82dd2451d45de99e57193efd Mon Sep 17 00:00:00 2001 > From: Jan Nieuwenhuizen > Date: Sat,

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

2016-02-12 Thread Christopher Allan Webber
Thompson, David writes: > On Fri, Feb 12, 2016 at 7:13 PM, Christopher Allan Webber > wrote: >> Ludovic Courtès writes: >> >>> Pjotr Prins skribis: >>> Patch b24765139c8940541b23f84592d3580d53f71d71 (define-public sqlite

Re: [PATCH] gnu: Add spice-protocol.

2016-02-12 Thread Leo Famulari
On Thu, Feb 11, 2016 at 07:56:56AM -0600, ren...@openmailbox.org wrote: > Hi, > > i attached spice-protocol patch for review. Thanks for the patch! > > SPICE protocol, is a required package to package spice-gtk(requiered for > GNOME Vinagre). > > Considerations: > > a) I use the source:

Re: [PATCH] gnu: Add tclxml.

2016-02-12 Thread Jan Nieuwenhuizen
Jan Nieuwenhuizen writes: Hi, Rebased against master, cleanups. Greetings, Jan >From bf4e4975f7470d0f82dd2451d45de99e57193efd Mon Sep 17 00:00:00 2001 From: Jan Nieuwenhuizen Date: Sat, 13 Feb 2016 00:27:17 +0100 Subject: [PATCH] gnu: Add tclxml. *

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

2016-02-12 Thread Thompson, David
On Fri, Feb 12, 2016 at 7:13 PM, Christopher Allan Webber wrote: > Ludovic Courtès writes: > >> Pjotr Prins skribis: >> >>> Patch b24765139c8940541b23f84592d3580d53f71d71 >>> >>> (define-public sqlite >>>(package >>> (name "sqlite") >>>

Re: [PATCH] gnu: Add erlang.

2016-02-12 Thread Leo Famulari
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. I wonder, do you happen to have any Erlang programs packaged that I can test this with? Or, would they require the creation of a full

Re: regarding commit 3ccdd4305ef76e7c5c10554a9d1c338c3362078f

2016-02-12 Thread Nils Gillmann
Leo Famulari writes: > On Fri, Feb 12, 2016 at 02:31:20PM +0100, Nils Gillmann wrote: >> I believe to keep it consistent (sorry for being inconsistent in >> the beginning), my email address should be a valid one >> (change to ) and

Re: gnu: Add python-wtforms.

2016-02-12 Thread Christopher Allan Webber
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,

Re: gnu: Add python-wtforms.

2016-02-12 Thread Leo Famulari
On Fri, Feb 12, 2016 at 03:42:43PM -0800, Christopher Allan Webber wrote: > 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.

Re: [PATCH] Update lisp.scm CCL and SBCL to latest versions

2016-02-12 Thread Leo Famulari
On Thu, Feb 11, 2016 at 07:50:18PM -0600, Christopher W Carpenter wrote: > From cc280d52f5d7724c86ce365e541330fd85c3dcd0 Mon Sep 17 00:00:00 2001 > From: "Christopher W. Carpenter (https://mordocai.net/gpg.html)" > > Date: Wed, 10 Feb 2016 18:45:56 -0600 > Subject: [PATCH

Re: Package transformations

2016-02-12 Thread Christopher Allan Webber
Ludovic Courtès writes: > Hello! > > I’ve pushed my “train hacks”, which are a continuation of refactoring > work I had done in December (ca. 64ec0e2): > > • ‘guix build’ has a new ‘--with-input’ option to rewrite the > dependency graph so you can run: > > guix build guix

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

2016-02-12 Thread Christopher Allan Webber
Ludovic Courtès writes: > Pjotr Prins skribis: > >> Patch b24765139c8940541b23f84592d3580d53f71d71 >> >> (define-public sqlite >>(package >> (name "sqlite") >> - (version "3.8.11.1") >> + (version "3.10.0") >> (source (origin >> >> is the cause of