[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#47593] [PATCH 5/6] gnu: python-xyz: Add python-quicktions.
From: |
Bonface Munyoki K . |
Subject: |
[bug#47593] [PATCH 5/6] gnu: python-xyz: Add python-quicktions. |
Date: |
Wed, 14 Apr 2021 00:10:31 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
Efraim Flashner <efraim@flashner.co.il> writes:
> On Sun, Apr 04, 2021 at 03:20:04PM +0300, BonfaceKilz wrote:
>> * gnu/packages/python-xyz.scm (python-quicktions): New variable.
>>
>> Co-authored-by: jgart <jgart@dismail.de>
>> ---
[..]
>> + (synopsis
>> + "Fast fractions data type for rational numbers. Cythonized version of
>> 'fractions.Fraction'.")
>
> I'd trim this to 'Fast fractions data type for rational numbers'
>
Noted.
>> + (description
>> + "Fast fractions data type for rational
>> numbers. Cythonized version of
>> 'fractions.Fraction'.")
>
> And this I'd change to 'This package provides fast fractions data type
> for rational numbers. It is the cythonized version of
> @quote{fractions.Fraction}.'
>
Makes sense. What's the "@quote{}" syntax. Where
would I get more of the documentation of the
syntax to put in the description?
>> + (license license:psfl)))
>> --
>> 2.31.0
>>
>
> Speaking of cython, you should add a snippet to the source to remove any
> cythionized source and make sure those files get rebuilt during the
> build phase. Looking at setup.py it looks like you should remove
> src/quictions.c and I think it tries to cythonize automatically.
For anyone reading this(in future), the cythonized
.pyx files can stop matching CPython APIs across
Python versions. An example of that here:
https://github.com/slaypni/fastdtw/pull/40
--
Bonface M. K. D4F09EB110177E03C28E2FE1F5BBAE1E0392253F
Humble GNU Emacs User / Bearer of scheme-y parens
Curator: <https://upbookclub.com> / Twitter: @BonfaceKilz
signature.asc
Description: PGP signature
[bug#47593] [PATCH 4/6] gnu: python-xyz: Add python-setuptools-declarative-requirements., BonfaceKilz, 2021/04/04
[bug#47593] [PATCH 1/6] gnu: python-xyz: Add python-roman., BonfaceKilz, 2021/04/04