From: Lukas Gradl <lgradl@openmailbox.org>
To: Leo Famulari <leo@famulari.name>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] Add opendht.
Date: Fri, 03 Jun 2016 10:27:32 -0500 [thread overview]
Message-ID: <87bn3iuwbf.fsf@openmailbox.org> (raw)
In-Reply-To: <20160531220415.GA31130@jasmine> (Leo Famulari's message of "Tue, 31 May 2016 18:04:15 -0400")
Hi Leo,
Thank you for your review!
Leo Famulari <leo@famulari.name> writes:
> On Tue, May 31, 2016 at 09:17:29AM -0500, Lukas Gradl wrote:
>> * gnu/packages/telephony.scm (opendht): New variable.
>
> I wonder if there is a better module? Distributed hash tables aren't
> specific to telephony.
I am thinking about putting it in crypto.scm. Do you think that is more
appropriate?
>
>> +(define-public opendht
>> + (let ((commit "13f8c13ac4ebb3b43474d91ca48b42a1019083f4"))
>> + ;; This is the commit used by the Ring Project.
>
> If Ring works with the latest tagged release, then I think we should use
> that instead of the Git commit. They released only 3 days ago, so it's
> unlikely that Ring needs some feature that's not in the tarball.
>
OK, Yes, I will look into that.
>> + (inputs
>> + `(("gnutls" ,gnutls)
>> + ("nettle" ,nettle)
>> + ("msgpack" ,msgpack)
>> + ("readline" ,readline)))
>
> I noticed they bundle Argon2, which is a password hashing library:
> https://github.com/savoirfairelinux/opendht/tree/master/src/argon2
>
> I think we should delete the bundled library in an origin snippet and
> make opendht use an external package.
Oops! Sorry that I missed that. I looked onto that and argon2 bundles
blake2. I am working on patches for the two of those right now.
>
>> + (native-inputs
>> + `(("autoconf" ,autoconf)
>> + ("pkg-config" ,pkg-config)
>> + ("automake" ,automake)
>> + ("libtool" ,libtool)))
>> + (arguments
>> + `(#:configure-flags '("--disable-tools" "--disable-python")
>> + #:phases (modify-phases %standard-phases
>> + (add-before 'configure 'autoconf
>> + (lambda _
>> + (zero? (system* "autoreconf" "-vfi")))))))
>
> If we package a tagged release, it's possible that we won't need to
> bootstrap.
>
>> + (license gpl3))))
>
> I skimmed the source files, and most of them said "either version 3 of
> the License, or (at your option) any later version.", so we should use
> gpl3+.
Ah, OK. I just looked at their README which just says GPL3. I will
change that.
I am planning to submit a new patch after argon2 + blake2. Hopefully
that will happen this weekend.
Thank you!
Best,
Lukas
next prev parent reply other threads:[~2016-06-03 15:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-31 14:17 [PATCH] Add opendht Lukas Gradl
2016-05-31 22:04 ` Leo Famulari
2016-06-03 15:27 ` Lukas Gradl [this message]
2016-06-03 19:53 ` Leo Famulari
2016-06-05 20:50 ` Lukas Gradl
2016-06-25 15:31 ` Leo Famulari
2016-06-25 16:41 ` Lukas Gradl
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87bn3iuwbf.fsf@openmailbox.org \
--to=lgradl@openmailbox.org \
--cc=guix-devel@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 public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).