Re: [PATCH 1/2] gnu: Add tremc.

[Marius Bakke <mbakke@fastmail.com>]

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

Leo Famulari <leo@famulari.name> writes:

> * gnu/packages/bittorrent.scm (tremc): New variable.
> ---
>  gnu/packages/bittorrent.scm | 45 +++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 45 insertions(+)
>
> diff --git a/gnu/packages/bittorrent.scm b/gnu/packages/bittorrent.scm
> index 716d8b766..a7294f38a 100644
> --- a/gnu/packages/bittorrent.scm
> +++ b/gnu/packages/bittorrent.scm
> @@ -2,6 +2,7 @@
>  ;;; Copyright © 2014 Taylan Ulrich Bayirli/Kammer <taylanbayirli@gmail.com>
>  ;;; Copyright © 2014, 2015, 2016 Ludovic Courtès <ludo@gnu.org>
>  ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
> +;;; Copyright © 2016, 2017 Leo Famulari <leo@famulari.name>
>  ;;; Copyright © 2016, 2017 Efraim Flashner <efraim@flashner.co.il>
>  ;;; Copyright © 2016 Tomáš Čech <sleep_walker@gnu.org>
>  ;;; Copyright © 2016 Tobias Geerinckx-Rice <me@tobias.gr>
> @@ -25,6 +26,7 @@
>  (define-module (gnu packages bittorrent)
>    #:use-module (guix packages)
>    #:use-module (guix download)
> +  #:use-module (guix git-download)
>    #:use-module (guix build-system gnu)
>    #:use-module (guix build-system python)
>    #:use-module (guix build-system glib-or-gtk)
> @@ -172,6 +174,49 @@ XML-RPC over SCGI.")
>      (home-page "https://github.com/rakshasa/rtorrent")
>      (license l:gpl2+)))
>  
> +(define-public tremc
> +  (let ((commit "401f2303c9b5a6e2e7b0808617d794576d4aa29e")
> +        (revision "0"))
> +    (package
> +      (name "tremc")
> +      (version (string-append "0.0.0-" revision "." (string-take commit 7)))

The script has a VERSION variable specifying 0.9.0, so I think we should
use that. It would of course be better if they provided tagged releases..

> +      (source
> +        (origin
> +          (method git-fetch)
> +          (uri (git-reference
> +                 (url "https://github.com/louipc/tremc.git")
> +                 (commit commit)))
> +          (sha256
> +           (base32
> +            "1h2720zn35iggmf9av65g119b0bhskwm1ng0zbkjryaf38nfzpin"))))
> +      (build-system python-build-system)
> +      (arguments
> +       `(#:tests? #f ; no test suite
> +         #:phases
> +         (modify-phases %standard-phases
> +           ;; The software is just a Python script that must be
> +           ;; copied into place.
> +           (delete 'build)
> +           (replace 'install
> +             (lambda* (#:key outputs #:allow-other-keys)
> +               (let* ((out (assoc-ref outputs "out"))
> +                      (bin (string-append out "/bin"))
> +                      (man (string-append out "/share/man/man1"))
> +                      ;; FIXME install zsh completions
> +                      (completions (string-append out "/etc/bash_completion.d")))
> +                 (install-file "tremc" bin)
> +                 (install-file "tremc.1" man)
> +                 (install-file
> +                   (string-append
> +                     "completion/bash/"
> +                     "transmission-remote-cli-bash-completion.sh")
> +                   completions)))))))
> +      (synopsis "Console client for the Transmission BitTorrent daemon")
> +      (description "Tremc is a console client, with a curses interface, for the
> +Transmission BitTorrent daemon.")

So all the imports in the script are part of the python standard
library, including "curses"? Neat. LGTM!

> +      (home-page "https://github.com/louipc/tremc")
> +      (license l:gpl3+))))
> +
>  (define-public transmission-remote-cli
>    (package
>      (name "transmission-remote-cli")
> -- 
> 2.11.0

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