GNU bug report logs - #27043
[PATCH] gnu: Add python-asn1crypto.

Previous Next

Full log

Message #16 received at 27043-done <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: 27043-done <at> debbugs.gnu.org
Subject: Re: bug#27043: [PATCH] gnu: Add python-asn1crypto.
Date: Thu, 25 May 2017 15:44:29 -0400

[Message part 1 (text/plain, inline)]

On Wed, May 24, 2017 at 01:24:41PM +0530, Arun Isaac wrote:
> Leo Famulari writes:
> > * gnu/packages/crypto.scm (python-asn1crypto, python2-asn1crypto): New variables.
> 
> I think we should put this package in gnu/packages/python.scm. Packages
> like python-cryptography are in python.scm.

There are performance issues with Guile 2.2, and the huge (gnu packages
python) module is really a pathological case [0]. So, if a new package
makes sense in other modules, I'll be adding them in those other
modules, at least until the compiler performance is improved.

https://lists.gnu.org/archive/html/guile-devel/2017-05/msg00033.html

> > +        (uri (pypi-uri "asn1crypto" version))
> 
> Release tarballs are availabe on the project's github page. Why not use
> those?
> 
> > +    (arguments
> > +     '(#:tests? #f)) ; Tests are not distributed via PyPi.
> 
> The github release tarballs have tests. Could you package them?

Aha! Good catch.

> 
> > +    (description "Fast ASN.1 parser and serializer with definitions for private
> > +keys, public keys, certificates, CRL, OCSP, CMS, PKCS#3, PKCS#7, PKCS#8,
> > +PKCS#12, PKCS#5, X.509 and TSP.")
> 
> The description should be in full sentences.

Indeed, fixed!

Thanks for the review!

Pushed as 10e65d5e5adde4e17bae47f0b59b36df264ded39

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.