Re: guix package modif testing

[Vincent Legoll]

> I think you are confusing the system you are running (Guix SD) and the
> system you are developing (the Guix git repo).

I'm just a few weeks of part time into guix, so yes, I clearly am.

I've got no problem using git though

I'll see what I can get out of all the comments I got

-- 
Vincent Legoll

libgd security update / i686 issues

[Leo Famulari]

libgd 2.2.3 has been released [0], which includes fixes for
CVE-2016-6207.

I built it on x86_64, and also cross-built to i686-linux. The 32-bit rounding
issue that Mark fixed with commit 27326064 was reported upstream [1],
and the suggested workaround is to add "-msse -mfpmath=sse" to CFLAGS
[2].

Having removed Mark's patch, I can cross-build to i686-linux using those
flags. The patch has gone stale with the 2.2.3 release:

---
gdimagerotate/bug00067.c: In function ‘main’:
gdimagerotate/bug00067.c:11:14: error: unused variable ‘filename’ 
[-Werror=unused-variable]
  char *path, filename[2048];
  ^
gdimagerotate/bug00067.c:11:8: error: unused variable ‘path’ 
[-Werror=unused-variable]
  char *path, filename[2048];
^
cc1: all warnings being treated as errors
Makefile:3120: recipe for target 'gdimagerotate/bug00067.o' failed
---

Should these CFLAGS values be applied unconditionally, as in the
attached patch, or should they be applied only while building on or for
specific architectures? Or something else?

[0]
https://github.com/libgd/libgd/releases/tag/gd-2.2.3

[1]
https://github.com/libgd/libgd/issues/242

[2]
https://github.com/libgd/libgd/commit/62ecc651e7780add5e4035bfc0e6cd060e90f6a9
>From d429ce44a39543b8f5e64f22bc722ee8bc22bd01 Mon Sep 17 00:00:00 2001
From: Leo Famulari 
Date: Thu, 28 Jul 2016 02:46:23 -0400
Subject: [PATCH] gnu: gd: Update to 2.2.3.

Fixes CVE-2016-6207.

* gnu/packages/gd.scm (gd): Update to 2.2.3.
[arguments]: Add "-msse -mfpmath=sse" to CFLAGS.
* gnu/packages/patches/gd-CVE-2016-5766.patch,
gnu/packages/patches/gd-CVE-2016-6128.patch,
gnu/packages/patches/gd-CVE-2016-6132.patch,
gnu/packages/patches/gd-CVE-2016-6214.patch,
gnu/packages/patches/gd-fix-test-on-i686.patch: Delete files.
* gnu/local.mk (dist_patch_DATA): Remove them.
---
 gnu/local.mk   |   5 -
 gnu/packages/gd.scm|  11 +-
 gnu/packages/patches/gd-CVE-2016-5766.patch|  81 
 gnu/packages/patches/gd-CVE-2016-6128.patch| 253 -
 gnu/packages/patches/gd-CVE-2016-6132.patch|  55 --
 gnu/packages/patches/gd-CVE-2016-6214.patch|  66 ---
 gnu/packages/patches/gd-fix-test-on-i686.patch |  34 
 7 files changed, 4 insertions(+), 501 deletions(-)
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-5766.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6128.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6132.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6214.patch
 delete mode 100644 gnu/packages/patches/gd-fix-test-on-i686.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index c143dd7..2f4dda1 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -512,11 +512,6 @@ dist_patch_DATA =		\
   %D%/packages/patches/gcc-cross-environment-variables.patch	\
   %D%/packages/patches/gcc-libvtv-runpath.patch			\
   %D%/packages/patches/gcc-5.0-libvtv-runpath.patch		\
-  %D%/packages/patches/gd-CVE-2016-5766.patch			\
-  %D%/packages/patches/gd-CVE-2016-6128.patch			\
-  %D%/packages/patches/gd-CVE-2016-6132.patch			\
-  %D%/packages/patches/gd-CVE-2016-6214.patch			\
-  %D%/packages/patches/gd-fix-test-on-i686.patch		\
   %D%/packages/patches/gegl-CVE-2012-4433.patch			\
   %D%/packages/patches/geoclue-config.patch			\
   %D%/packages/patches/ghostscript-CVE-2015-3228.patch		\
diff --git a/gnu/packages/gd.scm b/gnu/packages/gd.scm
index 3313ee6..46a2912 100644
--- a/gnu/packages/gd.scm
+++ b/gnu/packages/gd.scm
@@ -40,22 +40,19 @@
 ;; Note: With libgd.org now pointing to github.com, genuine old
 ;; tarballs are no longer available.  Notably, versions 2.0.x are
 ;; missing.
