Re: [PATCH 0/1] Split acme into both Python variants

[Leo Famulari]

On Wed, Dec 30, 2015 at 04:45:04PM +0100, Ludovic Courtès wrote:
> Leo Famulari <address@hidden> skribis:
> 
> > This patch splits the acme library into both Python variants. Currently,
> > its only user in Guix is the Python 2 official Let's Encrypt client, but
> > there is other software out there that supports Python 3.
> 
> Sounds reasonable.
> 
> > I tried and tried to make the python2-acme version inherit more from
> > python-acme but this is what I got working.
> >
> > I did try to apply the solution used in python2-pyopenssl and
> > python2-oauthlib [0] but I couldn't make it work. I think the issue is
> > that python2-acme uses both python2-pyopenssl and python2-cryptography,
> > so there are multiple layers of translation to watch out for.
> >
> > Suggestions welcome!
> 
> What did you try exactly?  The workaround is to explicitly list
> dependencies instead of using those computed by ‘package-with-python2’.

Despite trying the variations below, the build process kept failing as
described by Efraim in bug#22013 [0].

Here are the variations that I tested, based on reading python.scm:

(define-public python2-acme
  (let ((acme (package-with-python2 python-acme)))
    (package (inherit acme)
      (propagated-inputs
       `(("python2-cryptography" ,python2-cryptography)
         ,@(alist-delete "python-cryptography"
                         (package-propagated-inputs acme)))))))

(define-public python2-acme
  (let ((acme (package-with-python2 python-acme)))
    (package (inherit acme)
      (propagated-inputs
       `(("python2-pyopenssl" ,python2-pyopenssl)
         ("python2-cryptography" ,python2-cryptography)
         ,@(fold alist-delete (package-propagated-inputs acme)
                 '("python-pyopenssl" "python-cryptography")))))))

;; I don't understand this one
(define-public python2-acme
  (let ((acme (package-with-python2 python-acme)))
    (package (inherit acme)
      (propagated-inputs
       `(("python2-cryptography" ,python2-cryptography)
         ("python2-pyopenssl" ,python2-pyopenssl)
         ,@(alist-delete "python-pyopenssl"
                         (alist-delete "python-cryptography"
                                       (package-propagated-inputs acme))))))))

[0]
http://lists.gnu.org/archive/html/bug-guix/2015-11/msg00124.html

> 
> HTH,
> Ludo'.