summary refs log tree commit diff
path: root/gnu/packages
diff options
context:
space:
mode:
authorLeo Famulari <leo@famulari.name>2017-04-08 12:45:16 -0400
committerLeo Famulari <leo@famulari.name>2017-04-08 13:24:36 -0400
commitf26d6e4e9c5efbe060697f54456cfc9144350114 (patch)
tree6a7ab27710639faf0b4e60d638f1bdc68f0b5b72 /gnu/packages
parentf0e9fda6beaf3c44993b2cc7264fa05ad887b0a8 (diff)
downloadguix-f26d6e4e9c5efbe060697f54456cfc9144350114.tar.gz
gnu: certbot: Share python-acme's arguments.
* gnu/packages/tls.scm (certbot)[arguments]: Use substitute-keyword-arguments to
inherit from python-acme.
Diffstat (limited to 'gnu/packages')
-rw-r--r--gnu/packages/tls.scm39
1 files changed, 15 insertions, 24 deletions
diff --git a/gnu/packages/tls.scm b/gnu/packages/tls.scm
index ac5e406c7f..08cc6075b6 100644
--- a/gnu/packages/tls.scm
+++ b/gnu/packages/tls.scm
@@ -478,8 +478,6 @@ security, and applying best practice development processes.")
         "05cqadwzgfcianw3v0qxwja65dxnzw429f7dk8w0mnh21pib72bl"))))
     (build-system python-build-system)
 
-    ;; TODO factorize the 'docs' phase and share arguments between python-acme
-    ;; and certbot.
     (arguments
      `(#:phases
        (modify-phases %standard-phases
@@ -541,28 +539,21 @@ security, and applying best practice development processes.")
     (build-system python-build-system)
     (arguments
      `(#:python ,python-2
-       #:phases
-       (modify-phases %standard-phases
-         (add-after 'unpack 'patch-dependency
-           ;; This module is part of the Python standard library, so we don't
-           ;; need to use an external package.
-           ;; https://github.com/certbot/certbot/pull/2249
-           (lambda _
-             (substitute* "setup.py"
-               (("'argparse',") ""))
-             #t))
-         (add-after 'build 'docs
-           (lambda* (#:key outputs #:allow-other-keys)
-             (let* ((out (assoc-ref outputs "out"))
-                    (man1 (string-append out "/share/man/man1"))
-                    (man7 (string-append out "/share/man/man7"))
-                    (info (string-append out "/info")))
-               (and
-                 (zero? (system* "make" "-C" "docs" "man" "info"))
-                 (install-file "docs/_build/texinfo/Certbot.info" info)
-                 (install-file "docs/_build/man/certbot.1" man1)
-                 (install-file "docs/_build/man/certbot.7" man7)
-                 #t)))))))
+       ,@(substitute-keyword-arguments (package-arguments python-acme)
+           ((#:phases phases)
+            `(modify-phases ,phases
+              (replace 'docs
+                (lambda* (#:key outputs #:allow-other-keys)
+                  (let* ((out (assoc-ref outputs "out"))
+                         (man1 (string-append out "/share/man/man1"))
+                         (man7 (string-append out "/share/man/man7"))
+                         (info (string-append out "/info")))
+                    (and
+                      (zero? (system* "make" "-C" "docs" "man" "info"))
+                      (install-file "docs/_build/texinfo/Certbot.info" info)
+                      (install-file "docs/_build/man/certbot.1" man1)
+                      (install-file "docs/_build/man/certbot.7" man7)
+                      #t)))))))))
     ;; TODO: Add optional inputs for testing.
     (native-inputs
      `(("python2-nose" ,python2-nose)