Re: [PATCH] gnu: bash-completion: Update to 2.3.

2016-08-07 Thread Tomáš Čech
On Sat, Aug 06, 2016 at 11:06:05PM -0400, Leo Famulari wrote: On Fri, Aug 05, 2016 at 08:53:46PM +0200, Andreas Enge wrote: The autoreconf phase is strange. Why is it suddenly needed? If there is no way around it, you should at least add a comment. I notice that when projects move to GitHub, t

Re: [PATCH] gnunet-svn, gnunet-gtk-svn

2016-08-07 Thread ng0
ng0 writes: > The next 2.5 days will be unpleasant for me and probably difficult to > focus, but I know how to fix the 3 failing tests for us. > I'll have this fixed in thex next days. > > Appended is the failing tests output (for me as a backup) and the > combined gnunet-svn + gnunet-gtk-svn pat

[PATCHv2 1/2] gnu: bash-completion: Modify to modify-phases.

2016-08-07 Thread Tomáš Čech
* gnu/packages/bash.scm(bash-completion): Use modify-phases instead of alist-cons-after. --- gnu/packages/bash.scm | 46 +++--- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/gnu/packages/bash.scm b/gnu/packages/bash.scm index ff3acfc..c00

[PATCHv2 2/2] gnu: bash-completion: Update to 2.3. Update the patch.

2016-08-07 Thread Tomáš Čech
* gnu/packages/bash.scm(bash-completion): Update to 2.3. * gnu/packages/patches/bash-completion-directories.patch: New version of the patch for the same purpose. --- gnu/packages/bash.scm | 8 ++--- .../patches/bash-completion-directories.patch | 40 +--

[PATCH] Update girara to 0.2.6 (pre-requirement for zathura update)

2016-08-07 Thread ng0
From 91c1327d89c5d746c03ec8ce004317dfd0a63c88 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 10:05:56 + Subject: [PATCH 2/6] gnu: girara: Update to 0.2.6. * gnu/packages/gtk.scm (girara): Update to 0.2.6. --- gnu/packages/gtk.scm | 5 +++-- 1 file changed, 3 insertions(+), 2 deleti

[PATCH] Update zathura to 0.3.6.

2016-08-07 Thread ng0
From f899612384e45521ef85821583418f471619c390 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 09:58:42 + Subject: [PATCH 1/6] gnu: zathura: Update to 0.3.6. * gnu/packages/pdf.scm (zathura): Update to 0.3.6. --- gnu/packages/pdf.scm | 5 +++-- 1 file changed, 3 insertions(+), 2 dele

[PATCH] Update zathura-cb to 0.1.5.

2016-08-07 Thread ng0
From 13a242b2d12ae3c6ea72d6a936d094f0f7f95c21 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 10:10:07 + Subject: [PATCH 3/6] gnu: zathura-cb: Update to 0.1.5. * gnu/packages/pdf.scm (zathura-cb): Update to 0.1.5. --- gnu/packages/pdf.scm | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH] Update zathura-ps to 0.2.3.

2016-08-07 Thread ng0
From 0cc07d898d653d8ed0355479cf3f4530cbec8e95 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 10:12:50 + Subject: [PATCH 4/6] gnu: zathura-ps: Update to 0.2.3. * gnu/packages/pdf.scm (zathura-ps): Update to 0.2.3. --- gnu/packages/pdf.scm | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH] Update zathura-djvu to 0.2.5.

