[PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

[PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

[Taylan Ulrich Bayırlı/Kammer]

[-- Attachment #1: Type: text/plain, Size: 169 bytes --]

It came up on IRC that the guix package should probably propagate
gnutls, since it's required by 'guix download' for HTTPS URLs.

Should this go to core-updates or so?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-guix-0.8.1-Add-gnutls-to-propagated-inputs.patch --]
[-- Type: text/x-diff, Size: 1340 bytes --]

From 139aefdbd5b2eb07c08c46cdab62517b93a7b656 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Taylan=20Ulrich=20Bay=C4=B1rl=C4=B1/Kammer?=
 <taylanbayirli@gmail.com>
Date: Tue, 12 May 2015 16:32:08 +0200
Subject: [PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

* gnu/packages/package-management.scm (guix-0.8.1): Add gnutls to propagated
  inputs.
---
 gnu/packages/package-management.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/package-management.scm b/gnu/packages/package-management.scm
index 75efd0c..7f082ea 100644
--- a/gnu/packages/package-management.scm
+++ b/gnu/packages/package-management.scm
@@ -29,6 +29,7 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages gnupg)
   #:use-module (gnu packages databases)
+  #:use-module (gnu packages gnutls)
   #:use-module (gnu packages graphviz)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages autotools)
@@ -130,7 +131,8 @@
                        (base32
                         "1mi3brl7l58aww34rawhvja84xc7l1b4hmwdmc36fp9q9mfx0lg5"))))))
     (propagated-inputs
-     `(("guile-json" ,guile-json)
+     `(("gnutls" ,gnutls)
+       ("guile-json" ,guile-json)
        ("geiser" ,geiser)))                       ;for guix.el
 
     (home-page "http://www.gnu.org/software/guix")
-- 
2.2.1

Re: [PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

[Ludovic Courtès]

taylanbayirli@gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:

> It came up on IRC that the guix package should probably propagate
> gnutls, since it's required by 'guix download' for HTTPS URLs.

Could you run “make guix-tarball.x86_64-linux.tar.xz” before and after,
and report the size difference?

> Should this go to core-updates or so?

No, because it’s not an update of a core package (“core” in the sense
that everything depends on it.)

>      (propagated-inputs
> -     `(("guile-json" ,guile-json)
> +     `(("gnutls" ,gnutls)

It would be nice to add a margin comment saying that this is for ‘guix
download’.

I’d like to check the tarball size before making the decision.  It’s
probably fine to increase it somewhat, since it’s a useful addition, but
not too much.

Thanks,
Ludo’.

Re: [PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

[Ludovic Courtès]

ludo@gnu.org (Ludovic Courtès) skribis:

> taylanbayirli@gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:
>
>> It came up on IRC that the guix package should probably propagate
>> gnutls, since it's required by 'guix download' for HTTPS URLs.
>
> Could you run “make guix-tarball.x86_64-linux.tar.xz” before and after,
> and report the size difference?

The new tarball is 41 MiB instead of 37 MiB, which sounds acceptable.

Pushed, thanks!

Ludo’.

Re: [PATCH] gnu: guix-0.8.1: Add gnutls to propagated inputs.

[Taylan Ulrich Kammer]

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

> The new tarball is 41 MiB instead of 37 MiB, which sounds acceptable.
>
> Pushed, thanks!

Thanks for taking care of it. :-)

(We actually moved about two weeks ago and my workflow still hasn't
re-stabilized; apologies for delays.)

Taylan