From: Tobias Geerinckx-Rice <me@tobias.gr>
To: leo@famulari.name, 29542@debbugs.gnu.org
Subject: [bug#29542] rng-tools: New upstream location, new releases
Date: Sun, 3 Dec 2017 20:49:09 +0100 [thread overview]
Message-ID: <f238b944-2f7d-5063-2996-d234647a97a9@tobias.gr> (raw)
In-Reply-To: <20171203003126.GA353@jasmine.lan>
[-- Attachment #1.1: Type: text/plain, Size: 3503 bytes --]
Leo,
Leo Famulari wrote on 03/12/17 at 01:31:
> What do you think?
Hm. Looks legit, no?
> From 779b3b9f3532190592d7f85921829f6691947c46 Mon Sep 17 00:00:00 2001
> From: Leo Famulari <leo@famulari.name>
> Date: Sat, 2 Dec 2017 18:08:21 -0500
> Subject: [PATCH] gnu: rng-tools: Update to 6.1.
>
> Rng-tools development appears to have moved to GitHub [0]:
>
> "NOTE: The rng-tools project hasn't seen updates in some time, but is
> still in active use. To facilitate ongoing development, the rng-tools
> project is moved to here:
> https://github.com/nhorman/rng-tools"
>
> [0]
> https://sourceforge.net/p/gkernel/wiki/Home/
>
> * gnu/packages/linux.scm (rng-tools): Update to 6.1.
> [source]: Use new upstream URL.
> [arguments]: Add a 'bootstrap' phase.
> [native-inputs]: Add autoconf, automake, and pkg-config.
> [inputs]: Add curl, libxml2, sysfsutils, and curl.
s/curl/openssl/
I took a look at the recent commit[0] that added most of these:
“NIST has a randomness beacon available here:
https://www.nist.gov/programs-projects/nist-randomness-beacon
It generates entropy at a rate of 512 bits per minute. Its sent in
cleartext over the internet, making it unsuitable for cryptographic
function, it is useful in the generation of entropy for things like
monte carlo tests or other uses where shared pools of entropy might be
useful. As such, lets add the NIST beacon as an entropy source, but
disable it by default so users have to know to keep it enabled.”
Neat! :-)
I'd be remiss if I didn't point out that it adds 166.3 MiB to the
closure, though. On the one hand, that 's a 240% increase in closure
size for a feature that's ‘disabled by default’ (but read on).
On the other hand, this is a leaf package only installed by users who
want it, and I don't like removing features without better reason. I
also had to add ‘--without-nistbeacon’ to #:configure-flags so it's not
*that* disabled by default...
I'm in mild favour of keeping it, but suggest we add a comment above
those three inputs to point those hacking the recipe in the right direction.
> ---
> gnu/packages/linux.scm | 25 +++++++++++++++++++++----
> 1 file changed, 21 insertions(+), 4 deletions(-)
>
> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> index 995864f33..d1f9f5902 100644
> --- a/gnu/packages/linux.scm
> +++ b/gnu/packages/linux.scm
> @@ -57,6 +57,7 @@
> #:use-module (gnu packages calendar)
> #:use-module (gnu packages check)
> #:use-module (gnu packages crypto)
> + #:use-module (gnu packages curl)
> #:use-module (gnu packages compression)
> #:use-module (gnu packages databases)
> #:use-module (gnu packages datastructures)
> @@ -3489,16 +3490,32 @@ The following service daemons are also provided:
> (define-public rng-tools
> (package
> (name "rng-tools")
> - (version "5")
> + (version "6.1")
> (source (origin
> (method url-fetch)
> (uri (string-append
> - "http://downloads.sourceforge.net/sourceforge/gkernel/"
> - "rng-tools-" version ".tar.gz"))
> + "https://github.com/nhorman/rng-tools/archive/v"
> + version ".tar.gz"))
This is still missing a ‘file-name’.
The rest looks & builds nicely.
Thanks!
T G-R
[0]:
https://github.com/nhorman/rng-tools/commit/aefe862e187a1ef608f9826b2b4a5bbabd5e9945
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 248 bytes --]
next prev parent reply other threads:[~2017-12-03 19:47 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-12-03 0:31 [bug#29542] rng-tools: New upstream location, new releases Leo Famulari
2017-12-03 19:49 ` Tobias Geerinckx-Rice [this message]
2017-12-04 18:45 ` Leo Famulari
2017-12-05 23:11 ` Marius Bakke
2017-12-06 18:27 ` Seeding the Linux RNG at first boot Leo Famulari
2017-12-07 21:07 ` Ludovic Courtès
2017-12-07 23:47 ` Leo Famulari
2017-12-11 9:16 ` Ludovic Courtès
2017-12-11 16:08 ` Leo Famulari
2017-12-20 8:55 ` bug#29542: rng-tools: New upstream location, new releases Leo Famulari
2017-12-06 2:33 ` [bug#29542] " Tobias Geerinckx-Rice
2017-12-06 18:07 ` Leo Famulari
2017-12-06 2:38 ` Tobias Geerinckx-Rice
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f238b944-2f7d-5063-2996-d234647a97a9@tobias.gr \
--to=me@tobias.gr \
--cc=29542@debbugs.gnu.org \
--cc=leo@famulari.name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.