2016-08-07 Thread ng0
From f347977252d2d2f952f921a0a4c96ddaa68ad207 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 10:16:01 + Subject: [PATCH 5/6] gnu: zathura-djvu: Update to 0.2.5. * gnu/packages/pdf.scm (zathura-djvu): Update to 0.2.5. --- gnu/packages/pdf.scm | 4 ++-- 1 file changed, 2 insertions(+

[PATCH] Update zathura-pdf-poppler to 0.2.6.

2016-08-07 Thread ng0
From ae97bfe8f8c0e0b3d6179f8bb299382313a115a4 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 10:18:56 + Subject: [PATCH 6/6] gnu: zathura-pdf-poppler: Update to 0.2.6. * gnu/packages/pdf.scm (zathura-pdf-poppler): Update to 0.2.6. --- gnu/packages/pdf.scm | 4 ++-- 1 file changed,

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread David Craven
If I understand you correctly, you are saying the following: We shouldn't display an url to a tarball that may contain unfree software, because a user might run `guix package -s` download the tarball from the source-uris field and build and run the code, and accidentally run nonfree software on hi

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread Mathieu Lirzin
David Craven writes: > If I understand you correctly, you are saying the following: > > We shouldn't display an url to a tarball that may contain unfree > software, because a user might run `guix package -s` download the > tarball from the source-uris field and build and run the code, and > accid

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread David Craven
I meant the little -s flag... -s, --search=REGEXPsearch in synopsis and description using REGEXP and I meant download, build and run manually outside of guix. It still sounds like that's what you are saying... ;-)

[PATCH] gnu: Add ii.

2016-08-07 Thread ng0
From 76d9f586822ad0a33a12f372715de878d769268b Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 11:35:34 + Subject: [PATCH] gnu: Add ii. * gnu/packages/irc.scm (ii): New variable. --- gnu/packages/irc.scm | 25 + 1 file changed, 25 insertions(+) diff --git a/g

Re: [PACKAGE] sinit

2016-08-07 Thread ng0
Leo Famulari writes: > On Sat, Jul 30, 2016 at 06:19:47PM +0200, Vincent Legoll wrote: >> On Sat, Jul 30, 2016 at 3:46 PM, Vincent Legoll >> wrote: >> > Here is the accompanying patch for sinit to build properly under guix >> > >> > It will be submitted alongside the scm modifs. >> > >> > I may

[PATCH] gnu: dwm: Shorten the description.

2016-08-07 Thread ng0
This description was as long as the package itself. We should not do this, this is enough. Whoever wants to read up on dwm and tiling wm in general can read about it in dwm documentation and other twms. From 998375b857e807b76144571b4290610ca3e99b96 Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread David Craven
> My point is that if we are giving a direct link to the non-patched > source in our UI, the FSDG issue is the same. Just because I can't understand the concern, doesn't necessarily mean that it isn't a valid one...

[PATCH] gnu: st: Mov to terminals.scm.

2016-08-07 Thread ng0
From 3864654025329367b6f87ccfb12a4ff789d5155a Mon Sep 17 00:00:00 2001 From: ng0 Date: Sun, 7 Aug 2016 12:30:06 + Subject: [PATCH] gnu: st: Move to terminals.scm. * gnu/packages/suckless.scm (st): Move from here ... * gnu/packages/terminals.scm (st): ... to here. --- gnu/packages/suckless.sc

Re: [PATCH] gnu: st: Mov to terminals.scm.

2016-08-07 Thread ng0
Hi, I think it's pointless to have a suckless.scm which bundles all software which just happens to follow suckless ideology/goals or be hosted there. That's why I did not commit ii to suckless but to irc.scm, no one will look into suckless.scm when what they really are looking for is not $suckless

Re: [PATCH v2] gnu: node: Update to 6.3.1.

2016-08-07 Thread Jelle Licht
Leo Famulari writes: > On Fri, Aug 05, 2016 at 01:02:45PM +0200, Jelle Licht wrote: >> ;; FIXME: These tests fail in the build container, but they >> don't >> ;; seem to be indicative of real problems in practice. >> (for-each delete-file >> -

[PATCH] (3 patches): move dwm+dmenu to wm, move slock to xdisorg.

2016-08-07 Thread ng0
With these 3 moved to more logical places, only surf remains. We keep web browsers in sepearate places so far, that's nothing I want to work on today. A grep for "gnu packages suckless" only showed the gnu/services/desktop.scm calling it, which seems to be slock, I changed that in patch which moves

Re: [PATCH] (3 patches): move dwm+dmenu to wm, move slock to xdisorg.

2016-08-07 Thread ng0
Iforgot to mention: A requirement is the patch which changes the dwm description, sent before this series. Logically moving and changing description are not one and the same thing, so I separated them. ng0 writes: > With these 3 moved to more logical places, only surf remains. We keep > web brow

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread Mathieu Lirzin
David Craven writes: > I meant the little -s flag... > > -s, --search=REGEXPsearch in synopsis and description using REGEXP > > and I meant download, build and run manually outside of guix. > > It still sounds like that's what you are saying... ;-) oops my bad. So your summary of what I was

Re: [PATCH] gnu: st: Mov to terminals.scm.

2016-08-07 Thread Mathieu Lirzin
Hi, I don't have an opinion on the move, however when moving packages across modules your have to keep track of the copyrights. In this case Amirouche Boubekki copyright should be moved from (gnu packages suckless) to (gnu packages terminals). ng0 writes: > From 3864654025329367b6f87ccfb12a4f

Re: [PATCH] Update girara to 0.2.6 (pre-requirement for zathura update)

2016-08-07 Thread Alex Kost
ng0 (2016-08-07 13:22 +0300) wrote: > From 91c1327d89c5d746c03ec8ce004317dfd0a63c88 Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Sun, 7 Aug 2016 10:05:56 + > Subject: [PATCH 2/6] gnu: girara: Update to 0.2.6. > > * gnu/packages/gtk.scm (girara): Update to 0.2.6. I pushed this and the other u

Re: [PATCH] gnu: Add ii.

2016-08-07 Thread Alex Kost
ng0 (2016-08-07 14:37 +0300) wrote: > From 76d9f586822ad0a33a12f372715de878d769268b Mon Sep 17 00:00:00 2001 > From: ng0 > Date: Sun, 7 Aug 2016 11:35:34 + > Subject: [PATCH] gnu: Add ii. > > * gnu/packages/irc.scm (ii): New variable. Applied as 99840fbbee260354d134031ae69e6e94b0f39d48, than

Re: [PATCH] gnu: dwm: Shorten the description.

2016-08-07 Thread Alex Kost
ng0 (2016-08-07 15:02 +0300) wrote: > This description was as long as the package itself. We should not do > this, this is enough. Whoever wants to read up on dwm and tiling wm in > general can read about it in dwm documentation and other twms. I agree, so if there are no objections, I'm going t

Re: [PATCH] gnu: dwm: Shorten the description.

2016-08-07 Thread ng0
Hi, Alex Kost writes: > ng0 (2016-08-07 15:02 +0300) wrote: > >> This description was as long as the package itself. We should not do >> this, this is enough. Whoever wants to read up on dwm and tiling wm in >> general can read about it in dwm documentation and other twms. > > I agree, so if th

Re: [PATCH] gnu: st: Mov to terminals.scm.

2016-08-07 Thread ng0
Mathieu Lirzin writes: > Hi, > > I don't have an opinion on the move, however when moving packages across > modules your have to keep track of the copyrights. Oh... Okay, I forgot about that, I never moved packages. Thanks for pointing it out to me. I know that my personal copyright was limited

Re: [PATCH] gnu: st: Mov to terminals.scm.

2016-08-07 Thread Mathieu Lirzin
ng0 writes: > Mathieu Lirzin writes: > >> I don't have an opinion on the move, however when moving packages across >> modules your have to keep track of the copyrights. > > Oh... Okay, I forgot about that, I never moved packages. Thanks for > pointing it out to me. > I know that my personal copy

[PATCH]

2016-08-07 Thread ng0
This fixes the request to honor previous copyright holders. I noticed for at least dwm.scm -> suckless.scm this was not considered, so I have to assume this has not been looked at more carefully. A single commit on top of all the suckless.scm moves will be okay I hope. Can someone review if I got

[PATCH] update notmuch and python bindings

2016-08-07 Thread Troy Sankey
This patchset updates notmuch from 0.21 to 0.22.1, adds python3 bindings, and updates the existing python2 bindings. The notmuch upgrade was not necessary for adding the python3 bindings (my primary goal) but I'm throwing it in because the NEWS file does not indicate any breaking change. Also my

Re: [PATCH] update notmuch and python bindings

2016-08-07 Thread ng0
Hi, Troy Sankey writes: > This patchset updates notmuch from 0.21 to 0.22.1, adds python3 bindings, and > updates the existing python2 bindings. The notmuch upgrade was not necessary > for adding the python3 bindings (my primary goal) but I'm throwing it in > because the NEWS file does not indi

Re: [PATCH] update notmuch and python bindings

2016-08-07 Thread Troy Sankey
Quoting ng0 (2016-08-07 15:29:34) > > (inputs > > - `(("emacs" ,emacs) > > - ("glib" ,glib) > > + `(("glib" ,glib) > > Why is emacs removed? Please see the commit message :) Of course I could be misunderstanding something, but it does build successfully. I do not normally use

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread David Craven
Quoting FSDG: A free system distribution must not steer users towards obtaining any nonfree information for practical use, or encourage them to do so. We are not steering or encouraging users to do any thing by displaying the source url of the tarball, we are merely providing factual non-opinionat

Re: [PATCH] update notmuch and python bindings

2016-08-07 Thread ng0
Troy Sankey writes: > Quoting ng0 (2016-08-07 15:29:34) >> > (inputs >> > - `(("emacs" ,emacs) >> > - ("glib" ,glib) >> > + `(("glib" ,glib) >> >> Why is emacs removed? > > Please see the commit message :) > > Of course I could be misunderstanding something, but it does build >

Re: [PATCH] python-kivy and Adobe source-code-pro font

2016-08-07 Thread Jelle Licht
Dylan Jeffers writes: > Hi all, > Hello Dylan, Thanks for this patch! I love the source-code-pro font :). > First patch to guix-devel; tried my best to conform to the guidelines. Guix (usually) has a one commit per change policy. Contributions in guix usually follow a specific format. You mig

Re: [PATCH] update notmuch and python bindings

2016-08-07 Thread ng0
Troy Sankey writes: > This patchset updates notmuch from 0.21 to 0.22.1, adds python3 bindings, and > updates the existing python2 bindings. The notmuch upgrade was not necessary > for adding the python3 bindings (my primary goal) but I'm throwing it in > because the NEWS file does not indicate

Re: [PATCH] gnu: perl-io-socket-ssl: Add IDN support.

2016-08-07 Thread ng0
John Darrington writes: > On Sun, Jul 24, 2016 at 12:05:35PM -0400, Leo Famulari wrote: > On Sun, Jul 24, 2016 at 04:08:20PM +0300, Efraim Flashner wrote: > > On Sun, Jul 24, 2016 at 09:08:05AM +, ng0 wrote: > > > Efraim Flashner writes: > > > > LGTM! > > > > >

Re: [PATCH] ui: 'package->recutils' serializes the source field.

2016-08-07 Thread Mathieu Lirzin
David Craven writes: > Quoting FSDG: > A free system distribution must not steer users towards obtaining any > nonfree information for practical use, or encourage them to do so. > > We are not steering or encouraging users to do any thing by displaying > the source url of the tarball, That's an

Re: [PATCH] update notmuch and python bindings

2016-08-07 Thread Troy Sankey
Updated patches address all comments so far except for the removal of the emacs input. Also updated the test numbers just for fun. Quoting ng0 (2016-08-07 18:24:57) > I'd also advise to upgrade git to something more recent if your distro > allows it :) Debian stable :) It's a long term project