all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#27043: [PATCH] gnu: Add python-asn1crypto.
@ 2017-05-23 19:42 Leo Famulari
  2017-05-24  7:54 ` Arun Isaac
       [not found] ` <c493b2f5.AEEAK7CSRGoAAAAAAAAAAAO8ccgAAAACwQwAAAAAAAW9WABZJTxG@mailjet.com>
  0 siblings, 2 replies; 4+ messages in thread
From: Leo Famulari @ 2017-05-23 19:42 UTC (permalink / raw)
  To: 27043

* gnu/packages/crypto.scm (python-asn1crypto, python2-asn1crypto): New variables.
---
 gnu/packages/crypto.scm | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/crypto.scm b/gnu/packages/crypto.scm
index 856308afe..86cd777ef 100644
--- a/gnu/packages/crypto.scm
+++ b/gnu/packages/crypto.scm
@@ -51,7 +51,8 @@
   #:use-module (guix download)
   #:use-module (guix git-download)
   #:use-module (guix build-system cmake)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build-system python))
 
 (define-public libsodium
   (package
@@ -414,3 +415,27 @@ utility as a demonstration of the @code{scrypt} key derivation function.
 @code{Scrypt} is designed to be far more resistant against hardware brute-force
 attacks than alternative functions such as @code{PBKDF2} or @code{bcrypt}.")
     (license license:bsd-2)))
+
+(define-public python-asn1crypto
+  (package
+    (name "python-asn1crypto")
+    (version "0.22.0")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (pypi-uri "asn1crypto" version))
+        (sha256
+         (base32
+          "06pd1bglyisjnjkgc5dc24b498q9r8fmvwkfn0janr9i1mjdvfnb"))))
+    (build-system python-build-system)
+    (arguments
+     '(#:tests? #f)) ; Tests are not distributed via PyPi.
+    (home-page "https://github.com/wbond/asn1crypto")
+    (synopsis "ASN.1 parser and serializer in Python")
+    (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.")
+    (license license:expat)))
+
+(define-public python2-asn1crypto
+  (package-with-python2 python-asn1crypto))
-- 
2.13.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* bug#27043: [PATCH] gnu: Add python-asn1crypto.
  2017-05-23 19:42 bug#27043: [PATCH] gnu: Add python-asn1crypto Leo Famulari
@ 2017-05-24  7:54 ` Arun Isaac
  2017-05-24 16:13   ` Marius Bakke
       [not found] ` <c493b2f5.AEEAK7CSRGoAAAAAAAAAAAO8ccgAAAACwQwAAAAAAAW9WABZJTxG@mailjet.com>
  1 sibling, 1 reply; 4+ messages in thread
From: Arun Isaac @ 2017-05-24  7:54 UTC (permalink / raw)
  To: Leo Famulari; +Cc: 27043


Thanks for the patch! :-)

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.

> +(define-public python-asn1crypto
> +  (package
> +    (name "python-asn1crypto")
> +    (version "0.22.0")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (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?

> +    (home-page "https://github.com/wbond/asn1crypto")
> +    (synopsis "ASN.1 parser and serializer in Python")
> +    (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.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#27043: [PATCH] gnu: Add python-asn1crypto.
  2017-05-24  7:54 ` Arun Isaac
@ 2017-05-24 16:13   ` Marius Bakke
  0 siblings, 0 replies; 4+ messages in thread
From: Marius Bakke @ 2017-05-24 16:13 UTC (permalink / raw)
  To: Arun Isaac, Leo Famulari; +Cc: 27043

[-- Attachment #1: Type: text/plain, Size: 411 bytes --]

Arun Isaac <arunisaac@systemreboot.net> writes:

> 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.

I think crypto.scm is fine. "python.scm" is already rather bloated, so
it's good to add new libraries in topic-specific modules.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#27043: [PATCH] gnu: Add python-asn1crypto.
       [not found] ` <c493b2f5.AEEAK7CSRGoAAAAAAAAAAAO8ccgAAAACwQwAAAAAAAW9WABZJTxG@mailjet.com>
@ 2017-05-25 19:44   ` Leo Famulari
  0 siblings, 0 replies; 4+ messages in thread
From: Leo Famulari @ 2017-05-25 19:44 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 27043-done

[-- Attachment #1: Type: text/plain, Size: 1331 bytes --]

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-05-25 19:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-23 19:42 bug#27043: [PATCH] gnu: Add python-asn1crypto Leo Famulari
2017-05-24  7:54 ` Arun Isaac
2017-05-24 16:13   ` Marius Bakke
     [not found] ` <c493b2f5.AEEAK7CSRGoAAAAAAAAAAAO8ccgAAAACwQwAAAAAAAW9WABZJTxG@mailjet.com>
2017-05-25 19:44   ` Leo Famulari

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.