diff options
author | Leo Famulari <leo@famulari.name> | 2017-04-08 12:45:16 -0400 |
---|---|---|
committer | Leo Famulari <leo@famulari.name> | 2017-04-08 13:24:36 -0400 |
commit | f26d6e4e9c5efbe060697f54456cfc9144350114 (patch) | |
tree | 6a7ab27710639faf0b4e60d638f1bdc68f0b5b72 | |
parent | f0e9fda6beaf3c44993b2cc7264fa05ad887b0a8 (diff) | |
download | guix-patches-f26d6e4e9c5efbe060697f54456cfc9144350114.tar guix-patches-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.
-rw-r--r-- | gnu/packages/tls.scm | 39 |
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) |