-(version "2.2.2")
+(version "2.2.3")
 
 (source (origin
  (method url-fetch)
  (uri (string-append
"https://github.com/libgd/libgd/releases/download/gd-";
version "/libgd-" version ".tar.xz"))
- (patches (search-patches "gd-fix-test-on-i686.patch"
-  "gd-CVE-2016-5766.patch"
-  "gd-CVE-2016-6128.patch"
-  "gd-CVE-2016-6132.patch"
-  "gd-CVE-2016-6214.patch"))
  (sha256
   (base32
-   "1311g5mva2xlzqv3rjqjc4jjkn5lzls4skvr395h633zw1n7b7s8"
+   "0g3xz8jpz1pl2zzmssglrpa9nxiaa7rmcmvgpbrjz8k9cyynqsvl"
 (build-system gnu-build-system)
+(arguments
+ '(#:configure-flags '("CFLAGS=-msse -mfpmath=sse")))
 (native-inputs
  `(("pkg-config" ,pkg-config)))
 (inputs
diff --git a/gnu/packages/patches/gd-CVE-2016-5766.patch b/gnu/packages/patches/gd-CVE-2016-5766.patch
deleted file mode 100644
index 400cb0a..000
--- a/gnu/packages/patches/gd-CVE-2016-5766.patch
+++ /dev/null
@@ -1,81 +0,0 @@
-Fix CVE-2016-5766 (Integer Overflow in _gd2GetHeader() resulting in heap
-overflow).
-
-https://cve.mitre

Re: [PATCH] Add Elixir (was: )

[Ricardo Wurmus]

Pjotr Prins  writes:

> On Mon, Jul 25, 2016 at 08:13:33AM +0200, Ricardo Wurmus wrote:
>> Thanks again for the patch.  I hope you are willing to provide some
>> guidance when I have some problems understanding certain bits of the
>> build.
>
> I am happy to help out if you take the lead.

Thank you.  I applied the patch to my copy of master, but Elxir fails to
compile reliably.  Most of the times it fails with this error:

   == Compilation error on file lib/system.ex ==
   ** (exit) :epipe
   Makefile:77: recipe for target 'lib/elixir/ebin/Elixir.Kernel.beam' failed
   make: *** [lib/elixir/ebin/Elixir.Kernel.beam] Error 1

This is without any changes to your patch.  I wonder if this indicates a
lack of resources on my laptop.

Occasionally it does work and gets past the compilation of
“Elixir.Kernel.beam”.  Did you encounter this error before?

Before I can make some changes on top of your patch I need Elixir to
build (as a baseline).

~~ Ricardo

Re: Odd behavior with --dry-run and --upgrade

[Alex Kost]

Ludovic Courtès (2016-07-28 01:19 +0300) wrote:

> Alex Kost  skribis:
>
>> I have zero knowledge in grafting, but if I checked it right, the
>> attached patch should disable grafting for the emacs interface (when
>> dry-run is on).  If you think it should be a part of a bigger
>> dry-run+no-grafts patch, please use it.
>
> Two separate patches is fine, IMO.

OK, then I'll commit this patch when it will be ready.

>> From d7747453bf31a616d414dce293fc0556d601abcb Mon Sep 17 00:00:00 2001
>> From: Alex Kost 
>> Date: Wed, 27 Jul 2016 14:55:50 +0300
>> Subject: [PATCH] emacs: Disable grafts when dry-run is enabled.
>>
>> * emacs/guix-main.scm (process-package-actions): Set grafting according
>> to 'dry-run?'.
>> * guix/scripts.scm (build-package): Likewise.
>
> [...]
>
>>  (define-module (guix scripts)
>> +  #:use-module (guix grafts)
>>#:use-module (guix utils)
>>#:use-module (guix ui)
>>#:use-module (guix store)
>> @@ -106,6 +107,7 @@ true."
>>"Build PACKAGE using BUILD-OPTIONS acceptable by 'set-build-options'.
>>  Show what and how will/would be built."
>>(mbegin %store-monad
>> +(set-grafting (not dry-run?))
>>  (apply set-build-options*
>> #:use-substitutes? use-substitutes?
>> (strip-keyword-arguments '(#:dry-run?) build-options))
>
> Here it might be best to do something like this:
>
>   (mlet %store-monad ((grafting? ((lift0 %graft? %store-monad
> (set-grafting (and (not dry-run?) grafting?))
> …)
>
> This would make sure we don’t enable grafting if it turned out to be
> disabled.
>
> WDYT?

OK, you know better :-)

However, I tried it and it doesn't work for me (note: I know nothing
about monads, gexps, etc.).  When I try this:

   (mbegin %store-monad
 (set-grafting #f)
 …)

the grafting doesn't happen, but when I try this:

   (mlet %store-monad ((grafting? ((lift0 %graft? %store-monad
 (set-grafting #f)
 …)

grafting happens anyway.  I have no idea what the problem is.

-- 
Alex

Re: core-updates, next release, and all that

[Andreas Enge]

On Thu, Jul 28, 2016 at 01:10:27AM +0200, Ludovic Courtès wrote:
> I pushed something along these lines as
> 8b732bf6d93ad2cb529c3c5f886efe2625c5fb72, and also canceled the previous
> evaluation builds and started a new evaluation.

Great, thanks!

Andreas

Re: core-updates, next release, and all that

[Andreas Enge]

On Thu, Jul 28, 2016 at 01:10:27AM +0200, Ludovic Courtès wrote:
> I pushed something along these lines as
> 8b732bf6d93ad2cb529c3c5f886efe2625c5fb72, and also canceled the previous
> evaluation builds and started a new evaluation.

In fact, this seems to require the same number of rebuilds. Does changing the
arguments field not imply that the derivation will be considered different
also for other architectures?

Andreas

Re: [PATCH] Add Elixir (was: )

[Vincent Legoll]

On Thu, Jul 28, 2016 at 9:27 AM, Ricardo Wurmus  wrote:
> I wonder if this indicates a lack of resources on my laptop.
>
> Occasionally it does work and gets past the compilation of
> “Elixir.Kernel.beam”.  Did you encounter this error before?

Maybe it got OOM-killed

-- 
Vincent Legoll

Re: core-updates, next release, and all that

[Andreas Enge]

On Thu, Jul 28, 2016 at 10:24:25AM +0200, Andreas Enge wrote:
> In fact, this seems to require the same number of rebuilds. Does changing the
> arguments field not imply that the derivation will be considered different
> also for other architectures?

Now, I am wrong - a problem with the reloading of my webpage! I turned the
computer off yesterday, turned it on again today, and then icecat showed the
same pages as yesterday without reloading them!

Andreas

Re: rust work in progress conflicts

[ng0]

Hi,

ng0  writes:

> Jelle Licht  writes:
>
>> I have taken the liberty to try my hand at finishing this, as I figured
>> it would be a good way for me to get more familiar with 'the Guix way'
>> of packaging things.
>
> Thanks!
> Also, could you please use this email address for further CC
> things (I tend to avoid them completely if possible), as the
> @grrlz.net is no longer subscribed to the mailinglist afaik.
>
>> Wow, did I misjudge this rabbit hole though. It seems to be the case that
>> rust needs the (most recent) snapshotted binary stage-0 compiler as part
>> of the build process. This was not the case some years ago[1], but since
>> then, some 319 snapshots have been released.
>>
>> Now there are two approaches which might make sense to me:
>>
>> 1) We package a recent stage-0 binary (thus adding yet another random
>> binary to the mix)
>>
>> 2) We bootstrap all the way from the original rust compiler, written in
>> ocaml. This would then presumably need to be repeated for each snapshot,
>> leading to about 319 iterative compiler build. On my kind-of-okay i7,
>> compiling a single rust iteration takes about 25 to 40 minutes.
>
> This sounds expensive. Isn't there a chance to speed this up,
> some other developer with a small Cluster of CPUs for computing
> at hand?
>
>> I tentatively went with option 1, if only because I would like to see
>> results this decade still, and ran into several hurdles that became
>> quite manageable with help from the good people of #guix and
>> #rust-internals. One more issue yet remains: part of the rust
>> compilation process actually calls the 'cc linker'. This part does not
>> respect make flags, setenv calls or even rust's special configure flag
>> for setting cc.
>>
>> Option 1 does not seem feasible at this point of time, but there is some
>> light at the end of the tunnel: rust is at some point going to follow a
>> convention that will allow bootstrapping compilers via 'master from
>> beta, beta from stable and stable from previous stable'[2].
>>
>> I am currently thinking of a compromise; basically, at this moment go
>> for option 1, and once the policy previously described is properly
>> implemented by the rust team, start iteratively bootstrapping rust from
>> that point in time.
>>
>>
>> tldr: If we can get 'cc' in the build environment, we can have a 'dirty'
>> bootstrapped rust very soon. If we want to do it properly, it might take
>> a lot longer.  
>>
>> WDYT?
>>
>> [1]: https://news.ycombinator.com/item?id=8732669
>> [2]: https://botbot.me/mozilla/rust-internals/2016-04-29/?page=3, look
>> for eddyb
>
> It's good that there seems to be light at the end, though I did
> not expect rust to be that challenging when I started with it to
> just package panopticon[0] through cargo import which needs to be
> written once rust is done.
>
>
> [0]: https://panopticon.re
>
>>>---snip-snap
>
> -- 
> ♥Ⓐ ng0
>

So I picked up rust.scm again and forgot about this thread, only a
search for rust brought it up again.
As this will be a long task obviously, however we finish it, can we file
a bug on it so it is obvious that work is being done on it and there'll
be no dual work on this?

Recently released version 1.10.0 of rust merged the
"--disable-codegen-tests" I needed back when i worked on it.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: libgd security update / i686 issues

[Andreas Enge]

On Thu, Jul 28, 2016 at 03:23:37AM -0400, Leo Famulari wrote:
> Should these CFLAGS values be applied unconditionally, as in the
> attached patch, or should they be applied only while building on or for
> specific architectures? Or something else?

They only work on x86 processors, almost by definition: SSE stands for
a certain instruction set. So one would need to check whether the problem
occurs for other architectures. I would assume that it happens on all 32
bit architectures, in particular armhf. Their code is too fragile: One
should not rely on fine details of the processor architecture or instruction
set to hope for an expected rounding behaviour.

Andreas

Re: translation intermediate files

[ng0]

Tobias Geerinckx-Rice  writes:

> Vincent,
>
> On 27/07/2016 19:42, Vincent Legoll wrote:
>> My bad, I was asking because they appeared after a build, so I thought
>> they were generated...
>
> No, you're (also) right. They're a bit of both.
>
> The English source lines can be auto-updated when building, but the
> translated lines come from hard-working humans at the Translation
> Project, so we can't just .gitignore the whole file.
>
> I'm sure someone will correct me if I'm wrong.
>
> If modified .po files are stopping you from performing certain git
> operations, running ‘git reset --hard’ should get rid of them and touch
> the files so they don't get updated again. Only do this after committing
> any changes you may have made by hand, if any.
>
> Kind regards,
>
> T G-R
>

Yesterday I thought if we chould make it more clear in the manual
developer part what these files are, how to deal with them, where they
originate and that they should not be included in the patches.

There might already be a "translating Guix" section, so we could just
extend that and add a few lines wrapped around the link. I can imagine
it to be in the 8.3 (?) section where you are advised to check things
before sending in the patch.
This applies to any model we might or might not adapt in the future,
wether email or not, the translation files will be there and some people
might not know what to do with them.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: libgd security update / i686 issues

[Andreas Enge]

On Thu, Jul 28, 2016 at 03:23:37AM -0400, Leo Famulari wrote:
> I built it on x86_64, and also cross-built to i686-linux. The 32-bit rounding
> issue that Mark fixed with commit 27326064 was reported upstream [1],
> and the suggested workaround is to add "-msse -mfpmath=sse" to CFLAGS
> [2].

Well, the bug report states that the result is correct on armv7. Apparently
i686 is not IEEE compliant by default, while armv7 is. So it should be okay
to apply the flags only on i686. We assume that SSE, but not SSE2 or later
are supported, see our Qt package.

Andreas

Re: translation intermediate files

[Vincent Legoll]

> Yesterday I thought if we chould make it more clear in the manual
> developer part what these files are, how to deal with them, where they
> originate and that they should not be included in the patches.

Another option would be to build outside of the source tree, and only
put the generated/modified files in the build tree.

Is this already doable ?

-- 
Vincent Legoll

Re: [PATCH] gnu: Add perl-net-dns.

[ng0]

Hi,

I wanted to apply and test this on latest master but it fails on line
29. of networking.scm. Can you rebase this?

Danny Milosavljevic  writes:

> From: Danny Milosavljevic 
>
> * gnu/packages/networking.scm (perl-net-dns): New variable.

Could you fix this commit message to:

gnu: Add perl-net-dns.

* gnu/packages/networking.scm (perl-net-dns): New variable.
> ---
>  gnu/packages/networking.scm | 24 
>  1 file changed, 24 insertions(+)
>
> diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
> index 9e4f7bb..a271543 100644
> --- a/gnu/packages/networking.scm
> +++ b/gnu/packages/networking.scm
> @@ -29,6 +29,7 @@
>#:use-module (guix download)
>#:use-module (guix build-system glib-or-gtk)
>#:use-module (guix build-system gnu)
> +  #:use-module (guix build-system perl)
>#:use-module (gnu packages admin)
>#:use-module (gnu packages adns)
>#:use-module (gnu packages audio)
> @@ -375,3 +376,26 @@ sniffer}, that lets you capture and interactively browse 
> the contents of
>  network frames.")
>  (license license:gpl2+)
>  (home-page "https://www.wireshark.org/";)))
> +
> +(define-public perl-net-dns
> + (package
> +  (name "perl-net-dns")
> +  (version "1.06")
> +  (source
> +(origin
> +  (method url-fetch)
> +  (uri (string-append
> + "mirror://cpan/authors/id/N/NL/NLNETLABS/Net-DNS-"
> + version
> + ".tar.gz"))
> +  (sha256
> +(base32
> +  "07m5331132h9xkh1i6jv9d80f571yva27iqa31aq4sm31iw7nn53"
> +  (build-system perl-build-system)
> +  (inputs
> +`(("perl-digest-hmac" ,perl-digest-hmac)))
> +  (home-page "http://search.cpan.org/dist/Net-DNS";)
> +  (synopsis
> +"Perl Interface to the Domain Name System")
> +  (description "Net::DNS is the Perl Interface to the Domain Name System.")
> +  (license license:x11)))
>

-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: guix package modif testing

[Vincent Legoll]

I did, as a user of my guixsd system:

git clone [...] guix
cd guix
./bootstrap
[ERROR PKG_CONFIG / m4 / etc...]

guix environment guix
[env]> ./bootstrap
[env]> ./configure
[env]> make -j3

I get the BT @ http://paste.lisp.org/+6W42

What's wrong ?

-- 
Vincent Legoll

Re: [PATCH] gnu: Add perl-socket6.

[ng0]

Hi,

could you fix the commit message the same way pointed out in the
previous (perl-net-dns) package and rebase it on master?

This one fails at line 399 of networking.scm, I can not apply it.

Danny Milosavljevic  writes:

> From: Danny Milosavljevic 
>
> * gnu/packages/networking.scm (perl-socket6): New variable.
> ---
>  gnu/packages/networking.scm | 33 +
>  1 file changed, 33 insertions(+)
>
> diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
> index a271543..c56ec36 100644
> --- a/gnu/packages/networking.scm
> +++ b/gnu/packages/networking.scm
> @@ -399,3 +399,36 @@ network frames.")
>  "Perl Interface to the Domain Name System")
>(description "Net::DNS is the Perl Interface to the Domain Name System.")
>(license license:x11)))
> +
> +(define-public perl-socket6
> + (package
> +  (name "perl-socket6")
> +  (version "0.28")
> +  (source
> +(origin
> +  (method url-fetch)
> +  (uri (string-append
> + "mirror://cpan/authors/id/U/UM/UMEMOTO/Socket6-"
> + version
> + ".tar.gz"))
> +  (sha256
> +(base32
> +  "11j5jzqbzmwlws9zals43ry2f1nw9qy6im7yhn9ck5rikywrmm5z"
> +  (build-system perl-build-system)
> +  (arguments
> +   `(#:phases
> + (modify-phases %standard-phases
> + (replace 'configure
> +   (lambda* (#:key outputs #:allow-other-keys)
> + (let* ((out (assoc-ref outputs "out"))
> +(args `("Makefile.PL"
> +,(string-append "PREFIX=" out)
> +"INSTALLDIRS=site")))
> +   (setenv "CONFIG_SHELL" (which "sh"))
> +   (zero? (apply system* "perl" args
> +  (home-page "http://search.cpan.org/dist/Socket6";)
> +  (synopsis
> +"IPv6 related part of the C socket.h defines and structure manipulators 
> for Perl")
> +  (description "Socket6 binds the IPv6 related part of the C socket header
> +definitions and structure manipulators for Perl.")
> +  (license license:bsd-3)))
>

-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: core-updates, next release, and all that

[Andreas Enge]

Hello,

the main failures that remain on x86 are related to the gmime problem fixed
in master. So I merged master once again and will start another evaluation
of core-updates. Probably we are done then.

There is a little problem with some source that cannot be downloaded:
   http://hydra.gnu.org:3000/build/1327453
If this has not been fixed in master yet, maybe the bioinformatics people
would like to have a look.

Thanks!

Andreas

Re: [PATCH] Add python-pythondialog

[ng0]

Hi,

thanks for working on this.

Vincent Legoll  writes:

>>> If we inherit, we should let python2- inherit from python3.
>>>
>>> I was not sure about the right way to process with this, so it
>>> ended up in 2 packages.
>>
>> There has been no update on this for 3 weeks.
>> Could someone look into this and reply?
>
> Is that what you want ?
> This is still untested though, but I'll try the 2 packages

I don't understand. Is this completely untested or have you tested this
before sending the patch?

> -- 
> Vincent Legoll
> From 8b6becfd95235e1559cdb4a3760c5928d707e6cf Mon Sep 17 00:00:00 2001
> From: Vincent Legoll 
> Date: Wed, 27 Jul 2016 14:28:06 +0200
> Subject: [PATCH] Add python-pythondialog
>
> Signed-off-by: Vincent Legoll 
> ---
>  gnu/packages/python.scm | 21 -
>  1 file changed, 16 insertions(+), 5 deletions(-)
>
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index 80ff81f..533d731 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -6654,17 +6654,17 @@ facilities for defining, registering and looking up 
> components.")
>  (define-public python2-zope-component
>(package-with-python2 python-zope-component))
>  
> -(define-public python2-pythondialog
> +(define-public python-pythondialog
>(package
> -(name "python2-pythondialog")
> +(name "python-pythondialog")
>  (version "3.4.0")
>  (source
>   (origin
> (method url-fetch)
> -   (uri (pypi-uri "python2-pythondialog" version))
> +   (uri (pypi-uri "pythondialog" version))
> (sha256
>  (base32
> - "0d8k7lxk50imdyx85lv8j98i4c93a71iwpapnl1506rpkbm9qvd9"
> + "1728ghsran47jczn9bhlnkvk5bvqmmbihabgif5h705b84r1272c"
>  (build-system python-build-system)
>  (arguments
>   `(#:phases
> @@ -6678,7 +6678,6 @@ facilities for defining, registering and looking up 
> components.")
>   (("os.getenv\\(\"PATH\", \":/bin:/usr/bin\"\\)")
>(string-append "os.getenv(\"PATH\")  + \":" dialog 
> "/bin\"")))
> #t
> -   #:python ,python-2
> #:tests? #f)) ; no test suite
>  (propagated-inputs
>   `(("dialog" ,dialog)))
> @@ -6689,6 +6688,18 @@ provide an easy to use, pythonic and comprehensive 
> Python interface to dialog.
>  This allows one to make simple text-mode user interfaces on Unix-like 
> systems")
>  (license lgpl2.1)))
>  
> +(define-public python2-pythondialog
> +  (package-with-python2
> +(package (inherit python-pythondialog)
> +  (name "python2-pythondialog")
> +  (source
> +   (origin
> + (method url-fetch)
> + (uri (pypi-uri "python2-pythondialog" version))
> + (sha256
> +  (base32
> +   "0d8k7lxk50imdyx85lv8j98i4c93a71iwpapnl1506rpkbm9qvd9"))
> +
>  (define-public python-pyrfc3339
>(package
>  (name "python-pyrfc3339")
> -- 
> 1.9.1
>

-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: (pre-release) [PATCH] torsocks update to 2.2.0-rc1

[ng0]

Hi,

I have this in use on Gentoo and GuixSD systems now for some time.
I know that we do not (?) package release candidate versions, but this
would fix our much older current torsocks. 2.1.x fails for some reasons,
this fixes it. There should be a release hopefully soon, otherwise I'll
ask tor-dev for it.

Could we make an exception for torsocks here, or should we stick with
2.0.0 until 2.2.0 has a version release?

> From fe94e760b8705f2972f04b565f02528a953277ae Mon Sep 17 00:00:00 2001
> From: ng0 
> Date: Tue, 28 Jun 2016 13:08:43 +
> Subject: [PATCH] gnu: torsocks: Update to 2.2.0-rc1.
>
> * gnu/packages/tor.scm (torsocks): Update to 2.2.0-rc1.
> ---
>  gnu/packages/tor.scm | 14 +++---
>  1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/gnu/packages/tor.scm b/gnu/packages/tor.scm
> index 5155fc0..006334a 100644
> --- a/gnu/packages/tor.scm
> +++ b/gnu/packages/tor.scm
> @@ -2,6 +2,7 @@
>  ;;; Copyright © 2013, 2014, 2015 Ludovic Courtès 
>  ;;; Copyright © 2014, 2015 Mark H Weaver 
>  ;;; Copyright © 2016 Efraim Flashner 
> +;;; Copyright © 2016 ng0 
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -70,16 +71,15 @@ applications based on the TCP protocol.")
>  (define-public torsocks
>(package
>  (name "torsocks")
> -(version "2.0.0")
> +(version "2.2.0-rc1")
>  (source (origin
> -  (method git-fetch)
> -  (uri (git-reference
> -(url "https://git.torproject.org/torsocks.git";)
> -(commit (string-append "v" version
> +  (method url-fetch)
> +  (uri (string-append "https://people.torproject.org/";
> +  "~dgoulet/" name "/" name "-"
> +  version ".tar.bz2"))
>(sha256
> (base32
> -"0an2q5ail9z414riyjbkjkm29504hy778j914baz2gn5hlv2cfak"))
> -  (file-name (string-append name "-" version "-checkout"))
> +"184khqjzpg79vyc50q1k5i7rh8m48pjxa495clxbyyq8ihgqk4l6"))
>(patches (search-patches "torsocks-dns-test.patch"
>  (build-system gnu-build-system)
>  (arguments
> --
> 2.9.0
>

-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: core-updates, next release, and all that

[Ben Woodcroft]

On 28/07/16 19:31, Andreas Enge wrote:

Hello,

the main failures that remain on x86 are related to the gmime problem fixed
in master. So I merged master once again and will start another evaluation
of core-updates. Probably we are done then.

There is a little problem with some source that cannot be downloaded:
http://hydra.gnu.org:3000/build/1327453
If this has not been fixed in master yet, maybe the bioinformatics people
would like to have a look.


Fixed in 'f2e791750'. This was another case of BioConductor deleting out 
of date sources, and more encouragement to move to SVN as discussed 
previously.

https://lists.gnu.org/archive/html/guix-devel/2016-06/msg00683.html

Thanks for the alert and other efforts.
ben

Re: core-updates, next release, and all that

[Andreas Enge]

On Thu, Jul 28, 2016 at 08:21:03PM +1000, Ben Woodcroft wrote:
> Fixed in 'f2e791750'. This was another case of BioConductor deleting out of
> date sources, and more encouragement to move to SVN as discussed previously.
> https://lists.gnu.org/archive/html/guix-devel/2016-06/msg00683.html

Great, thanks! I cherry-picked into core-updates, hoping that git will be
intelligent enough to not complain when we merge back core-updates into
master.

Andreas

Re: [PATCH] Add Elixir (was: )

[Pjotr Prins]

On Thu, Jul 28, 2016 at 09:27:22AM +0200, Ricardo Wurmus wrote:
> 
> Pjotr Prins  writes:
> 
> > On Mon, Jul 25, 2016 at 08:13:33AM +0200, Ricardo Wurmus wrote:
> >> Thanks again for the patch.  I hope you are willing to provide some
> >> guidance when I have some problems understanding certain bits of the
> >> build.
> >
> > I am happy to help out if you take the lead.
> 
> Thank you.  I applied the patch to my copy of master, but Elxir fails to
> compile reliably.  Most of the times it fails with this error:
> 
>== Compilation error on file lib/system.ex ==
>** (exit) :epipe
>Makefile:77: recipe for target 'lib/elixir/ebin/Elixir.Kernel.beam' failed
>make: *** [lib/elixir/ebin/Elixir.Kernel.beam] Error 1
> 
> This is without any changes to your patch.  I wonder if this indicates a
> lack of resources on my laptop.
> 
> Occasionally it does work and gets past the compilation of
> “Elixir.Kernel.beam”.  Did you encounter this error before?

I may have seen it.

> Before I can make some changes on top of your patch I need Elixir to
> build (as a baseline).

I'll take a look over the weekend. Flying home right now.

Pj.

[WIP PATCH] linux-initrd: Allow own set of kernel modules.

[Tomáš Čech]

* gnu/system/linux-initrd.scm(base-initrd): Add `linux-modules'
  parameter. Rename former `linux-modules' to
  `default-linux-modules'. Introduce used-linux-modules to make the code
  more readable.
---
 gnu/system/linux-initrd.scm | 12 +---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/system/linux-initrd.scm b/gnu/system/linux-initrd.scm
index bbaa5c0..4934c92 100644
--- a/gnu/system/linux-initrd.scm
+++ b/gnu/system/linux-initrd.scm
@@ -138,6 +138,7 @@ MODULES and taken from LINUX."
   qemu-networking?
   (virtio? #t)
   volatile-root?
+  (linux-modules #f)
   (extra-modules '()))
   "Return a monadic derivation that builds a generic initrd, with kernel
 modules taken from LINUX.  FILE-SYSTEMS is a list of file-systems to be
@@ -174,7 +175,7 @@ loaded at boot time in the order in which they appear."
 (lambda (fs)
   (string=? (file-system-type fs) type)))
 
-  (define linux-modules
+  (define default-linux-modules
 ;; Modules added to the initrd and loaded from the initrd.
 `("ahci"  ;for SATA controllers
   "usb-storage" "uas" ;for the installation image etc.
@@ -220,8 +221,13 @@ loaded at boot time in the order in which they appear."
  (open source target)))
  mapped-devices))
 
+  (define used-linux-modules
+(if (equal? linux-modules #f)
+default-linux-modules
+linux-modules))
+
   (mlet %store-monad ((kodir (flat-linux-module-directory linux
-  linux-modules)))
+  used-linux-modules)))
 (expression->initrd
  (with-imported-modules '((guix build bournish)
   (guix build utils)
@@ -251,7 +257,7 @@ loaded at boot time in the order in which they appear."
(boot-system #:mounts '#$(map file-system->spec file-systems)
 #:pre-mount (lambda ()
   (and #$@device-mapping-commands))
-#:linux-modules '#$linux-modules
+#:linux-modules '#$used-linux-modules
 #:linux-module-directory '#$kodir
 #:qemu-guest-networking? #$qemu-networking?
 #:volatile-root? '#$volatile-root?)))
-- 
2.9.2

Re: [PATCH] Add python-pythondialog

[Vincent Legoll]

>> Is that what you want ?
>> This is still untested though, but I'll try the 2 packages
>
> I don't understand. Is this completely untested or have you tested this
> before sending the patch?

This patch is untested. I didn't manage to test it (see the other thread:
"guix package modif testing").

-- 
Vincent Legoll

Re: [PATCH] Add python-pythondialog

[Vincent Legoll]

And actually the patch is wrong, I think it's missing a closing paren
for python2-pythondialog...

-- 
Vincent Legoll

[PATCH 0/3] Perl log patches fixed

[Alex Sassmannshausen]

Hello,

In series, the three revised patches for perl-log.  As seems all to often the
case, I got pulled to other things.  But I fixed the inputs and wrapped
descriptions in @code where appropriate.

I'll push these shortly unless you have any further comments.

Ben: apologies for the delay in getting log4perl in in the end!

Cheers,

Alex

Alex Sassmannshausen (3):
  gnu: Add perl-log-any.
  gnu: Add perl-log-log4perl.
  gnu: Add perl-log-any-adapter-log4perl.

 gnu/packages/perl.scm | 79 +++
 1 file changed, 79 insertions(+)

-- 
2.8.4

[PATCH 1/3] gnu: Add perl-log-any.

[Alex Sassmannshausen]

* gnu/packages/perl.scm (perl-log-any): New variable.
---
 gnu/packages/perl.scm | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index f42abee..7fb1101 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -2939,6 +2939,37 @@ either uses the first module it finds or throws an 
error.")
 versa.")
 (license (package-license perl
 
+(define-public perl-log-any
+  (package
+(name "perl-log-any")
+(version "1.040")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append "mirror://cpan/authors/id/D/DA/DAGOLDEN/Log-Any-"
+   version ".tar.gz"))
+   (sha256
+(base32
+ "0r1q7cclgwl24gzdnjzvd8y0r7j17dngjk492x35w198zhdj2ncp"
+(build-system perl-build-system)
+(home-page "http://search.cpan.org/dist/Log-Any";)
+(synopsis "Bringing loggers and listeners together")
+(description "@code{Log::Any} provides a standard log production API for
+modules.  @code{Log::Any::Adapter} allows applications to choose the mechanism
+for log consumption, whether screen, file or another logging mechanism like
+@code{Log::Dispatch} or @code{Log::Log4perl}.
+
+A CPAN module uses @code{Log::Any} to get a log producer object.  An
+application, in turn, may choose one or more logging mechanisms via
+@code{Log::Any::Adapter}, or none at all.
+
+@code{Log::Any} has a very tiny footprint and no dependencies beyond Perl
+itself, which makes it appropriate for even small CPAN modules to use.  It
+defaults to 'null' logging activity, so a module can safely log without
+worrying about whether the application has chosen (or will ever choose) a
+logging mechanism.")
+(license (package-license perl
+
 (define-public perl-log-report-optional
   (package
 (name "perl-log-report-optional")
-- 
2.8.4

[PATCH 3/3] gnu: Add perl-log-any-adapter-log4perl.

[Alex Sassmannshausen]

* gnu/packages/perl.scm (perl-log-any-adapter-log4perl): New variable.
---
 gnu/packages/perl.scm | 25 +
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index abdc819..6513892 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -2970,6 +2970,31 @@ worrying about whether the application has chosen (or 
will ever choose) a
 logging mechanism.")
 (license (package-license perl
 
+(define-public perl-log-any-adapter-log4perl
+  (package
+(name "perl-log-any-adapter-log4perl")
+(version "0.08")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append
+ "mirror://cpan/authors/id/D/DA/DAGOLDEN/Log-Any-Adapter-Log4perl-"
+ version
+ ".tar.gz"))
+   (sha256
+(base32
+ "0xf4fnrsznvg0hf36q481124ja1hp3lybki1xjgnk82z9990jmxn"
+(build-system perl-build-system)
+(propagated-inputs
+ `(("perl-log-any" ,perl-log-any)
+   ("perl-log-log4perl" ,perl-log-log4perl)))
+(home-page
+ "http://search.cpan.org/dist/Log-Any-Adapter-Log4perl";)
+(synopsis "Log::Any adapter for Log::Log4perl")
+(description "@code{Log::Any::Adapter::Log4perl} provides a
+@code{Log::Any} adapter using @code{Log::Log4perl} for logging.")
+(license (package-license perl
+
 (define-public perl-log-log4perl
   (package
 (name "perl-log-log4perl")
-- 
2.8.4

[PATCH 2/3] gnu: Add perl-log-log4perl.

[Alex Sassmannshausen]

* gnu/packages/perl.scm (perl-log-log4perl): New variable.
---
 gnu/packages/perl.scm | 23 +++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 7fb1101..abdc819 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -2970,6 +2970,29 @@ worrying about whether the application has chosen (or 
will ever choose) a
 logging mechanism.")
 (license (package-license perl
 
+(define-public perl-log-log4perl
+  (package
+(name "perl-log-log4perl")
+(version "1.47")
+(source
+ (origin
+   (method url-fetch)
+   (uri (string-append
+ "mirror://cpan/authors/id/M/MS/MSCHILLI/Log-Log4perl-"
+ version
+ ".tar.gz"))
+   (sha256
+(base32
+ "0vxraq9navx5mgf8y8g6l5rbl3dv2ml8bishka5m69hj07nxs0ch"
+(build-system perl-build-system)
+(home-page
+ "http://search.cpan.org/dist/Log-Log4perl";)
+(synopsis "Log4j implementation for Perl")
+(description "@code{Log::Log4perl} lets you remote-control and fine-tune
+the logging behaviour of your system from the outside.  It implements the
+widely popular (Java-based) Log4j logging package in pure Perl.")
+(license (package-license perl
+
 (define-public perl-log-report-optional
   (package
 (name "perl-log-report-optional")
-- 
2.8.4

Re: [PATCH] gnu: Add perl-socket6.

[Danny Milosavljevic]

> could you fix the commit message the same way pointed out in the
> previous (perl-net-dns) package and 

>rebase it on master?

Sorry, I cannot do that manually for like 20 packages. 

This is Scheme we are talking about - one of the most regular languages there 
is. There should be a patching and diffing program which works on the tree 
level and we should be using that.

*shakes head* This is not C or anything where we would have to live with that.

Anyway, for the time being I wrote a small script to automate the process of 
reapplying these from the mailing list and then resubmit. It's running right 
now.

Even after I resubmitted all the patches, they will still only apply when 
applied in chronological order. Therefore, I will submit a patch series.

Script is attached for reference.


reapply-guix-patch-to-master
Description: Binary data


reapply-guix-patches-to-master
Description: Binary data

[PATCH v3 00/16] Add missing dependencies of Spamassassin

[Danny Milosavljevic]

Add missing dependencies of Spamassassin.

This patch adds almost all of the dependencies of Spamassassin.

Danny Milosavljevic (16):
  gnu: Add perl-db-file.
  gnu: Add perl-net-dns.
  gnu: Add perl-socket6.
  gnu: Add perl-net-dns-resolver-programmable.
  gnu: Add perl-netaddr-ip.
  gnu: Add perl-net-patricia.
  gnu: Add perl-file-find-object.
  gnu: Add perl-file-find-object-rule.
  gnu: Add perl-test-trailingspace.
  gnu: Add perl-encode-detect.
  gnu: Add perl-net-cidr-lite.
  gnu: Add perl-geo-ip.
  gnu: Add perl-crypt-openssl-rsa.
  gnu: Add perl-crypt-openssl-bignum.
  gnu: Add perl-mail-spf.
  gnu: Add perl-io-socket-inet6.

 gnu/packages/databases.scm  |  32 ++
 gnu/packages/mail.scm   |  41 
 gnu/packages/networking.scm | 231 
 gnu/packages/perl.scm   | 119 +++
 gnu/packages/tls.scm|  65 +
 5 files changed, 488 insertions(+)

[PATCH v3 01/16] gnu: Add perl-db-file.

[Danny Milosavljevic]

gnu: Add perl-db-file.

* gnu/packages/databases.scm (perl-db-file): New variable.
---
 gnu/packages/databases.scm | 32 
 1 file changed, 32 insertions(+)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index f088062..3bdb37b 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1046,3 +1046,35 @@ trees (LSM), for sustained throughput under random 
insert workloads.")
   (description
 "The DB::File module provides Perl bindings to the Berkeley DB version 
1.x.")
   (license (package-license perl
+
+(define-public perl-db-file
+ (package
+  (name "perl-db-file")
+  (version "1.838")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/P/PM/PMQS/DB_File-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0yp5d5zr8dk9g6xdh7ygi5bq63q7nxvhd58dk2i3ki4nb7yv2yh9"
+  (build-system perl-build-system)
+  (inputs `(("bdb" ,bdb)))
+  (native-inputs `(("perl-test-pod" ,perl-test-pod)))
+  (arguments
+ `(#:phases (modify-phases %standard-phases
+  (add-before
+   'configure 'modify-config.in
+   (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "config.in"
+   (("/usr/local/BerkeleyDB") (assoc-ref inputs "bdb")))
+ #t)
+  (home-page "http://search.cpan.org/dist/DB_File";)
+  (synopsis
+"Perl5 access to Berkeley DB version 1.x")
+  (description
+"DB::File provides Perl bindings to Berkeley DB version 1.x.")
+  (license (package-license perl

[PATCH v3 05/16] gnu: Add perl-netaddr-ip.

[Danny Milosavljevic]

gnu: Add perl-netaddr-ip.

* gnu/packages/networking.scm (perl-netaddr-ip): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a69ef0e..b575fce 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -502,3 +502,36 @@ emulation of DNS")
   (description "Net::DNS::Resolver::Programmable is a programmable DNS 
resolver for
 offline emulation of DNS.")
   (license (package-license perl
+
+(define-public perl-netaddr-ip
+ (package
+  (name "perl-netaddr-ip")
+  (version "4.079")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/M/MI/MIKER/NetAddr-IP-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1rx0dinrz9fk9qcg4rwqq5n1dm3xv2arymixpclcv2q2nzgq4npc"
+  (build-system perl-build-system)
+  (arguments
+`(#:phases
+   (modify-phases %standard-phases
+ (replace 'configure
+   (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+(args `("Makefile.PL"
+,(string-append "PREFIX=" out)
+"INSTALLDIRS=site")))
+   (setenv "CONFIG_SHELL" (which "sh"))
+   (zero? (apply system* "perl" args
+  (home-page
+"http://search.cpan.org/dist/NetAddr-IP";)
+  (synopsis
+"Manages IPv4 and IPv6 addresses and subnets")
+  (description "NetAddr::IP manages IPv4 and IPv6 addresses and subsets.")
+  (license (package-license perl

[PATCH v3 04/16] gnu: Add perl-net-dns-resolver-programmable.

[Danny Milosavljevic]

gnu: Add perl-net-dns-resolver-programmable.

* gnu/packages/networking.scm (perl-net-dns-resolver-programmable): New 
variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a6318b2..a69ef0e 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -469,3 +469,36 @@ application stack itself.")
   (description "Socket6 binds the IPv6 related part of the C socket header
 definitions and structure manipulators for Perl.")
   (license license:bsd-3)))
+
+(define-public perl-net-dns-resolver-programmable
+ (package
+  (name "perl-net-dns-resolver-programmable")
+  (version "v0.003")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ 
"mirror://cpan/authors/id/J/JM/JMEHNLE/net-dns-resolver-programmable/"
+ "Net-DNS-Resolver-Programmable-" version ".tar.gz"))
+  (sha256
+(base32
+  "1v3nl2kaj4fs55n1617n53q8sa3mir06898vpy1rq98zjih24h4d"))
+  (patches
+   (list (origin
+   (method url-fetch)
+   (uri 
"https://rt.cpan.org/Public/Ticket/Attachment/1575108/841078/patch.txt";)
+   (sha256
+ (base32
+   
"027fzq1dryqwhkprz4r32vj78qfdnx1mhjx6piag7p62p5npmdic")))
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs `(("perl-net-dns" ,perl-net-dns)))
+  (home-page
+"http://search.cpan.org/dist/Net-DNS-Resolver-Programmable";)
+  (synopsis
+"programmable DNS resolver class for offline
+emulation of DNS")
+  (description "Net::DNS::Resolver::Programmable is a programmable DNS 
resolver for
+offline emulation of DNS.")
+  (license (package-license perl

[PATCH v3 03/16] gnu: Add perl-socket6.

[Danny Milosavljevic]

gnu: Add perl-socket6.

* gnu/packages/networking.scm (perl-socket6): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 0568836..a6318b2 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -436,3 +436,36 @@ application stack itself.")
 "Perl Interface to the Domain Name System")
   (description "Net::DNS is the Perl Interface to the Domain Name System.")
   (license license:x11)))
+
+(define-public perl-socket6
+ (package
+  (name "perl-socket6")
+  (version "0.28")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/U/UM/UMEMOTO/Socket6-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "11j5jzqbzmwlws9zals43ry2f1nw9qy6im7yhn9ck5rikywrmm5z"
+  (build-system perl-build-system)
+  (arguments
+   `(#:phases
+ (modify-phases %standard-phases
+ (replace 'configure
+   (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+(args `("Makefile.PL"
+,(string-append "PREFIX=" out)
+"INSTALLDIRS=site")))
+   (setenv "CONFIG_SHELL" (which "sh"))
+   (zero? (apply system* "perl" args
+  (home-page "http://search.cpan.org/dist/Socket6";)
+  (synopsis
+"IPv6 related part of the C socket.h defines and structure manipulators 
for Perl")
+  (description "Socket6 binds the IPv6 related part of the C socket header
+definitions and structure manipulators for Perl.")
+  (license license:bsd-3)))

[PATCH v3 02/16] gnu: Add perl-net-dns.

[Danny Milosavljevic]

gnu: Add perl-net-dns.

* gnu/packages/networking.scm (perl-net-dns): New variable.
---
 gnu/packages/networking.scm | 24 
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 1ef401f..0568836 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -24,6 +24,7 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages networking)
+  #:use-module (guix build-system perl)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix download)
@@ -412,3 +413,26 @@ HTTP(S) request, and receive the reply headers.  It is 
somewhat similar to
 by firewalls or when you want to monitor the response time of the actual web
 application stack itself.")
 (license license:gpl2))); with permission to link with OpenSSL
+
+(define-public perl-net-dns
+ (package
+  (name "perl-net-dns")
+  (version "1.06")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/N/NL/NLNETLABS/Net-DNS-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "07m5331132h9xkh1i6jv9d80f571yva27iqa31aq4sm31iw7nn53"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-digest-hmac" ,perl-digest-hmac)))
+  (home-page "http://search.cpan.org/dist/Net-DNS";)
+  (synopsis
+"Perl Interface to the Domain Name System")
+  (description "Net::DNS is the Perl Interface to the Domain Name System.")
+  (license license:x11)))

[PATCH v3 09/16] gnu: Add perl-test-trailingspace.

[Danny Milosavljevic]

gnu: Add perl-test-trailingspace.

* gnu/packages/perl.scm (perl-test-trailingspace): New variable.
---
 gnu/packages/perl.scm | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 5a867e0..944dbf8 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6689,3 +6689,38 @@ File::Find replacement in Perl.")
 interface to File::Find::Object.")
   (license (package-license perl
 
+
+(define-public perl-test-trailingspace
+ (package
+  (name "perl-test-trailingspace")
+  (version "0.0300")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/Test-TrailingSpace-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "01slmrcjfq38mpdg3hlb7lnnbgsqbn26958y3hzx0zwrif40pigr"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-file-find-object" ,perl-file-find-object)
+  ("perl-class-xsaccessor" ,perl-class-xsaccessor)))
+  (inputs
+`(("perl-file-find-object-rule"
+   ,perl-file-find-object-rule)
+  ("perl-text-glob"
+   ,perl-text-glob)
+  ("perl-number-compare"
+   ,perl-number-compare)))
+  (home-page
+"http://search.cpan.org/dist/Test-TrailingSpace";)
+  (synopsis
+"test for trailing space in Perl source files.")
+  (description "Test::TrailingSpace tests for trailing spaces
+in Perl source files.")
+  (license x11)))
+

[PATCH v3 07/16] gnu: Add perl-file-find-object.

[Danny Milosavljevic]

gnu: Add perl-file-find-object.

* gnu/packages/perl.scm (perl-file-find-object): New variable.
---
 gnu/packages/perl.scm | 28 
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index f42abee..8fad274 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6630,3 +6630,31 @@ such that being individual extensions would be 
wasteful.")
 (license (package-license perl
 
 ;;; END: Core module overrides
+
+(define-public perl-file-find-object
+ (package
+  (name "perl-file-find-object")
+  (version "v0.2.13")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/File-Find-Object-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0gf13b76b824s73r5rp00v8xrd6dnb5yi5jjavfc394scqv6ldh4"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs
+`(("perl-class-xsaccessor" ,perl-class-xsaccessor)))
+  (home-page
+"http://search.cpan.org/dist/File-Find-Object";)
+  (synopsis
+"Object-oriented File::Find replacement in Perl")
+  (description "File::Find::Object is an object-oriented
+File::Find replacement in Perl.")
+  (license artistic2.0)))
+

[PATCH v3 11/16] gnu: Add perl-net-cidr-lite.

[Danny Milosavljevic]

gnu: Add perl-net-cidr-lite.

* gnu/packages/networking.scm (perl-net-cidr-lite): Add variable.
---
 gnu/packages/networking.scm | 22 ++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index eb118d5..1f1389e 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -565,3 +565,25 @@ offline emulation of DNS.")
 "Manages IPv4 and IPv6 addresses and subnets")
   (description "NetAddr::IP manages IPv4 and IPv6 addresses and subsets.")
   (license (package-license perl
+
+(define-public perl-net-cidr-lite
+ (package
+  (name "perl-net-cidr-lite")
+  (version "0.21")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/D/DO/DOUGW/Net-CIDR-Lite-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "14shj73zbqmfjbp0qz1fs9j4p2dpvz5hfkm4qfdjbydflbl2b8fg"
+  (build-system perl-build-system)
+  (home-page
+"http://search.cpan.org/dist/Net-CIDR-Lite";)
+  (synopsis
+"Perl extension for merging IPv4 or IPv6 CIDR addresses")
+  (description "Net::CIDR::Lite merges IPv4 or IPv6 CIDR addresses.")
+  (license license:gpl1+)))

[PATCH v3 08/16] gnu: Add perl-file-find-object-rule.

[Danny Milosavljevic]

gnu: Add perl-file-find-object-rule.

* gnu/packages/perl.scm (perl-file-find-object-rule): New variable.
---
 gnu/packages/perl.scm | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 8fad274..5a867e0 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6658,3 +6658,34 @@ such that being individual extensions would be 
wasteful.")
 File::Find replacement in Perl.")
   (license artistic2.0)))
 
+
+(define-public perl-file-find-object-rule
+ (package
+  (name "perl-file-find-object-rule")
+  (version "0.0305")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/File-Find-Object-Rule-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0hs4n3w99q4ylkhg3qhzcwkxqn7zblfj1zjdgl06ca30afkk4cv6"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs
+`(("perl-class-xsaccessor" ,perl-class-xsaccessor)
+  ("perl-file-find-object" ,perl-file-find-object)
+  ("perl-number-compare" ,perl-number-compare)
+  ("perl-text-glob" ,perl-text-glob)))
+  (home-page
+"http://search.cpan.org/dist/File-Find-Object-Rule";)
+  (synopsis
+"Alternative interface to File::Find::Object")
+  (description "File::Find::Object::Rule is an alternative Perl
+interface to File::Find::Object.")
+  (license (package-license perl
+

[PATCH v3 14/16] gnu: Add perl-crypt-openssl-bignum.

[Danny Milosavljevic]

gnu: Add perl-crypt-openssl-bignum.

* gnu/packages/tls.scm (perl-crypt-openssl-bignum, 
perl-crypt-openssl-arguments): New variables.
---
 gnu/packages/tls.scm | 36 
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/tls.scm b/gnu/packages/tls.scm
index d992c51..fba130b 100644
--- a/gnu/packages/tls.scm
+++ b/gnu/packages/tls.scm
@@ -556,3 +556,39 @@ servers or clients for more complicated applications.")
 "RSA encoding and decoding, using the openSSL libraries")
   (description "Crypt::OpenSSL::RSA does RSA encoding and decoding (using the 
OpenSSL libraries).")
   (license (package-license perl
+
+(define perl-crypt-arguments
+   `(#:phases (modify-phases %standard-phases
+  (add-before 'configure 'patch-Makefile.PL
+(lambda* (#:key inputs #:allow-other-keys)
+  (substitute* "Makefile.PL"
+(("'LIBS'.*=>.*") (string-append "'LIBS' => ['-L"
+ (assoc-ref inputs "openssl")
+ "/lib -lcrypto'],")))
+  #t)
+
+(define-public perl-crypt-openssl-bignum
+ (package
+  (name "perl-crypt-openssl-bignum")
+  (version "0.06")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/K/KM/KMX/Crypt-OpenSSL-Bignum-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "05yzrdglrrzp191krf77zrwfkmzrfwrsrx1vyskbj94522lszk67"
+  (build-system perl-build-system)
+  (inputs `(("openssl" ,openssl)))
+  (arguments perl-crypt-arguments)
+  (home-page
+"http://search.cpan.org/dist/Crypt-OpenSSL-Bignum";)
+  (synopsis
+"OpenSSL's multiprecision integer arithmetic in Perl")
+  (description "Crypt::OpenSSL::Bignum provides multiprecision integer
+arithmetic in Perl.")
+  ;; At your option either gpl1+ or the Artistic License
+  (license (package-license perl

[PATCH v3 12/16] gnu: Add perl-geo-ip.

[Danny Milosavljevic]

gnu: Add perl-geo-ip.

* gnu/packages/networking.scm (perl-geo-ip): Add variable.
---
 gnu/packages/networking.scm | 23 +++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 1f1389e..9e6d1b5 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -24,6 +24,29 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages networking)
+;; TODO: Use the geolite-mirror-simple.pl script from the example
+;; directory to stay current with the databases. How?
+(define-public perl-geo-ip
+ (package
+  (name "perl-geo-ip")
+  (version "1.45")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/M/MA/MAXMIND/Geo-IP-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0qinkq2br1cjicbgqb5bvrhm73h7f9f4fgc6bjfs5r6x7316bdqf"
+  (build-system perl-build-system)
+  (home-page "http://search.cpan.org/dist/Geo-IP";)
+  (synopsis
+"Look up location and network information by IP Address in Perl")
+  (description "The Perl module 'Geo::IP'. 
+It looks up location and network information by IP Address.")
+  (license (package-license perl
 ;; Maybe makes sense to add patricialib dependency (it's bundled right now).
 ;; See  for the original package it was extracted from.
 ;; However, the site above doesn't actually work on my computer.

[PATCH v3 10/16] gnu: Add perl-encode-detect.

[Danny Milosavljevic]

gnu: Add perl-encode-detect.

* gnu/packages/perl.scm (perl-encode-detect): Add variable.
---
 gnu/packages/perl.scm | 25 +
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 944dbf8..2ac70bd 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6724,3 +6724,28 @@ interface to File::Find::Object.")
 in Perl source files.")
   (license x11)))
 
+
+(define-public perl-encode-detect
+ (package
+  (name "perl-encode-detect")
+  (version "1.01")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/J/JG/JGMYERS/Encode-Detect-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1wdv9ffgs4xyfh5dnh09dqkmmlbf5m1hxgdgb3qy6v6vlwx8jkc3"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-module-build" ,perl-module-build)))
+  (home-page
+"http://search.cpan.org/dist/Encode-Detect";)
+  (synopsis
+"Perl Encode::Encoding subclass that detects the encoding of data")
+  (description "Encode::Detect detects the encoding of data for Perl.")
+  (license mpl1.1)))
+

[PATCH v3 13/16] gnu: Add perl-crypt-openssl-rsa.

[Danny Milosavljevic]

gnu: Add perl-crypt-openssl-rsa.

* gnu/packages/tls.scm (perl-crypt-openssl-rsa): Add variable.
---
 gnu/packages/tls.scm | 29 +
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/tls.scm b/gnu/packages/tls.scm
index bdc1d7c..d992c51 100644
--- a/gnu/packages/tls.scm
+++ b/gnu/packages/tls.scm
@@ -527,3 +527,32 @@ finally access to the SSL api of the SSLeay/OpenSSL 
package so you can write
 servers or clients for more complicated applications.")
 (license (package-license perl))
 (home-page "http://search.cpan.org/~mikem/Net-SSLeay-1.66/";)))
+
+(define-public perl-crypt-openssl-rsa
+ (package
+  (name "perl-crypt-openssl-rsa")
+  (version "0.28")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/P/PE/PERLER/Crypt-OpenSSL-RSA-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1gnpvv09b2gpifwdzc5jnhama3d1a4c39lzj9hcaicsb8rvzjmsk"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-crypt-openssl-bignum"
+   ,perl-crypt-openssl-bignum)
+  ("perl-crypt-openssl-random"
+   ,perl-crypt-openssl-random)
+  ("openssl" ,openssl)))
+  (arguments perl-crypt-arguments)
+  (home-page
+"http://search.cpan.org/dist/Crypt-OpenSSL-RSA";)
+  (synopsis
+"RSA encoding and decoding, using the openSSL libraries")
+  (description "Crypt::OpenSSL::RSA does RSA encoding and decoding (using the 
OpenSSL libraries).")
+  (license (package-license perl

[PATCH v3 15/16] gnu: Add perl-mail-spf.

[Danny Milosavljevic]

gnu: Add perl-mail-spf.

* gnu/package/mail.scm (perl-mail-spf): New variable.
---
 gnu/packages/mail.scm | 41 +
 1 file changed, 41 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index f7a7623..4169464 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -1262,3 +1262,44 @@ synchronizing with a remote address book, 
@command{vdirsyncer} is recommended.
 Khard can also be used from within the email client @command{mutt}.")
 (home-page "https://github.com/scheibler/khard";)
 (license gpl3+)))
+
+(define-public perl-mail-spf
+ (package
+  (name "perl-mail-spf")
+  (version "v2.9.0")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/J/JM/JMEHNLE/mail-spf/Mail-SPF-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0qk1rfgfm5drj4iyniiabrasrpqv570vzhgz66lwgb67y4amkjv1"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-net-dns-resolver-programmable"
+   ,perl-net-dns-resolver-programmable)))
+  (arguments
+   `(#:phases (modify-phases %standard-phases
+   (add-before 'configure 'modify-Build.PL
+ (lambda* (#:key outputs #:allow-other-keys)
+   (substitute* "Build.PL"
+ (("'/usr/sbin'") (string-append "'"
+ (assoc-ref outputs "out")
+ "/sbin'")))
+ #t)
+  (inputs
+`(("perl-error" ,perl-error)
+  ("perl-net-dns" ,perl-net-dns)
+  ("perl-netaddr-ip" ,perl-netaddr-ip)
+  ("perl-uri" ,perl-uri)))
+  (home-page
+"http://search.cpan.org/dist/Mail-SPF";)
+  (synopsis
+"Perl implementation of Sender Policy Framework")
+  (description "Mail::SPF is the Sender Policy Framework implemented
+in Perl.")
+  (license bsd-3)))

[PATCH v3 16/16] gnu: Add perl-io-socket-inet6

[Danny Milosavljevic]

gnu: Add perl-io-socket-inet6.

* gnu/packages/networking.scm (perl-io-socket-inet6): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 9e6d1b5..9f010ba 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -610,3 +610,36 @@ offline emulation of DNS.")
 "Perl extension for merging IPv4 or IPv6 CIDR addresses")
   (description "Net::CIDR::Lite merges IPv4 or IPv6 CIDR addresses.")
   (license license:gpl1+)))
+
+(define-public perl-io-socket-inet6
+ (package
+  (name "perl-io-socket-inet6")
+  (version "2.72")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/IO-Socket-INET6-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1fqypz6qa5rw2d5y2zq7f49frwra0aln13nhq5gi514j2zx21q45"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-test-pod" ,perl-test-pod)
+  ("perl-test-pod-coverage" ,perl-test-pod-coverage)
+  ;; this doesn't actually work if we do add it.
+  #| ("perl-test-trailingspace" ,perl-test-trailingspace) |#))
+  (inputs `(("perl-socket6" ,perl-socket6)))
+  (arguments `(;; Need network socket API
+   #:tests? #f))
+  (home-page
+"http://search.cpan.org/dist/IO-Socket-INET6";)
+  (synopsis
+"Perl object interface for AF_INET/AF_INET6 domain sockets")
+  (description "IO::Socket::INET6 is an interface for
+AF_INET/AF_INET6 domain sockets in Perl.")
+  (license (package-license perl
+

[PATCH] doc: Run guix refresh -l to find out about dependent packages.

[ng0]

This adds an item about guix refresh to the contributing section of the
docs.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org
>From 5fead8acf58b4ce000c60381a02db688561dbf05 Mon Sep 17 00:00:00 2001
From: ng0 
Date: Thu, 28 Jul 2016 11:06:32 +
Subject: [PATCH] doc: Run guix refresh -l to find out about dependent
 packages.

* doc/contributing.texi (Submitting Patches): Includes an item to
explain that core-updates and core-updates-next exist and that your
change will be applied on them if it affects too many dependent
packages, example: libgcrypt.
---
 doc/contributing.texi | 15 +++
 1 file changed, 15 insertions(+)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index c0755bb..cfad190 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -328,6 +328,21 @@ extensions---or to the operating system kernel---e.g., reliance on
 @code{uname} or @file{/proc} files.
 
 @item
+Run @command{guix refresh --list-dependent} (@pxref{Invoking guix refresh})
+to find out how many packages would rebuild based on your patch.
+When your patch touches important or a big number of packages, it is
+prefered to be applied on the git branch core-updates or core-updates-next.
+
+@example
+guix refresh -l libgcrypt
+Building the following 822 packages would ensure 2221 dependent packages
+are rebuilt:
+@end example
+
+This is a good example for a number which indicates that a change to
+libgcrypt would not simply be applied on master.
+
+@item
 When writing documentation, please use gender-neutral wording when
 referring to people, such as
 @uref{https://en.wikipedia.org/wiki/Singular_they, singular
-- 
2.9.2

[PATCH v3 06/16] gnu: Add perl-net-patricia.

[Danny Milosavljevic]

gnu: Add perl-net-patricia.

* gnu/packages/networking.scm (perl-net-patricia): Add variable.
---
 gnu/packages/networking.scm | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index b575fce..0312fe5 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -535,3 +535,34 @@ offline emulation of DNS.")
 "Manages IPv4 and IPv6 addresses and subnets")
   (description "NetAddr::IP manages IPv4 and IPv6 addresses and subsets.")
   (license (package-license perl
+
+;; Maybe makes sense to add patricialib dependency (it's bundled right now).
+;; See  for the original package it was extracted from.
+;; However, the site above doesn't actually work on my computer.
+(define-public perl-net-patricia
+ (package
+  (name "perl-net-patricia")
+  (version "1.22")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/G/GR/GRUBER/Net-Patricia-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0ln5f57vc8388kyh9vhx2infrdzfhbpgyby74h1qsnhwds95m0vh"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-net-cidr-lite" ,perl-net-cidr-lite)
+  ("perl-socket6" ,perl-socket6)))
+  (home-page
+"http://search.cpan.org/dist/Net-Patricia";)
+  (synopsis
+"Patricia Trie Perl module for fast IP address lookups")
+  (description
+"Net::Patricia does IP address lookups quickly in Perl.")
+  ;; The bindings are licensed under GPL2 or later.
+  ;; libpatricia is licensed under 2-clause BSD.
+  (license (list license:gpl2+ license:bsd-2

[PATCH v4 00/16] Add missing dependencies of Spamassassin

[Danny Milosavljevic]

This patch adds almost all of the dependencies of Spamassassin.

Danny Milosavljevic (16):
  gnu: Add perl-db-file.
  gnu: Add perl-net-dns.
  gnu: Add perl-socket6.
  gnu: Add perl-net-dns-resolver-programmable.
  gnu: Add perl-netaddr-ip.
  gnu: Add perl-net-patricia.
  gnu: Add perl-file-find-object.
  gnu: Add perl-file-find-object-rule.
  gnu: Add perl-test-trailingspace.
  gnu: Add perl-encode-detect.
  gnu: Add perl-net-cidr-lite.
  gnu: Add perl-geo-ip.
  gnu: Add perl-crypt-openssl-rsa.
  gnu: Add perl-crypt-openssl-bignum.
  gnu: Add perl-mail-spf.
  gnu: Add perl-io-socket-inet6.

 gnu/packages/databases.scm  |  32 ++
 gnu/packages/mail.scm   |  41 
 gnu/packages/networking.scm | 231 
 gnu/packages/perl.scm   | 119 +++
 gnu/packages/tls.scm|  65 +
 5 files changed, 488 insertions(+)

Re: [PATCH] gnu: Add perl-net-dns.

[Danny Milosavljevic]

Obsolete

[PATCH v4 16/16] gnu: Add perl-io-socket-inet6

[Danny Milosavljevic]

gnu: Add perl-io-socket-inet6

* gnu/packages/networking.scm (perl-io-socket-inet6): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 660c025..1ff3be4 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -612,3 +612,36 @@ offline emulation of DNS.")
   (description "The Perl module 'Geo::IP'. 
 It looks up location and network information by IP Address.")
   (license (package-license perl
+
+(define-public perl-io-socket-inet6
+ (package
+  (name "perl-io-socket-inet6")
+  (version "2.72")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/IO-Socket-INET6-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1fqypz6qa5rw2d5y2zq7f49frwra0aln13nhq5gi514j2zx21q45"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-test-pod" ,perl-test-pod)
+  ("perl-test-pod-coverage" ,perl-test-pod-coverage)
+  ;; this doesn't actually work if we do add it.
+  #| ("perl-test-trailingspace" ,perl-test-trailingspace) |#))
+  (inputs `(("perl-socket6" ,perl-socket6)))
+  (arguments `(;; Need network socket API
+   #:tests? #f))
+  (home-page
+"http://search.cpan.org/dist/IO-Socket-INET6";)
+  (synopsis
+"Perl object interface for AF_INET/AF_INET6 domain sockets")
+  (description "IO::Socket::INET6 is an interface for
+AF_INET/AF_INET6 domain sockets in Perl.")
+  (license (package-license perl
+

[PATCH v4 14/16] gnu: Add perl-crypt-openssl-bignum.

[Danny Milosavljevic]

gnu: Add perl-crypt-openssl-bignum.

* gnu/packages/tls.scm (perl-crypt-openssl-bignum, 
perl-crypt-openssl-arguments): New variables.
---
 gnu/packages/tls.scm | 36 
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/tls.scm b/gnu/packages/tls.scm
index d992c51..fba130b 100644
--- a/gnu/packages/tls.scm
+++ b/gnu/packages/tls.scm
@@ -556,3 +556,39 @@ servers or clients for more complicated applications.")
 "RSA encoding and decoding, using the openSSL libraries")
   (description "Crypt::OpenSSL::RSA does RSA encoding and decoding (using the 
OpenSSL libraries).")
   (license (package-license perl
+
+(define perl-crypt-arguments
+   `(#:phases (modify-phases %standard-phases
+  (add-before 'configure 'patch-Makefile.PL
+(lambda* (#:key inputs #:allow-other-keys)
+  (substitute* "Makefile.PL"
+(("'LIBS'.*=>.*") (string-append "'LIBS' => ['-L"
+ (assoc-ref inputs "openssl")
+ "/lib -lcrypto'],")))
+  #t)
+
+(define-public perl-crypt-openssl-bignum
+ (package
+  (name "perl-crypt-openssl-bignum")
+  (version "0.06")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/K/KM/KMX/Crypt-OpenSSL-Bignum-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "05yzrdglrrzp191krf77zrwfkmzrfwrsrx1vyskbj94522lszk67"
+  (build-system perl-build-system)
+  (inputs `(("openssl" ,openssl)))
+  (arguments perl-crypt-arguments)
+  (home-page
+"http://search.cpan.org/dist/Crypt-OpenSSL-Bignum";)
+  (synopsis
+"OpenSSL's multiprecision integer arithmetic in Perl")
+  (description "Crypt::OpenSSL::Bignum provides multiprecision integer
+arithmetic in Perl.")
+  ;; At your option either gpl1+ or the Artistic License
+  (license (package-license perl

[PATCH v4 15/16] gnu: Add perl-mail-spf.

[Danny Milosavljevic]

gnu: Add perl-mail-spf.

* gnu/package/mail.scm (perl-mail-spf): New variable.
---
 gnu/packages/mail.scm | 41 +
 1 file changed, 41 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index f7a7623..4169464 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -1262,3 +1262,44 @@ synchronizing with a remote address book, 
@command{vdirsyncer} is recommended.
 Khard can also be used from within the email client @command{mutt}.")
 (home-page "https://github.com/scheibler/khard";)
 (license gpl3+)))
+
+(define-public perl-mail-spf
+ (package
+  (name "perl-mail-spf")
+  (version "v2.9.0")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/J/JM/JMEHNLE/mail-spf/Mail-SPF-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0qk1rfgfm5drj4iyniiabrasrpqv570vzhgz66lwgb67y4amkjv1"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-net-dns-resolver-programmable"
+   ,perl-net-dns-resolver-programmable)))
+  (arguments
+   `(#:phases (modify-phases %standard-phases
+   (add-before 'configure 'modify-Build.PL
+ (lambda* (#:key outputs #:allow-other-keys)
+   (substitute* "Build.PL"
+ (("'/usr/sbin'") (string-append "'"
+ (assoc-ref outputs "out")
+ "/sbin'")))
+ #t)
+  (inputs
+`(("perl-error" ,perl-error)
+  ("perl-net-dns" ,perl-net-dns)
+  ("perl-netaddr-ip" ,perl-netaddr-ip)
+  ("perl-uri" ,perl-uri)))
+  (home-page
+"http://search.cpan.org/dist/Mail-SPF";)
+  (synopsis
+"Perl implementation of Sender Policy Framework")
+  (description "Mail::SPF is the Sender Policy Framework implemented
+in Perl.")
+  (license bsd-3)))

[PATCH v4 13/16] gnu: Add perl-crypt-openssl-rsa.

[Danny Milosavljevic]

gnu: Add perl-crypt-openssl-rsa.

* gnu/packages/tls.scm (perl-crypt-openssl-rsa): Add variable.
---
 gnu/packages/tls.scm | 29 +
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/tls.scm b/gnu/packages/tls.scm
index bdc1d7c..d992c51 100644
--- a/gnu/packages/tls.scm
+++ b/gnu/packages/tls.scm
@@ -527,3 +527,32 @@ finally access to the SSL api of the SSLeay/OpenSSL 
package so you can write
 servers or clients for more complicated applications.")
 (license (package-license perl))
 (home-page "http://search.cpan.org/~mikem/Net-SSLeay-1.66/";)))
+
+(define-public perl-crypt-openssl-rsa
+ (package
+  (name "perl-crypt-openssl-rsa")
+  (version "0.28")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/P/PE/PERLER/Crypt-OpenSSL-RSA-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1gnpvv09b2gpifwdzc5jnhama3d1a4c39lzj9hcaicsb8rvzjmsk"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-crypt-openssl-bignum"
+   ,perl-crypt-openssl-bignum)
+  ("perl-crypt-openssl-random"
+   ,perl-crypt-openssl-random)
+  ("openssl" ,openssl)))
+  (arguments perl-crypt-arguments)
+  (home-page
+"http://search.cpan.org/dist/Crypt-OpenSSL-RSA";)
+  (synopsis
+"RSA encoding and decoding, using the openSSL libraries")
+  (description "Crypt::OpenSSL::RSA does RSA encoding and decoding (using the 
OpenSSL libraries).")
+  (license (package-license perl

[PATCH v4 04/16] gnu: Add perl-net-dns-resolver-programmable.

[Danny Milosavljevic]

gnu: Add perl-net-dns-resolver-programmable.

* gnu/packages/networking.scm (perl-net-dns-resolver-programmable): New 
variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a6318b2..a69ef0e 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -469,3 +469,36 @@ application stack itself.")
   (description "Socket6 binds the IPv6 related part of the C socket header
 definitions and structure manipulators for Perl.")
   (license license:bsd-3)))
+
+(define-public perl-net-dns-resolver-programmable
+ (package
+  (name "perl-net-dns-resolver-programmable")
+  (version "v0.003")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ 
"mirror://cpan/authors/id/J/JM/JMEHNLE/net-dns-resolver-programmable/"
+ "Net-DNS-Resolver-Programmable-" version ".tar.gz"))
+  (sha256
+(base32
+  "1v3nl2kaj4fs55n1617n53q8sa3mir06898vpy1rq98zjih24h4d"))
+  (patches
+   (list (origin
+   (method url-fetch)
+   (uri 
"https://rt.cpan.org/Public/Ticket/Attachment/1575108/841078/patch.txt";)
+   (sha256
+ (base32
+   
"027fzq1dryqwhkprz4r32vj78qfdnx1mhjx6piag7p62p5npmdic")))
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs `(("perl-net-dns" ,perl-net-dns)))
+  (home-page
+"http://search.cpan.org/dist/Net-DNS-Resolver-Programmable";)
+  (synopsis
+"programmable DNS resolver class for offline
+emulation of DNS")
+  (description "Net::DNS::Resolver::Programmable is a programmable DNS 
resolver for
+offline emulation of DNS.")
+  (license (package-license perl

[PATCH v4 02/16] gnu: Add perl-net-dns.

[Danny Milosavljevic]

gnu: Add perl-net-dns.

* gnu/packages/networking.scm (perl-net-dns): New variable.
---
 gnu/packages/networking.scm | 24 
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 1ef401f..0568836 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -24,6 +24,7 @@
 ;;; along with GNU Guix.  If not, see .
 
 (define-module (gnu packages networking)
+  #:use-module (guix build-system perl)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix download)
@@ -412,3 +413,26 @@ HTTP(S) request, and receive the reply headers.  It is 
somewhat similar to
 by firewalls or when you want to monitor the response time of the actual web
 application stack itself.")
 (license license:gpl2))); with permission to link with OpenSSL
+
+(define-public perl-net-dns
+ (package
+  (name "perl-net-dns")
+  (version "1.06")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/N/NL/NLNETLABS/Net-DNS-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "07m5331132h9xkh1i6jv9d80f571yva27iqa31aq4sm31iw7nn53"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-digest-hmac" ,perl-digest-hmac)))
+  (home-page "http://search.cpan.org/dist/Net-DNS";)
+  (synopsis
+"Perl Interface to the Domain Name System")
+  (description "Net::DNS is the Perl Interface to the Domain Name System.")
+  (license license:x11)))

[PATCH v4 10/16] gnu: Add perl-encode-detect.

[Danny Milosavljevic]

gnu: Add perl-encode-detect.

* gnu/packages/perl.scm (perl-encode-detect): Add variable.
---
 gnu/packages/perl.scm | 25 +
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 944dbf8..2ac70bd 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6724,3 +6724,28 @@ interface to File::Find::Object.")
 in Perl source files.")
   (license x11)))
 
+
+(define-public perl-encode-detect
+ (package
+  (name "perl-encode-detect")
+  (version "1.01")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/J/JG/JGMYERS/Encode-Detect-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1wdv9ffgs4xyfh5dnh09dqkmmlbf5m1hxgdgb3qy6v6vlwx8jkc3"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-module-build" ,perl-module-build)))
+  (home-page
+"http://search.cpan.org/dist/Encode-Detect";)
+  (synopsis
+"Perl Encode::Encoding subclass that detects the encoding of data")
+  (description "Encode::Detect detects the encoding of data for Perl.")
+  (license mpl1.1)))
+

[PATCH v4 09/16] gnu: Add perl-test-trailingspace.

[Danny Milosavljevic]

gnu: Add perl-test-trailingspace.

* gnu/packages/perl.scm (perl-test-trailingspace): New variable.
---
 gnu/packages/perl.scm | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 5a867e0..944dbf8 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6689,3 +6689,38 @@ File::Find replacement in Perl.")
 interface to File::Find::Object.")
   (license (package-license perl
 
+
+(define-public perl-test-trailingspace
+ (package
+  (name "perl-test-trailingspace")
+  (version "0.0300")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/Test-TrailingSpace-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "01slmrcjfq38mpdg3hlb7lnnbgsqbn26958y3hzx0zwrif40pigr"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)
+  ("perl-file-find-object" ,perl-file-find-object)
+  ("perl-class-xsaccessor" ,perl-class-xsaccessor)))
+  (inputs
+`(("perl-file-find-object-rule"
+   ,perl-file-find-object-rule)
+  ("perl-text-glob"
+   ,perl-text-glob)
+  ("perl-number-compare"
+   ,perl-number-compare)))
+  (home-page
+"http://search.cpan.org/dist/Test-TrailingSpace";)
+  (synopsis
+"test for trailing space in Perl source files.")
+  (description "Test::TrailingSpace tests for trailing spaces
+in Perl source files.")
+  (license x11)))
+

[PATCH v4 12/16] gnu: Add perl-geo-ip.

[Danny Milosavljevic]

gnu: Add perl-geo-ip.

* gnu/packages/networking.scm (perl-geo-ip): Add variable.
---
 gnu/packages/networking.scm | 24 
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 134ad5d..660c025 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -588,3 +588,27 @@ offline emulation of DNS.")
 "Perl extension for merging IPv4 or IPv6 CIDR addresses")
   (description "Net::CIDR::Lite merges IPv4 or IPv6 CIDR addresses.")
   (license license:gpl1+)))
+
+;; TODO: Use the geolite-mirror-simple.pl script from the example
+;; directory to stay current with the databases. How?
+(define-public perl-geo-ip
+ (package
+  (name "perl-geo-ip")
+  (version "1.45")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/M/MA/MAXMIND/Geo-IP-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0qinkq2br1cjicbgqb5bvrhm73h7f9f4fgc6bjfs5r6x7316bdqf"
+  (build-system perl-build-system)
+  (home-page "http://search.cpan.org/dist/Geo-IP";)
+  (synopsis
+"Look up location and network information by IP Address in Perl")
+  (description "The Perl module 'Geo::IP'. 
+It looks up location and network information by IP Address.")
+  (license (package-license perl

[PATCH v4 08/16] gnu: Add perl-file-find-object-rule.

[Danny Milosavljevic]

gnu: Add perl-file-find-object-rule.

* gnu/packages/perl.scm (perl-file-find-object-rule): New variable.
---
 gnu/packages/perl.scm | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index 8fad274..5a867e0 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6658,3 +6658,34 @@ such that being individual extensions would be 
wasteful.")
 File::Find replacement in Perl.")
   (license artistic2.0)))
 
+
+(define-public perl-file-find-object-rule
+ (package
+  (name "perl-file-find-object-rule")
+  (version "0.0305")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/File-Find-Object-Rule-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0hs4n3w99q4ylkhg3qhzcwkxqn7zblfj1zjdgl06ca30afkk4cv6"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs
+`(("perl-class-xsaccessor" ,perl-class-xsaccessor)
+  ("perl-file-find-object" ,perl-file-find-object)
+  ("perl-number-compare" ,perl-number-compare)
+  ("perl-text-glob" ,perl-text-glob)))
+  (home-page
+"http://search.cpan.org/dist/File-Find-Object-Rule";)
+  (synopsis
+"Alternative interface to File::Find::Object")
+  (description "File::Find::Object::Rule is an alternative Perl
+interface to File::Find::Object.")
+  (license (package-license perl
+

[PATCH v4 11/16] gnu: Add perl-net-cidr-lite.

[Danny Milosavljevic]

gnu: Add perl-net-cidr-lite.

* gnu/packages/networking.scm (perl-net-cidr-lite): Add variable.
---
 gnu/packages/networking.scm | 22 ++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 0312fe5..134ad5d 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -566,3 +566,25 @@ offline emulation of DNS.")
   ;; The bindings are licensed under GPL2 or later.
   ;; libpatricia is licensed under 2-clause BSD.
   (license (list license:gpl2+ license:bsd-2
+
+(define-public perl-net-cidr-lite
+ (package
+  (name "perl-net-cidr-lite")
+  (version "0.21")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/D/DO/DOUGW/Net-CIDR-Lite-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "14shj73zbqmfjbp0qz1fs9j4p2dpvz5hfkm4qfdjbydflbl2b8fg"
+  (build-system perl-build-system)
+  (home-page
+"http://search.cpan.org/dist/Net-CIDR-Lite";)
+  (synopsis
+"Perl extension for merging IPv4 or IPv6 CIDR addresses")
+  (description "Net::CIDR::Lite merges IPv4 or IPv6 CIDR addresses.")
+  (license license:gpl1+)))

[PATCH v4 06/16] gnu: Add perl-net-patricia.

[Danny Milosavljevic]

gnu: Add perl-net-patricia.

* gnu/packages/networking.scm (perl-net-patricia): Add variable.
---
 gnu/packages/networking.scm | 31 +++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index b575fce..0312fe5 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -535,3 +535,34 @@ offline emulation of DNS.")
 "Manages IPv4 and IPv6 addresses and subnets")
   (description "NetAddr::IP manages IPv4 and IPv6 addresses and subsets.")
   (license (package-license perl
+
+;; Maybe makes sense to add patricialib dependency (it's bundled right now).
+;; See  for the original package it was extracted from.
+;; However, the site above doesn't actually work on my computer.
+(define-public perl-net-patricia
+ (package
+  (name "perl-net-patricia")
+  (version "1.22")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/G/GR/GRUBER/Net-Patricia-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0ln5f57vc8388kyh9vhx2infrdzfhbpgyby74h1qsnhwds95m0vh"
+  (build-system perl-build-system)
+  (inputs
+`(("perl-net-cidr-lite" ,perl-net-cidr-lite)
+  ("perl-socket6" ,perl-socket6)))
+  (home-page
+"http://search.cpan.org/dist/Net-Patricia";)
+  (synopsis
+"Patricia Trie Perl module for fast IP address lookups")
+  (description
+"Net::Patricia does IP address lookups quickly in Perl.")
+  ;; The bindings are licensed under GPL2 or later.
+  ;; libpatricia is licensed under 2-clause BSD.
+  (license (list license:gpl2+ license:bsd-2

[PATCH v4 07/16] gnu: Add perl-file-find-object.

[Danny Milosavljevic]

gnu: Add perl-file-find-object.

* gnu/packages/perl.scm (perl-file-find-object): New variable.
---
 gnu/packages/perl.scm | 28 
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/perl.scm b/gnu/packages/perl.scm
index f42abee..8fad274 100644
--- a/gnu/packages/perl.scm
+++ b/gnu/packages/perl.scm
@@ -6630,3 +6630,31 @@ such that being individual extensions would be 
wasteful.")
 (license (package-license perl
 
 ;;; END: Core module overrides
+
+(define-public perl-file-find-object
+ (package
+  (name "perl-file-find-object")
+  (version "v0.2.13")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/S/SH/SHLOMIF/File-Find-Object-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0gf13b76b824s73r5rp00v8xrd6dnb5yi5jjavfc394scqv6ldh4"
+  (build-system perl-build-system)
+  (native-inputs
+`(("perl-module-build" ,perl-module-build)))
+  (inputs
+`(("perl-class-xsaccessor" ,perl-class-xsaccessor)))
+  (home-page
+"http://search.cpan.org/dist/File-Find-Object";)
+  (synopsis
+"Object-oriented File::Find replacement in Perl")
+  (description "File::Find::Object is an object-oriented
+File::Find replacement in Perl.")
+  (license artistic2.0)))
+

[PATCH v4 01/16] gnu: Add perl-db_file.

[Danny Milosavljevic]

gnu: Add perl-db_file.

* gnu/packages/databases.scm (perl-db_file): New variable.
---
 gnu/packages/databases.scm | 32 
 1 file changed, 32 insertions(+)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index f088062..3bdb37b 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1046,3 +1046,35 @@ trees (LSM), for sustained throughput under random 
insert workloads.")
   (description
 "The DB::File module provides Perl bindings to the Berkeley DB version 
1.x.")
   (license (package-license perl
+
+(define-public perl-db-file
+ (package
+  (name "perl-db-file")
+  (version "1.838")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/P/PM/PMQS/DB_File-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "0yp5d5zr8dk9g6xdh7ygi5bq63q7nxvhd58dk2i3ki4nb7yv2yh9"
+  (build-system perl-build-system)
+  (inputs `(("bdb" ,bdb)))
+  (native-inputs `(("perl-test-pod" ,perl-test-pod)))
+  (arguments
+ `(#:phases (modify-phases %standard-phases
+  (add-before
+   'configure 'modify-config.in
+   (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "config.in"
+   (("/usr/local/BerkeleyDB") (assoc-ref inputs "bdb")))
+ #t)
+  (home-page "http://search.cpan.org/dist/DB_File";)
+  (synopsis
+"Perl5 access to Berkeley DB version 1.x")
+  (description
+"DB::File provides Perl bindings to Berkeley DB version 1.x.")
+  (license (package-license perl

[PATCH v4 05/16] gnu: Add perl-netaddr-ip.

[Danny Milosavljevic]

gnu: Add perl-netaddr-ip.

* gnu/packages/networking.scm (perl-netaddr-ip): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a69ef0e..b575fce 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -502,3 +502,36 @@ emulation of DNS")
   (description "Net::DNS::Resolver::Programmable is a programmable DNS 
resolver for
 offline emulation of DNS.")
   (license (package-license perl
+
+(define-public perl-netaddr-ip
+ (package
+  (name "perl-netaddr-ip")
+  (version "4.079")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/M/MI/MIKER/NetAddr-IP-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "1rx0dinrz9fk9qcg4rwqq5n1dm3xv2arymixpclcv2q2nzgq4npc"
+  (build-system perl-build-system)
+  (arguments
+`(#:phases
+   (modify-phases %standard-phases
+ (replace 'configure
+   (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+(args `("Makefile.PL"
+,(string-append "PREFIX=" out)
+"INSTALLDIRS=site")))
+   (setenv "CONFIG_SHELL" (which "sh"))
+   (zero? (apply system* "perl" args
+  (home-page
+"http://search.cpan.org/dist/NetAddr-IP";)
+  (synopsis
+"Manages IPv4 and IPv6 addresses and subnets")
+  (description "NetAddr::IP manages IPv4 and IPv6 addresses and subsets.")
+  (license (package-license perl

[PATCH v4 03/16] gnu: Add perl-socket6.

[Danny Milosavljevic]

gnu: Add perl-socket6.

* gnu/packages/networking.scm (perl-socket6): New variable.
---
 gnu/packages/networking.scm | 33 +
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 0568836..a6318b2 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -436,3 +436,36 @@ application stack itself.")
 "Perl Interface to the Domain Name System")
   (description "Net::DNS is the Perl Interface to the Domain Name System.")
   (license license:x11)))
+
+(define-public perl-socket6
+ (package
+  (name "perl-socket6")
+  (version "0.28")
+  (source
+(origin
+  (method url-fetch)
+  (uri (string-append
+ "mirror://cpan/authors/id/U/UM/UMEMOTO/Socket6-"
+ version
+ ".tar.gz"))
+  (sha256
+(base32
+  "11j5jzqbzmwlws9zals43ry2f1nw9qy6im7yhn9ck5rikywrmm5z"
+  (build-system perl-build-system)
+  (arguments
+   `(#:phases
+ (modify-phases %standard-phases
+ (replace 'configure
+   (lambda* (#:key outputs #:allow-other-keys)
+ (let* ((out (assoc-ref outputs "out"))
+(args `("Makefile.PL"
+,(string-append "PREFIX=" out)
+"INSTALLDIRS=site")))
+   (setenv "CONFIG_SHELL" (which "sh"))
+   (zero? (apply system* "perl" args
+  (home-page "http://search.cpan.org/dist/Socket6";)
+  (synopsis
+"IPv6 related part of the C socket.h defines and structure manipulators 
for Perl")
+  (description "Socket6 binds the IPv6 related part of the C socket header
+definitions and structure manipulators for Perl.")
+  (license license:bsd-3)))

Re: [PATCH] add go@1.6

[Alex Griffin]

On Thu, Jul 28, 2016, at 05:11 AM, Matthew Jordan wrote:
> Doesn't sh already handle picking the shell/interpretor?  That's why I
> put it there. sh or an sh compatible shell is usually the first to
> start. Feel free to correct me if I'm wrong.

In Guix, sh is provided by bash, but when bash is launched as sh it gets
put into a compatibility mode which might not be what script authors
expect. In this case I guess it doesn't really matter, it's a small
thing and it works either way.

> This sounds like the beginning of a long conversation. Suffice to say
> it's doesn't bother me either way.  However I would rather change this
> in later patch, if it has to be changed.

Okay, sounds fine to me.

Thanks for all your hard work!
-- 
Alex Griffin

Re: [PATCH] gnu: Add perl-socket6.

[ng0]

Danny Milosavljevic  writes:

>> could you fix the commit message the same way pointed out in the
>> previous (perl-net-dns) package and 
>
>>rebase it on master?
>
> Sorry, I cannot do that manually for like 20 packages. 

I did not see that they are all related, only the 3 of them I looked
at.

> This is Scheme we are talking about - one of the most regular languages there 
> is. There should be a patching and diffing program which works on the tree 
> level and we should be using that.
>
> *shakes head* This is not C or anything where we would have to live with that.
>
> Anyway, for the time being I wrote a small script to automate the process of 
> reapplying these from the mailing list and then resubmit. It's running right 
> now.
>
> Even after I resubmitted all the patches, they will still only apply when 
> applied in chronological order. Therefore, I will submit a patch series.
>
> Script is attached for reference.

Thanks,

I'm relatively new to scheme, so this should be useful for future
reference for myself and other newcomers.

I'll review them later today, as soon as I have time.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: core-updates, next release, and all that

[Ben Woodcroft]

On 28/07/16 20:34, Andreas Enge wrote:

On Thu, Jul 28, 2016 at 08:21:03PM +1000, Ben Woodcroft wrote:

Fixed in 'f2e791750'. This was another case of BioConductor deleting out of
date sources, and more encouragement to move to SVN as discussed previously.
https://lists.gnu.org/archive/html/guix-devel/2016-06/msg00683.html

Great, thanks! I cherry-picked into core-updates, hoping that git will be
intelligent enough to not complain when we merge back core-updates into
master.


There was a number of further bioconductor updates. I pushed these to 
core-updates, and I'll soon push them to master as well. Would you mind 
starting another evaluation please?


Thanks,
ben

Re: [PATCH v4 00/16] Add missing dependencies of Spamassassin

[ng0]

Danny Milosavljevic  writes:

> This patch adds almost all of the dependencies of Spamassassin.
>
> Danny Milosavljevic (16):
>   gnu: Add perl-db-file.
>   gnu: Add perl-net-dns.
>   gnu: Add perl-socket6.
>   gnu: Add perl-net-dns-resolver-programmable.
>   gnu: Add perl-netaddr-ip.
>   gnu: Add perl-net-patricia.
>   gnu: Add perl-file-find-object.
>   gnu: Add perl-file-find-object-rule.
>   gnu: Add perl-test-trailingspace.
>   gnu: Add perl-encode-detect.
>   gnu: Add perl-net-cidr-lite.
>   gnu: Add perl-geo-ip.
>   gnu: Add perl-crypt-openssl-rsa.
>   gnu: Add perl-crypt-openssl-bignum.
>   gnu: Add perl-mail-spf.
>   gnu: Add perl-io-socket-inet6.
>
>  gnu/packages/databases.scm  |  32 ++
>  gnu/packages/mail.scm   |  41 
>  gnu/packages/networking.scm | 231 
> 
>  gnu/packages/perl.scm   | 119 +++
>  gnu/packages/tls.scm|  65 +
>  5 files changed, 488 insertions(+)
>

Should I review v3 or v4? It would be easier if the patches would have
references in one single thread, but I can work this way too. If this
makes all the packages before obsolete I'll mark them as such and review
only these versions.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Whole-Package Extractor Program (extracts a whole package from a patch (that would be adding an entire package) and applies it to a master clone and commits)

[Danny Milosavljevic]

Hi,

On Thu, 28 Jul 2016 11:36:03 +
ng0  wrote:

>I did not see that they are all related, only the 3 of them I looked at.

Yeah, that was my fault. I should have done a patch series in the first place - 
given that these perl package patches touch the same files and so probably 
won't apply out-of-order.

> I'm relatively new to scheme, so this should be useful for future
> reference for myself and other newcomers.

The Python script I attached (new version attached again) can only handle the 
special case of a patch which patches well-formatted (i.e. newline where it 
should be etc) scm files which adds packages and removes nothing.

It uses heuristics and is not the right fix - but half a solution is better 
than none.

What it can do is you give it a patch or E-Mail name on the commandline and it 
will read it and extract the "+" blocks and just append them to the guix git 
master. There are also a few special cases handled like adapting the 
define-module form if necessary but it's not really safe in general - i.e. you 
have to verify the result manually.

Still beats rewriting all the patches manually :)

For example you give it a patch containing

  diff a/foo.scm b/foo.scm
  +++ b/foo.scm
  @@ 22 number doesn't matter
  +(define foo 2)

and it will append

  (define foo 2)

to the end of the file "foo.scm" in a fixed directory ~/src/guix-master/guix .

A better solution would be to add something like "guix edit" to guix which 
extracts the package and returns an S-Expression to send to the list - not 
forgetting eventually required module header changes (imports etc). Otherwise 
it will always depend on the order the patches from different people are 
applied. That won't scale.

reapply-guix-patch-to-master
Description: Binary data

Re: [PATCH v4 00/16] Add missing dependencies of Spamassassin

[Danny Milosavljevic]

> Should I review v3 or v4? 

v4. I made a mistake in v3 - that's why it has a hole in the series - I didn't 
send it.

> It would be easier if the patches would have references in one single thread, 
> but I can work this way too. 

Huh, they should be a tree with the cover letter at the root now.

>If this
> makes all the packages before obsolete I'll mark them as such and review
> only these versions.

It does make all the "perl" packages (only perl) by me from before obsolete.

Re: [PATCH] doc: Run guix refresh -l to find out about dependent packages.

[Vincent Legoll]

Maybe, it'd better to get a real native English speaker review, but
until he comes, here are my comments:

"how many packages would rebuild"
"how many packages have to be rebuilt"

"for a number which indicates that a change to"
"for why your change to"

IANANES...

-- 
Vincent Legoll

Re: [PATCH] add go@1.6

[Matthew Jordan]

>> (version "1.6.2")

I'll update the version in an updated patch. Assuming no problems I'll
be happy to submit it.

>>(zero? (system* "sh" "all.bash")
>
> This seems to work, but I think it should just be `(zero? (system*
> "./all.bash"))`. It has a shebang, and expects bash, not sh.

Doesn't sh already handle picking the shell/interpretor?  That's why I
put it there. sh or an sh compatible shell is usually the first to
start. Feel free to correct me if I'm wrong.

> If it doesn't cause any problems, we should delete pkg/bootstrap/* so
> that it doesn't retain any references to go@1.4. I think that would
> drastically reduce the size of its closure.

I'll look into this and get back to you.

> Also, I'm not sure I'm sold on splitting up into multiple outputs, but I
> haven't thought about it much and don't have a strong opinion. The
> "tests" output is only 5M and "doc" is only 10M, vs 270M for the default
> output.

This sounds like the beginning of a long conversation. Suffice to say
it's doesn't bother me either way.  However I would rather change this
in later patch, if it has to be changed.

Respectuflly,

Matthew Jordan

Re: gnu/system/u-boot.scm

[Ludovic Courtès]

Hi Danny,

Danny Milosavljevic  skribis:

> On Tue, 26 Jul 2016 22:49:35 +0200
> l...@gnu.org (Ludovic Courtès) wrote:
>
>> >   (u-boot  u-boot-configuration-u-boot   ; package
>> >(default (@ (gnu packages u-boot) (make-u-boot-package 
>> > board  
>> 
>> The default value has invalid syntax.  Should be simply:
>> 
>>   (default (make-u-boot-package board))
>> 
>> but I think this doesn’t work (‘board’ will be unbound; yeah,
>> counter-intuitive.)
>> 
>> You could instead do (default #f) and call ‘make-u-boot-package’ when
>> that value is #f.
>> 
>> > (define (eye-candy config root-fs system port)
>> >   "dummy"
>> >   (mlet* %store-monad ((image #f))
>> > (return (and image
>> >  #~(format #$port "")
>> >  
>> 
>> Simply remove it.  :-)
>
> Yeah, but there's a 
>
>   (mlet %store-monad ((sugar (eye-candy config store-fs system #~port)))
>
> in the same file.
>
> Can I remove that and #$sugar , too? Will it still work?

Yes.

> Also, I'm trying to s/grub.cfg/bootloader-configuration-file/g right now, but 
> I wonder
>
> (1) Whether it's possible to determine the basename of the config-file 
> derivation in order to find out what bootloader to install
> (2) Whether we want to do it that way
>
> .
>
> If so, we could have a install-bootloader routine that detects what the 
> filename of the bootloader-configuration-file object is and then calls either 
> install-grub or install-u-boot.

I think we need two separate procedures on the build side:
‘install-grub’, and ‘install-u-boot’.

Choosing between GRUB and U-Boot should happen on the “host side”,
mostly likely in (gnu system).

HTH,
Ludo’.

Re: guix bootloader selection - wip patch

[Ludovic Courtès]

Danny Milosavljevic  skribis:

> so far I came up with the patch to Guix below for the actual bootloader 
> selection.

Nice!

> Some places are still broken. Search for "FIXME" below.
>
> For example I need a way to find out what the bootloader config file is 
> supposed to be called in the new routine 'install-bootloader . It will get 
> (derivation->output-path bootloader-configuration-file) as argument. Given 
> it, can I still find out whether the filename is "grub.cfg" or 
> "extlinux.conf"? Is that safe enough?

[...]

> +(define* (install-u-boot extlinux.conf device mount-point)
> +  "Install U-Boot with EXTLINUX.CONF on DEVICE, which is assumed to be 
> mounted on
> +MOUNT-POINT. FIXME is that correct?"
> +(install-bootloader-config extlinux.conf
> +   (string-append mount-point
> +  "/extlinux.conf"))
> +(unless (zero? (system* "u-boot-install"
> +(string-append "--boot-directory=" mount-point)
> +device))
> +  (error "failed to install U-Boot")))

Really, installing U-Boot is as simple as this?  If it is, that’s
perfect.  :-)

> -(define* (operating-system-grub.cfg os #:optional (old-entries '()))
> -  "Return the GRUB configuration file for OS.  Use OLD-ENTRIES to populate 
> the
> +(define (bootloader-configuration-device bootloader-configuration)
> +(match bootloader-configuration
> +  (($  config)
> +   (grub-configuration-device config))
> +  (($  config)
> +   (u-boot-configuration-device config

Very good, this is where the bootloader selection should occur.  It
seems to me that it’s enough to find out whether to call ‘install-grub’
or ‘install-u-boot’, no?

I would write it as:

  (match bootloader-configuration
((? grub-configuration? config)
 (grub-configuration-device config))
((? u-boot-configuration? config)
 (u-boot-configuration-device config)))

which does the same thing but allows us to avoid exporting
 and  (better to keep them
private so that external code doesn’t rely on the structure layout.)

I see three separate things here:

  1. Replacing “grub” by “bootloader” in the API (cosmetic change);

  2. Adding the build-side code to install U-Boot;

  3. Adding the host-side code to handle  and do
 the right thing.

To facilitate review, could you separate these three things?

Also, not critical, but could you send patches as ‘text/x-patch’ MIME
attachments so that my email client can perform color highlighting?  :-)

I’m happy to see fast progress on this, thank you!

Ludo’.

Re: [GSoC] Continuous integration tool à la Hydra.

[Ludovic Courtès]

Mathieu Lirzin  skribis:

> Hello,
>
> l...@gnu.org (Ludovic Courtès) writes:
>
>> Mathieu Lirzin  skribis:
>>
> [...]
>>> CREATE TABLE Specifications (
>>>   idINTEGER NOT NULL PRIMARY KEY AUTOINCREMENT,
>>>   repo_name TEXT NOT NULL,
>>>   url   TEXT NOT NULL,
>>>   load_path TEXT NOT NULL,
>>>   file  TEXT NOT NULL,
>>>   proc  TEXT NOT NULL,
>>>   arguments TEXT NOT NULL,
>>>   -- The following columns are optional.
>>>   branchTEXT,
>>>   tag   TEXT,
>>>   revision  TEXT
>>> );
>>>
>>> CREATE TABLE Evaluations (
>>>   derivationTEXT NOT NULL PRIMARY KEY,
>>>   job_name  TEXT NOT NULL,
>>>   specification INTEGER NOT NULL,
>>>   FOREIGN KEY (specification) REFERENCES Specifications (id)
>>> );
>>
>> An evaluation leads to several derivations (for Guix, roughly one
>> derivation per package and per system type), but the table above seems
>> to suggest that each evaluation is mapped to only one derivation?
>
> In my "confused" mind each derivation was considered an individual
> evaluation.  But as you pointed out this makes more sense to decouple
> those.  Fixed in commit d493a58823aed8c556bf795d02207e57718b96c9

Perfect, thanks!

Ludo’.

Re: [PATCH ?] gnu: Add dash.

[Ludovic Courtès]

Hi!

Tobias Geerinckx-Rice  skribis:

> * gnu/packages/dash.shm: New file.
> ---
>
> Guix!
>
> IMO this belongs in a (new) shell.scm with fish, tcsh and zsh.
> Maybe bash, too. On the other hand:
>
> Certain shells might have an 'ecosystem' of 'packages', such as
> bash-completion in bash.scm, that might justify leaving them in
> their own file. I'm thinking specifically of zsh here.
>
> I'd suggest merging them all and splitting out if and when needed,
> but prefer to ask the wisdom of the list before doing that tedious
> deed.

I would keep Bash separate, but I agree it’s a good idea to merge the
other ones in one module.

If you want, you could do that and then add Dash to that file?

[...]

> +(home-page "http://gondor.apana.org.au/~herbert/dash";)
> +(synopsis "POSIX-compliant shell optimised for size")
> +(description
> + "dash is a POSIX-compliant @command{/bin/sh} implementation that aims 
> to be
> +as small as possible, often without sacrificing speed.  It is significantly
> +faster than the GNU Bourne-Again Shell (@command{bash}) at most tasks.  dash 
> is
> +a direct descendant of NetBSD's Almquist Shell (@command{ash}).")
> +(license (list bsd-3
> +   gpl2+; mksignames.c

I’d tend to remove “significantly” :-), but otherwise LGTM, thanks!

Ludo’.

Re: make check-system failure: encrypted-root-os

[Ludovic Courtès]

Hi Chris,

Chris Marusich  skribis:

> Every time I try to run "make check-system", the test
> "encrypted-root-os" fails.  It fails on commit
> b1bf155ffd9b160afdf05aff39d3f0b0f6d11589 (July 18, when I it was
> introduced), and it fails on debc6360e111e8efc8a938b2aef28e5b3616ada8
> (July 26).  I can't find a test log, but the last output from the
> command line was this:

I committed this test by mistake, which is why the commit log doesn’t
mention it, and also the reason why it doesn’t work.  ;-)

My bad!

(Glad you found it though; it suggests a backdoor wouldn’t go unnoticed
for too long…)

> + mount LABEL=my-root /mnt
> [   17.853136] EXT4-fs (dm-0): mounted filesystem with ordered data mode. 
> Opts: (null)
> + herd start cow-store /mnt
> Service cow-store has been started.
> + mkdir /mnt/etc
> + cp /etc/target-config.scm /mnt/etc/config.scm
> + cat /mnt/etc/config
> cat: /mnt/etc/config: No such file or directory
> environment variable `PATH' set to 
> `/gnu/store/9rd90hq8zak77bbx2pk9aqdvkajkv13j-qemu-minimal-2.6.0/bin'
> QEMU runs as PID 6
> connected to QEMU's monitor
> read QEMU monitor prompt
> connected to guest REPL
> marionette is ready
>
> ;;; (uname #("Linux" "gnu" "4.7.0-gnu" "#1 SMP 1" "x86_64"))
> note: keeping build directory `/tmp/guix-build-installation.drv-3'
> builder for `/gnu/store/drq919a4vikymgmhv14x2x1wqp4wwab2-installation.drv' 
> failed with exit code 1
> @ build-failed /gnu/store/drq919a4vikymgmhv14x2x1wqp4wwab2-installation.drv - 
> 1 builder for `/gnu/store/drq919a4vikymgmhv14x2x1wqp4wwab2-installation.drv' 
> failed with exit code 1
> cannot build derivation 
> `/gnu/store/rrif23d8cbbn1wr26rjjcn01yqcch724-encrypted-root-os.drv': 1 
> dependencies couldn't be built
> TOTAL: 5
> PASS: /gnu/store/avcz2mjh8wl418z9irnzn42ky5yy5mxg-basic
> PASS: /gnu/store/6363gfph3lsz73b6lbjc70jza10225zm-mcron
> PASS: /gnu/store/3zzcb9jr5mgdyj6jrsbbwxk4x1avr4ic-nss-mdns
> PASS: /gnu/store/7a82ixw0a800j3lh72fbx8a2fry7cvmn-installed-os
> FAIL: /gnu/store/l24vacjwnwcwr0lrrhwwb94zisz2r0j1-encrypted-root-os
> Makefile:4837: recipe for target 'check-system' failed
> make: *** [check-system] Error 1

Here the test fails during the installation phase, so there’s no
additional log file produced.  You should find a couple of
/tmp/guix-build-* directories, but they are of no use; everything is in
the build log above (here, “cat /mnt/etc/config” fails, which aborts the
installation.)

> I also tried running the test as root, just in case there was some kind
> of permissions problem related to KVM or something.  However, it still
> failed.

No KVM problem here: the transcript that we see above is that of the
installation process running in QEMU/KVM.  See (gnu tests install) for
the complete installation script.

This test was meant to test the patch at ,
which is not committed yet.

I’ll try to resume work on this since I messed up in the first place.

Thanks!

Ludo’.

Re: guix package modif testing

[Ludovic Courtès]

Vincent Legoll  skribis:

> I did, as a user of my guixsd system:
>
> git clone [...] guix
> cd guix
> ./bootstrap
> [ERROR PKG_CONFIG / m4 / etc...]
>
> guix environment guix
> [env]> ./bootstrap
> [env]> ./configure
> [env]> make -j3
>
> I get the BT @ http://paste.lisp.org/+6W42

Namely:

--8<---cut here---start->8---
   ?: 6 [primitive-load-path "gnu/packages/python" ...]
In ice-9/eval.scm:
 432: 5 [eval # ()]
 453: 4 [eval # ()]
 387: 3 [eval # ()]
 411: 2 [eval # #]
 411: 1 [eval # #]
In unknown file:
   ?: 0 [string-append "https://pypi.io/packages/source/"; "p" ...]

ERROR: In procedure string-append:
ERROR: In procedure string-append: Wrong type (expecting string): #
Makefile:4851: recipe for target 'make-go' failed
--8<---cut here---end--->8---

> What's wrong ?

Do you have changes in your work directory?

The problem here is that the ‘version’ parameter of ‘pypi-uri’ (in
guix/build-system/python.scm) points to Guile’s ‘version’ procedure,
whereas it should be a version string.

Ludo’.

Re: Odd behavior with --dry-run and --upgrade

[Ludovic Courtès]

Alex Kost  skribis:

> Ludovic Courtès (2016-07-28 01:19 +0300) wrote:
>
>> Alex Kost  skribis:

[...]

>>> From d7747453bf31a616d414dce293fc0556d601abcb Mon Sep 17 00:00:00 2001
>>> From: Alex Kost 
>>> Date: Wed, 27 Jul 2016 14:55:50 +0300
>>> Subject: [PATCH] emacs: Disable grafts when dry-run is enabled.
>>>
>>> * emacs/guix-main.scm (process-package-actions): Set grafting according
>>> to 'dry-run?'.
>>> * guix/scripts.scm (build-package): Likewise.
>>
>> [...]
>>
>>>  (define-module (guix scripts)
>>> +  #:use-module (guix grafts)
>>>#:use-module (guix utils)
>>>#:use-module (guix ui)
>>>#:use-module (guix store)
>>> @@ -106,6 +107,7 @@ true."
>>>"Build PACKAGE using BUILD-OPTIONS acceptable by 'set-build-options'.
>>>  Show what and how will/would be built."
>>>(mbegin %store-monad
>>> +(set-grafting (not dry-run?))
>>>  (apply set-build-options*
>>> #:use-substitutes? use-substitutes?
>>> (strip-keyword-arguments '(#:dry-run?) build-options))
>>
>> Here it might be best to do something like this:
>>
>>   (mlet %store-monad ((grafting? ((lift0 %graft? %store-monad
>> (set-grafting (and (not dry-run?) grafting?))
>> …)
>>
>> This would make sure we don’t enable grafting if it turned out to be
>> disabled.
>>
>> WDYT?
>
> OK, you know better :-)
>
> However, I tried it and it doesn't work for me (note: I know nothing
> about monads, gexps, etc.).  When I try this:
>
>(mbegin %store-monad
>  (set-grafting #f)
>  …)
>
> the grafting doesn't happen, but when I try this:
>
>(mlet %store-monad ((grafting? ((lift0 %graft? %store-monad
>  (set-grafting #f)
>  …)
>
> grafting happens anyway.  I have no idea what the problem is.

Hmm.  Not sure why.  What about this:

--- a/guix/scripts.scm
+++ b/guix/scripts.scm
@@ -22,6 +22,7 @@
   #:use-module (guix utils)
   #:use-module (guix ui)
   #:use-module (guix store)
+  #:use-module (guix grafts)
   #:use-module (guix monads)
   #:use-module (guix packages)
   #:use-module (guix derivations)
@@ -105,11 +106,14 @@ true."
 #:rest build-options)
   "Build PACKAGE using BUILD-OPTIONS acceptable by 'set-build-options'.
 Show what and how will/would be built."
-  (mbegin %store-monad
+  (mlet %store-monad ((grafting? ((lift0 %graft? %store-monad
 (apply set-build-options*
#:use-substitutes? use-substitutes?
(strip-keyword-arguments '(#:dry-run?) build-options))
-(mlet %store-monad ((derivation (package->derivation package)))
+(mlet %store-monad ((derivation (package->derivation package
+ #:graft?
+ (and (not dry-run?)
+  grafting?
   (mbegin %store-monad
 (maybe-build (list derivation)
  #:use-substitutes? use-substitutes?

Using #:graft? is cleaner anyway.

Ludo’.

Re: (pre-release) [PATCH] torsocks update to 2.2.0-rc1

[Ludovic Courtès]

ng0  skribis:

> Could we make an exception for torsocks here, or should we stick with
> 2.0.0 until 2.2.0 has a version release?

I don’t think so.  :-)  I use torsocks 2.0.0 just fine, FWIW.

A release candidate is normally promptly followed by a release, so we
should use that as soon as it’s available.

Ludo’.

Re: [WIP PATCH] linux-initrd: Allow own set of kernel modules.

[Ludovic Courtès]

Tomáš Čech  skribis:

> * gnu/system/linux-initrd.scm(base-initrd): Add `linux-modules'
>   parameter. Rename former `linux-modules' to
>   `default-linux-modules'. Introduce used-linux-modules to make the code
>   more readable.
> ---
>  gnu/system/linux-initrd.scm | 12 +---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/system/linux-initrd.scm b/gnu/system/linux-initrd.scm
> index bbaa5c0..4934c92 100644
> --- a/gnu/system/linux-initrd.scm
> +++ b/gnu/system/linux-initrd.scm
> @@ -138,6 +138,7 @@ MODULES and taken from LINUX."
>qemu-networking?
>(virtio? #t)
>volatile-root?
> +  (linux-modules #f)
>(extra-modules '()))
>"Return a monadic derivation that builds a generic initrd, with kernel
>  modules taken from LINUX.  FILE-SYSTEMS is a list of file-systems to be

It seems to serve the same purpose as #:extra-modules, no?

  https://www.gnu.org/software/guix/manual/html_node/Initial-RAM-Disk.html

Ludo’.

Re: [PATCH 0/3] Perl log patches fixed

[Ludovic Courtès]

Alex Sassmannshausen  skribis:

> In series, the three revised patches for perl-log.  As seems all to often the
> case, I got pulled to other things.  But I fixed the inputs and wrapped
> descriptions in @code where appropriate.
>
> I'll push these shortly unless you have any further comments.

Sure, thank you!

Ludo’.

Re: [PATCH] doc: Run guix refresh -l to find out about dependent packages.

[Ludovic Courtès]

ng0  skribis:

> --- a/doc/contributing.texi
> +++ b/doc/contributing.texi
> @@ -328,6 +328,21 @@ extensions---or to the operating system kernel---e.g., 
> reliance on
>  @code{uname} or @file{/proc} files.
>  
>  @item
> +Run @command{guix refresh --list-dependent} (@pxref{Invoking guix refresh})
> +to find out how many packages would rebuild based on your patch.
> +When your patch touches important or a big number of packages, it is
> +prefered to be applied on the git branch core-updates or core-updates-next.
> +
> +@example
> +guix refresh -l libgcrypt
> +Building the following 822 packages would ensure 2221 dependent packages
> +are rebuilt:
> +@end example
> +
> +This is a good example for a number which indicates that a change to
> +libgcrypt would not simply be applied on master.

‘guix refresh -l’ is already mentioned in that section:

  https://www.gnu.org/software/guix/manual/html_node/Submitting-Patches.html

The specific branch names are not mentioned, but that’s also because
this process is still in flux.  “core-updates” initially meant “updates
to the core”, where “core” designates GCC, libc, Binutils, and a few
other packages that make up the implicit inputs of gnu-build-system.

Python, libgcrypt, and similar packages are not “core” packages,
strictly speaking.  Ideally, we’d run ‘python-updates’,
‘libgcrypt-updates’, etc. branches, but whether we do this depends on
our build farm capacity and on its current load.

In short, I think it’s best not too give too many details about a
process that’s not set in stone.  :-)

Thoughts?

Ludo’.

Re: [WIP PATCH] linux-initrd: Allow own set of kernel modules.

[Tomáš Čech]

On Thu, Jul 28, 2016 at 03:05:34PM +0200, Ludovic Courtès wrote:

Tomáš Čech  skribis:


* gnu/system/linux-initrd.scm(base-initrd): Add `linux-modules'
  parameter. Rename former `linux-modules' to
  `default-linux-modules'. Introduce used-linux-modules to make the code
  more readable.
---
 gnu/system/linux-initrd.scm | 12 +---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/system/linux-initrd.scm b/gnu/system/linux-initrd.scm
index bbaa5c0..4934c92 100644
--- a/gnu/system/linux-initrd.scm
+++ b/gnu/system/linux-initrd.scm
@@ -138,6 +138,7 @@ MODULES and taken from LINUX."
   qemu-networking?
   (virtio? #t)
   volatile-root?
+  (linux-modules #f)
   (extra-modules '()))
   "Return a monadic derivation that builds a generic initrd, with kernel
 modules taken from LINUX.  FILE-SYSTEMS is a list of file-systems to be


It seems to serve the same purpose as #:extra-modules, no?

 https://www.gnu.org/software/guix/manual/html_node/Initial-RAM-Disk.html


It gives me full control over the modules and not just appending to default set.

 #f --> default-kernel-modules (as it is now)
 '(some extraordinary special modules for other purpose) --> it will use this 
set
 '() --> it will use no module at all (my typical usecase)


But yes, it makes `extra-modules' useless.

I'd like to find a way, how to make default-kernel-modules available
as some lazy evaluated list (because what will go there is decided
when base-initrd is evaluated).

This is one of the steps for user defined kernel (besides specifying
kernel configuration) and I'm using it already.

If you're not against such change, it also would need to adjust
documentation (will do in next round).


S_W


signature.asc
Description: Digital signature

Re: [PATCH] gnu: gnome: Propagate file-roller.

[Ludovic Courtès]

Alex Griffin  skribis:

> From 4a582632170f58a4dcb4d675bc519a5e999b75d0 Mon Sep 17 00:00:00 2001
> From: Alex Griffin 
> Date: Wed, 27 Jul 2016 09:40:35 -0500
> Subject: [PATCH] gnu: gnome: Propagate file-roller.
>
> * gnu/packages/gnome.scm (gnome)[propagated-inputs]: Add file-roller.

Applied, thanks!

Ludo’.

Re: [PATCH] gnu: Add asciinema.

[Ludovic Courtès]

Alex Griffin  skribis:

> From 56dfd86e92c86d2f816b9b0f6f71330cecbc2caf Mon Sep 17 00:00:00 2001
> From: Alex Griffin 
> Date: Wed, 27 Jul 2016 16:22:32 -0500
> Subject: [PATCH] gnu: Add asciinema.
>
> * gnu/packages/terminals.scm (asciinema): New variable.

Applied, thanks!

Ludo’.

Re: [PATCH] doc: Run guix refresh -l to find out about dependent packages.

[Ludovic Courtès]

ng0  skribis:

> * doc/contributing.texi (Submitting Patches): Includes an item to
> explain that core-updates and core-updates-next exist and that your
> change will be applied on them if it affects too many dependent
> packages, example: libgcrypt.

In the end I added something a bit more general in commit
189b1543cbc2db50e6dd309e84bce4eb503f0493.

Ludo’.

Re: [PATCH 2/3] doc: Send changes in your patch which are related.

[Ludovic Courtès]

ng0  skribis:

> From 0e84643a50d578b6b5bb68119b35dc92d90dafd5 Mon Sep 17 00:00:00 2001
> From: ng0 
> Date: Thu, 21 Jul 2016 13:24:36 +
> Subject: [PATCH 2/3] doc: Send changes in your patch which are related.
>
> * doc/contributing.texi (Submitting Patches): New @item:
> This gives information about sending related changes and examples on
> what we view as related changes.

Applied with slightly different wording, thanks!

Ludo’.

Re: [WIP PATCH] linux-initrd: Allow own set of kernel modules.

[Ludovic Courtès]

Tomáš Čech  skribis:

> On Thu, Jul 28, 2016 at 03:05:34PM +0200, Ludovic Courtès wrote:
>>Tomáš Čech  skribis:
>>
>>> * gnu/system/linux-initrd.scm(base-initrd): Add `linux-modules'
>>>   parameter. Rename former `linux-modules' to
>>>   `default-linux-modules'. Introduce used-linux-modules to make the code
>>>   more readable.
>>> ---
>>>  gnu/system/linux-initrd.scm | 12 +---
>>>  1 file changed, 9 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/gnu/system/linux-initrd.scm b/gnu/system/linux-initrd.scm
>>> index bbaa5c0..4934c92 100644
>>> --- a/gnu/system/linux-initrd.scm
>>> +++ b/gnu/system/linux-initrd.scm
>>> @@ -138,6 +138,7 @@ MODULES and taken from LINUX."
>>>qemu-networking?
>>>(virtio? #t)
>>>volatile-root?
>>> +  (linux-modules #f)
>>>(extra-modules '()))
>>>"Return a monadic derivation that builds a generic initrd, with kernel
>>>  modules taken from LINUX.  FILE-SYSTEMS is a list of file-systems to be
>>
>>It seems to serve the same purpose as #:extra-modules, no?
>>
>>  https://www.gnu.org/software/guix/manual/html_node/Initial-RAM-Disk.html
>
> It gives me full control over the modules and not just appending to default 
> set.
>
>  #f --> default-kernel-modules (as it is now)
>  '(some extraordinary special modules for other purpose) --> it will use this 
> set
>  '() --> it will use no module at all (my typical usecase)

Oh, OK.  I had assumed that the last use case didn’t exist ;-), which is
why we have #:extra-modules instead of just #:modules.

> But yes, it makes `extra-modules' useless.
>
> I'd like to find a way, how to make default-kernel-modules available
> as some lazy evaluated list (because what will go there is decided
> when base-initrd is evaluated).

What about this: instead of adding #:linux-modules to ‘base-initrd’,
let’s split ‘base-initrd’ into two procedures:

  1. ‘raw-initrd’ wouldn’t do anything high-level and wouldn’t try to
 guess what modules are needed;

  2. ‘base-initrd’ would keep the interface it currently has, but would
 be rewritten in terms of ‘raw-initrd’:

   (define (base-initrd …)
 ;; …
 (raw-initrd file-system
 #:linux-modules the-full-list-of-modules))

How does that sound?

Ludo’.

Re: [PATCH] doc: Run guix refresh -l to find out about dependent packages.

[ng0]

Vincent Legoll  writes:

> Maybe, it'd better to get a real native English speaker review, but
> until he comes, here are my comments:
>
> "how many packages would rebuild"
> "how many packages have to be rebuilt"
>
> "for a number which indicates that a change to"
> "for why your change to"
>
> IANANES...
>
> -- 
> Vincent Legoll

Yes, it was not perfect.  I can do better, but it was just something I
added because I thought it was missing, to open discussion on the
patch.
Aall of the languages I speak including my native one start to sound
weird occasionally when I'm stressed, including but not limited to
"creative" approaches to language.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: [PATCH] doc: Run guix refresh -l to find out about dependent packages.

[ng0]

Ludovic Courtès  writes:

> ng0  skribis:
>
>> * doc/contributing.texi (Submitting Patches): Includes an item to
>> explain that core-updates and core-updates-next exist and that your
>> change will be applied on them if it affects too many dependent
>> packages, example: libgcrypt.
>
> In the end I added something a bit more general in commit
> 189b1543cbc2db50e6dd309e84bce4eb503f0493.
>
> Ludo’.

Okay, this reads more general and better with the information you gave
about the branches in the previous message.

Thanks.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: [PATCH v4 00/16] Add missing dependencies of Spamassassin

[ng0]

Danny Milosavljevic  writes:

>> Should I review v3 or v4? 
>
> v4. I made a mistake in v3 - that's why it has a hole in the series - I 
> didn't send it.
>
>> It would be easier if the patches would have references in one single 
>> thread, but I can work this way too. 
>
> Huh, they should be a tree with the cover letter at the root now.

Sorry, my mistake this should have been more precise:

I meant to say:
It can lead to confusion when there are several threads with different
versions of the patches, including singles ones, not bundled, and
bundled ones.

I will start looking into these patches now, thanks.

>>If this
>> makes all the packages before obsolete I'll mark them as such and review
>> only these versions.
>
> It does make all the "perl" packages (only perl) by me from before obsolete.

-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

Re: libgd security update / i686 issues

[Leo Famulari]

On Thu, Jul 28, 2016 at 10:40:49AM +0200, Andreas Enge wrote:
> Well, the bug report states that the result is correct on armv7. Apparently
> i686 is not IEEE compliant by default, while armv7 is. So it should be okay
> to apply the flags only on i686. We assume that SSE, but not SSE2 or later
> are supported, see our Qt package.

Thanks for the advice. What do you think about the attached patch?
>From b29dacff62fc7483ea6812a4a09cd68c5578ee90 Mon Sep 17 00:00:00 2001
From: Leo Famulari 
Date: Thu, 28 Jul 2016 02:46:23 -0400
Subject: [PATCH] gnu: gd: Update to 2.2.3.

Fixes CVE-2016-6207.

* gnu/packages/gd.scm (gd): Update to 2.2.3.
[arguments]: Add "-msse -mfpmath=sse" to CFLAGS on i686.
* gnu/packages/patches/gd-CVE-2016-5766.patch,
gnu/packages/patches/gd-CVE-2016-6128.patch,
gnu/packages/patches/gd-CVE-2016-6132.patch,
gnu/packages/patches/gd-CVE-2016-6214.patch,
gnu/packages/patches/gd-fix-test-on-i686.patch: Delete files.
* gnu/local.mk (dist_patch_DATA): Remove them.
---
 gnu/local.mk   |   5 -
 gnu/packages/gd.scm|  17 +-
 gnu/packages/patches/gd-CVE-2016-5766.patch|  81 
 gnu/packages/patches/gd-CVE-2016-6128.patch| 253 -
 gnu/packages/patches/gd-CVE-2016-6132.patch|  55 --
 gnu/packages/patches/gd-CVE-2016-6214.patch|  66 ---
 gnu/packages/patches/gd-fix-test-on-i686.patch |  34 
 7 files changed, 10 insertions(+), 501 deletions(-)
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-5766.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6128.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6132.patch
 delete mode 100644 gnu/packages/patches/gd-CVE-2016-6214.patch
 delete mode 100644 gnu/packages/patches/gd-fix-test-on-i686.patch

diff --git a/gnu/local.mk b/gnu/local.mk
index c143dd7..2f4dda1 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -512,11 +512,6 @@ dist_patch_DATA =		\
   %D%/packages/patches/gcc-cross-environment-variables.patch	\
   %D%/packages/patches/gcc-libvtv-runpath.patch			\
   %D%/packages/patches/gcc-5.0-libvtv-runpath.patch		\
-  %D%/packages/patches/gd-CVE-2016-5766.patch			\
-  %D%/packages/patches/gd-CVE-2016-6128.patch			\
-  %D%/packages/patches/gd-CVE-2016-6132.patch			\
-  %D%/packages/patches/gd-CVE-2016-6214.patch			\
-  %D%/packages/patches/gd-fix-test-on-i686.patch		\
   %D%/packages/patches/gegl-CVE-2012-4433.patch			\
   %D%/packages/patches/geoclue-config.patch			\
   %D%/packages/patches/ghostscript-CVE-2015-3228.patch		\
diff --git a/gnu/packages/gd.scm b/gnu/packages/gd.scm
index 3313ee6..3614d51 100644
--- a/gnu/packages/gd.scm
+++ b/gnu/packages/gd.scm
@@ -24,6 +24,7 @@
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system perl)
   #:use-module (guix download)
+  #:use-module (guix utils)
   #:use-module (gnu packages)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages image)
@@ -40,22 +41,24 @@
 ;; Note: With libgd.org now pointing to github.com, genuine old
 ;; tarballs are no longer available.  Notably, versions 2.0.x are
 ;; missing.
-(version "2.2.2")
+(version "2.2.3")
 
 (source (origin
  (method url-fetch)
  (uri (string-append
"https://github.com/libgd/libgd/releases/download/gd-";
version "/libgd-" version ".tar.xz"))
- (patches (search-patches "gd-fix-test-on-i686.patch"
-  "gd-CVE-2016-5766.patch"
-  "gd-CVE-2016-6128.patch"
-  "gd-CVE-2016-6132.patch"
-  "gd-CVE-2016-6214.patch"))
  (sha256
   (base32
-   "1311g5mva2xlzqv3rjqjc4jjkn5lzls4skvr395h633zw1n7b7s8"
+   "0g3xz8jpz1pl2zzmssglrpa9nxiaa7rmcmvgpbrjz8k9cyynqsvl"
 (build-system gnu-build-system)
+(arguments
+ `(#:configure-flags 
+   (list ,@(let ((system (or (%current-target-system)
+ (%current-system
+ (if (string-prefix? "i686" system)
+   '("CFLAGS=-msse -mfpmath=sse")
+   '())
 (native-inputs
  `(("pkg-config" ,pkg-config)))
 (inputs
diff --git a/gnu/packages/patches/gd-CVE-2016-5766.patch b/gnu/packages/patches/gd-CVE-2016-5766.patch
deleted file mode 100644
index 400cb0a..000
--- a/gnu/packages/patches/gd-CVE-2016-5766.patch
+++ /dev/null
@@ -1,81 +0,0 @@
-Fix CVE-2016-5766 (Integer Overflow in _gd2GetHeader() resulting in heap
-overflow).
-
-https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-5766
-
-Adapted from upstream commits:
-https://github.com/libgd/libgd/commit/aba3db8ba159465ecec1089027a24835a6da9cc0
-https://github.com/libgd/libgd/commit/a6a0e7feabb2a9738086a5dc96348f233c87fa79
-
-Since `patch` cannot apply Git binary diffs, we omit the addition of
-'tests/gd2/php_bug_72339.c' and

Re: [PATCH 4/7] gnu: Add spice-gtk.

[David Craven]

Hi Leo and Ludo!

Thanks! Just wasn't sure if you (Leo) had archived this thread =P

Re: make check-system failure: encrypted-root-os

[Chris Marusich]

l...@gnu.org (Ludovic Courtès) writes:

> I committed this test by mistake, which is why the commit log doesn’t
> mention it, and also the reason why it doesn’t work.  ;-)

OK, thanks for confirming!  Glad to know this had a simple answer.

-- 
Chris


signature.asc
Description: PGP signature

Re: [WIP PATCH] linux-initrd: Allow own set of kernel modules.

[Tobias Geerinckx-Rice]

Hullo,

On 28/07/2016 17:32, Ludovic Courtès wrote:
> Tomáš Čech  skribis:
>>  #f --> default-kernel-modules (as it is now)
>>  '(some extraordinary special modules for other purpose) --> it will use 
>> this set
>>  '() --> it will use no module at all (my typical usecase)
> 
> Oh, OK.  I had assumed that the last use case didn’t exist ;-), which is
> why we have #:extra-modules instead of just #:modules.

How could anyone not want PATA support in their kernel? ;-)

Thanks, Tomáš! I had a similar patch ready, but it still tiptoed about
to keep #:extra-modules around. This is much nicer.

Kind regards,

T G-R

Re: [PATCH 4/7] gnu: Add spice-gtk.

[Leo Famulari]

On Mon, Jul 25, 2016 at 12:14:57AM +0200, Ludovic Courtès wrote:
> David Craven  skribis:
> 
> > What are valid reasons to propagate inputs?
> 
> If an installed .h file of spice-gtk #includes a .h file of a
> dependency, that dependency should be a propagated input.  Likewise if
> its .pc file “Requires” the .pc file of a dependency.

Okay, so many of David's original choices about what to propagate were
correct. Thanks for the clarification, Ludo!

Re: [PATCH v4 00/16] Add missing dependencies of Spamassassin

[ng0]

Update: I am not sure how to handle more than 1 inlined patch with my
new email application.  There will be some delay… I extracted them
manually and introduced formating errors this way, so there has to be
something notmuch or emacs can do.
-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org