From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] gnu: Update python2-pbkdf2 (refactored) Date: Fri, 17 Jun 2016 20:37:45 -0400 Message-ID: <20160618003745.GC19441@jasmine> References: <47f8d0533eac6b9d414fbee04a548b50@d4n1.org> <69c24b56f827136c7990f919c4f15bbe@d4n1.org> <6585f1df16ff68eaba53178343983e0b@d4n1.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:45276) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bE4H0-0002cF-Hg for guix-devel@gnu.org; Fri, 17 Jun 2016 20:38:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bE4Gy-0007Zm-6q for guix-devel@gnu.org; Fri, 17 Jun 2016 20:38:05 -0400 Content-Disposition: inline In-Reply-To: <6585f1df16ff68eaba53178343983e0b@d4n1.org> 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" To: Daniel Pimentel Cc: guix-devel@gnu.org, Guix-devel On Thu, Jun 16, 2016 at 09:12:56AM -0300, Daniel Pimentel wrote: Thanks! Can you rebase this patch on the current master branch? This patch's context is your patch for python2-jsonrpclib, but I changed the license in that patch, so this patch cannot be applied. In the meantime, some comments: > Subject: [PATCH 2/2] gnu: Add python2-pbkdf2 Can you add the line for the changelog? See the line I added to python2-jsonrpclib's commit message for reference. > --- > gnu/packages/python.scm | 25 +++++++++++++++++++++++++ > 1 file changed, 25 insertions(+) > > diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm > index 6dece06..0d1391b 100644 > --- a/gnu/packages/python.scm > +++ b/gnu/packages/python.scm > @@ -9244,3 +9244,28 @@ It supports both the original 1.0 specification, as well as the > new (proposed) 2.0 spec, which includes batch submission, keyword arguments, > etc.") > (license license:expat))) > + > +(define-public python2-pbkdf2 > + (package > + (name "python2-pbkdf2") > + (version "1.3") > + (source (origin > + (method url-fetch) > + (uri (string-append > + "https://pypi.python.org/packages/source/p/pbkdf2/pbkdf2-" > + version ".tar.gz")) Can you make these lines shorter than 80 characters? > + (sha256 > + (base32 > + "0yb99rl2mbsaamj571s1mf6vgniqh23v98k4632150hjkwv9fqxc")))) > + (build-system python-build-system) > + (inputs > + `(("python2-setuptools" ,python2-setuptools))) Can you check if it works with setuptools as native-input? > + (arguments > + `(#:tests? #f > + #:python ,python-2)) This #:python line should be aligned with #:tests. > + (home-page "https://www.dlitz.net/software/python-pbkdf2/") Unfortunately this server is inaccesible. They have enabled HTTP Strict Transport Security (HSTS), but their TLS certificate has expired. Hopefully they fix this. No change required to the patch, although it would be friendly if you notified them. > + (synopsis "A module that implements the password-based key derivation function PBKDF2") This is too long. I don't have a good suggestion. How about "Password-Based Key Derivation Function 2 in Python"?