From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ricardo Wurmus Subject: Re: [PATCH] Add cereal + sparsehash Date: Tue, 1 Dec 2015 11:44:06 +0100 Message-ID: References: <20151130200606.GD10869@debian> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:34446) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3iSA-0007T1-Hk for guix-devel@gnu.org; Tue, 01 Dec 2015 05:46:35 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a3iS6-00029w-HG for guix-devel@gnu.org; Tue, 01 Dec 2015 05:46:34 -0500 Received: from pegasus.bbbm.mdc-berlin.de ([141.80.25.20]:56292) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3iS6-00029r-Ac for guix-devel@gnu.org; Tue, 01 Dec 2015 05:46:30 -0500 In-Reply-To: <20151130200606.GD10869@debian> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Andreas Enge Cc: guix-devel Andreas Enge writes: > On Thu, Nov 26, 2015 at 01:53:25PM +0100, Ricardo Wurmus wrote: >> The second patch adds =E2=80=9Csparsehash=E2=80=9D to the =E2=80=9Ccry= pto=E2=80=9D module. It=E2=80=99s >> probably not the best module for this package =E2=80=94 can you sugges= t any >> other location for a hash table library? > > If it is not a cryptographically secure hash, it should go anywhere, bu= t > not to the cryptography module. Both are called by the same name in Eng= lish, > but are completely different things. Could you move it somewhere else? I=E2=80=99d like to do this, but I=E2=80=99m pretty bad at naming things.= How about a new module =E2=80=9Cdatastructures.scm=E2=80=9D where this could comforta= bly fit in? Or is this too vague? Or should it go in the equally vague =E2=80=9Ccode.sc= m=E2=80=9D (which actually is for tools operating on source code)? ~~ Ricardo