GNU bug report logs - #51587
[PATCH 0/3] Add rss2email.

Previous Next

Full log

Message #20 received at 51587 <at> debbugs.gnu.org (full text, mbox):

From: Vinicius Monego <monego <at> posteo.net>
To: Morgan.J.Smith <at> outlook.com, 51587 <at> debbugs.gnu.org
Subject: Re: [PATCH 3/3] gnu: Add rss2email.
Date: Fri, 05 Nov 2021 15:55:43 +0000

Em qua, 2021-11-03 às 12:37 -0400, Morgan.J.Smith <at> outlook.com escreveu:
> From: Morgan Smith <Morgan.J.Smith <at> outlook.com>
> 
> * gnu/packages/mail.scm (rss2email): New variable.
> ---
>  gnu/packages/mail.scm | 38 ++++++++++++++++++++++++++++++++++++++

Please add your copyright line in this file.

>  1 file changed, 38 insertions(+)
> 
> diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
> index c5d8cf04ff..04d27d0337 100644
> --- a/gnu/packages/mail.scm
> +++ b/gnu/packages/mail.scm
> @@ -4614,3 +4614,41 @@ (define-public crm114
>  Bigrams, WINNOW, Correlation, KNN/Hyperspace, or Bit Entropy (or by
> other
>  means--it's all programmable).")
>      (license license:gpl3)))
> +
> +(define-public rss2email
> +  (package
> +    (name "rss2email")
> +    (version "3.13.1")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/rss2email/rss2email")
> +                    (commit (string-append "v" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +               
> "0g1yr3v3ibdh2jqil64fbdbplx5m2yzxr893fqfkwcc5c7fbwl4d"))))
> +    (build-system python-build-system)
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (replace 'check
> +           (lambda* (#:key tests? inputs outputs #:allow-other-keys)

Inputs and outputs are not being used in the body.

> +             (when tests?
> +               (with-directory-excursion "test"
> +                 ;; Skip networking tests
> +                 (substitute* "test.py"
> +                   (("( *)class (:?TestSend|TestFetch).*" match
> indent)
> +                    (string-append indent
> "@unittest.skip(\"Networking stuff skipped\")\n"
> +                                   indent match)))
> +                 (invoke "python" "-m" "unittest"))))))))
> +    (propagated-inputs
> +     `(("python-feedparser" ,python-feedparser)
> +       ("python-html2text" ,python-html2text)))

Is this package used only as executable? Inputs can probably be normal
inputs here.

> +    (home-page "https://github.com/rss2email/rss2email")
> +    (synopsis "Converts RSS/Atom newsfeeds to email")
> +    (description "When run it fetches RSS/Atom newsfeeds, converts
> them into an email, and
> +emails them.")

Full sentence.

> +    ;; GPL version 2 or 3. NOT 2+.

Comments should also follow double spaces when starting a new sentence
in the same line.

> +    (license (list license:gpl2
> +                   license:gpl3))))

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.