bug#44512: closed ([PATCH] gnu: Add python-telethon.)

[GNU bug Tracking System]

Your message dated Wed, 18 Nov 2020 22:48:38 +0200
with message-id <20201118204838.GB2306@E5400>
and subject line Re: [bug#44512] [PATCH] gnu: Add python-telethon.
has caused the debbugs.gnu.org bug report #44512,
regarding [PATCH] gnu: Add python-telethon.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
44512: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=44512
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] gnu: Add python-telethon. Date: Sun, 8 Nov 2020 10:13:16 +0100 (CET)

0001-gnu-Add-python-telethon.patch
Description: Text Data

--- End Message ---

--- Begin Message --- Subject: Re: [bug#44512] [PATCH] gnu: Add python-telethon. Date: Wed, 18 Nov 2020 22:48:38 +0200

I made a couple of changes before pushing the patch

On Sun, Nov 08, 2020 at 10:13:16AM +0100, Adam Kandur via Guix-patches via 
wrote:
> 
> 

> From 40a037719317f943e4d311e6c03b1398feaffe7b Mon Sep 17 00:00:00 2001
> From: Adam <rndd@tuta.io>
> Date: Sun, 8 Nov 2020 12:10:32 +0300
> Subject: [PATCH] gnu: Add python-telethon.
> 
> * gnu/packages/python-xyz.scm (python-telethon): New variable.
> ---
>  gnu/packages/python-xyz.scm | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
> index 5bbd547..d7dfaf2 100644
> --- a/gnu/packages/python-xyz.scm
> +++ b/gnu/packages/python-xyz.scm
> @@ -22560,6 +22560,29 @@ a Python library to convert RELAX NG schemata in 
> Compact syntax (rnc) to
>  equivalent schemata in the XML-based default RELAX NG syntax.")
>      (license license:expat)))
>  
> +(define-public python-telethon
> +  (package
> +    (name "python-telethon")

We normally put this after the source, so I moved it there.

> +    (build-system python-build-system)
> +    (propagated-inputs
> +     `(("python-rsa" ,python-rsa)
> +       ("python-pyaes" ,python-pyaes)
> +       ("python-pytest" ,python-pytest)))

pytest should be a native-input. I also added python-pytest-asyncio and
python-pytest-trio to native-inputs.

> +    (version "v1.17.5")

the 'v' isn't part of the version string. Instead the 'commit' as part
of the source becomes '(commit (string-apend "v" version))'

> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +          (url "https://github.com/LonamiWebs/Telethon";)
> +          (commit version)))

I see you noticed the pypi release didn't come with tests :)

> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32 "0l9fhdrq576vllgi9aam45xzw5xi6jhgdv5zz8i4ygssdp7cm8jl"))))

I replaced the 'check phase with a phase which runs
(invoke "py.test" "-v")

> +    (home-page "https://docs.telethon.dev";)
> +    (synopsis "Telethon is an asyncio Python 3 MTProto library to interact 
> with Telegram's API as a user or through a bot account (bot API 
> alternative).")
> +    (description "Telegram is a popular messaging application. This library 
> is meant to make it easy for you to write Python programs that can interact 
> with Telegram. Think of it as a wrapper that has already done the heavy job 
> for you, so you can focus on developing an application.")
> +    (license license:expat)))

I wasn't a fan of the synopsis or description, so I took the synopsis
which came from 'guix import pypi pypi telethon' and I shortened the
description to mostly just include that it interacts with Telegram.

> +
>  (define-public python-citeproc-py
>    (package
>      (name "python-citeproc-py")
> -- 
> 2.28.0
> 

Thanks for the patch! Pushed.


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

signature.asc
Description: PGP signature

--- End Message ---