GNU bug report logs -
#48427
[PATCH 3/5] gnu: Add python-aiostream.
Previous Next
Reported by: Stefan Reichör <stefan <at> xsteve.at>
Date: Fri, 14 May 2021 21:19:02 UTC
Severity: normal
Tags: patch
Done: Jelle Licht <jlicht <at> fsfe.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 48427 in the body.
You can then email your comments to 48427 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#48427
; Package
guix-patches
.
(Fri, 14 May 2021 21:19:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Stefan Reichör <stefan <at> xsteve.at>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Fri, 14 May 2021 21:19:03 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/python-xyz.scm (python-aiostream): New variable.
---
gnu/packages/python-xyz.scm | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 98368129ed..8e769c08f2 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -20187,6 +20187,36 @@ generators and Python 3.7's context managers into Python 3.5.")
manager compatible with @code{asyncio}.")
(license license:asl2.0)))
+(define-public python-aiostream
+ (package
+ (name "python-aiostream")
+ (version "0.4.3")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (pypi-uri "aiostream" version))
+ (sha256
+ (base32
+ "1k33bxrp24pa7c3hygzkv0lqvjil689aj442hzhgzv8vsbqmwd1n"))))
+ (build-system python-build-system)
+ (arguments
+ `(#:tests? #f)) ; tests fail, not sure why
+ ;; (native-inputs
+ ;; `(("python-pytest" ,python-pytest)
+ ;; ("python-pytest-asyncio" ,python-pytest-asyncio)
+ ;; ("python-pytest-cov" ,python-pytest-cov)
+ ;; ("python-pytest-runner" ,python-pytest-runner)))
+ (home-page
+ "https://github.com/vxgmichel/aiostream")
+ (synopsis
+ "Generator-based operators for asynchronous iteration")
+ (description
+ "aiostream provides a collection of stream operators that can be
+combined to create asynchronous pipelines of operations. It can be seen as an
+asynchronous version of itertools, although some aspects are slightly
+different.")
+ (license license:gpl3+)))
+
(define-public python-glob2
(package
(name "python-glob2")
--
2.25.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#48427
; Package
guix-patches
.
(Tue, 18 May 2021 22:32:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 48427 <at> debbugs.gnu.org (full text, mbox):
Hi,
Em sex, 2021-05-14 às 23:17 +0200, Stefan Reichör escreveu:
> * gnu/packages/python-xyz.scm (python-aiostream): New variable.
Thank you.
>
> +(define-public python-aiostream
> + (package
> + (name "python-aiostream")
> + (version "0.4.3")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (pypi-uri "aiostream" version))
> + (sha256
> + (base32
> +
> "1k33bxrp24pa7c3hygzkv0lqvjil689aj442hzhgzv8vsbqmwd1n"))))
> + (build-system python-build-system)
> + (arguments
> + `(#:tests? #f)) ; tests fail, not sure why
Tests are not included in the PyPI tarball. That happens often in the
Python ecosystem, and in those cases we fetch directly from the
upstream repository.
After changing the source and adding #:test-target "pytest" to the
arguments (requires python-pytest-runner which is already there), I
could succesfully build the project with tests:
(source
(origin
(method git-fetch)
(uri (git-reference
(url "https://github.com/vxgmichel/aiostream")
(commit (string-append "v" version))))
(file-name (git-file-name name version))
(sha256
(base32
"1r3x9qwl08yscmzvhafc6gsmq84lr17s6p7a1qxr49cmdvjzsc13"))))
(arguments
`(#:test-target "pytest"))
Note that python-build-system will run "python setup.py test" which is
used for unittest tests. If the project uses Pytest, as many do, we
often have to be explicit in the build either by adding #:test-target
"pytest" to the arguments and python-pytest-runner to native-inputs, or
by replacing the check phase and invoking pytest manually. Since
pytest-runner was deprecated recently, I believe the latter is the
better option from now on. The build could succeed without being
explicit, but you will see a message like "0 tests ran succesfully".
That may have happened in patches 2 and 5 from the series.
> + (home-page
> + "https://github.com/vxgmichel/aiostream")
> + (synopsis
> + "Generator-based operators for asynchronous iteration")
We don't break lines for the arguments of home-page and synopsis. The
pypi importer will add them by default, but they should be removed
while editing.
In the description, I'd replace "aiostream" with "@code{aiostream}".
The LICENSE file states GPL 3, though there are no copyright headers to
say "any later version". setup.py also says "gplv3". I'm guessing the
license should be gpl3 only, though I'm not sure about that one.
LGTM otherwise.
Could you check the rest of the series to see if the tests are running
correctly and then send a v2? I am not a committer and can't apply it
myself, so I have to leave that part for somebody else.
Vinicius
Information forwarded
to
guix-patches <at> gnu.org
:
bug#48427
; Package
guix-patches
.
(Sat, 29 May 2021 20:39:01 GMT)
Full text and
rfc822 format available.
Message #11 received at 48427 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/python-xyz.scm (python-aiostream): New variable.
---
Vinicius, thanks for your review.
Here is my attempt to follow your hints
gnu/packages/python-xyz.scm | 29 +++++++++++++++++++++++++++++
1 file changed, 29 insertions(+)
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 9b9510f043..a465ecbda6 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -20232,6 +20232,35 @@ generators and Python 3.7's context managers into Python 3.5.")
manager compatible with @code{asyncio}.")
(license license:asl2.0)))
+(define-public python-aiostream
+ (package
+ (name "python-aiostream")
+ (version "0.4.3")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/vxgmichel/aiostream")
+ (commit (string-append "v" version))))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "1r3x9qwl08yscmzvhafc6gsmq84lr17s6p7a1qxr49cmdvjzsc13"))))
+ (build-system python-build-system)
+ (arguments
+ `(#:test-target "pytest"))
+ (native-inputs
+ `(("python-pytest" ,python-pytest)
+ ("python-pytest-asyncio" ,python-pytest-asyncio)
+ ("python-pytest-cov" ,python-pytest-cov)
+ ("python-pytest-runner" ,python-pytest-runner)))
+ (home-page "https://github.com/vxgmichel/aiostream")
+ (synopsis "Generator-based operators for asynchronous iteration")
+ (description "aiostream provides a collection of stream operators that can
+be combined to create asynchronous pipelines of operations. It can be seen as
+an asynchronous version of itertools, although some aspects are slightly
+different.")
+ (license license:gpl3)))
+
(define-public python-glob2
(package
(name "python-glob2")
--
2.25.1
Reply sent
to
Jelle Licht <jlicht <at> fsfe.org>
:
You have taken responsibility.
(Mon, 29 May 2023 11:18:01 GMT)
Full text and
rfc822 format available.
Notification sent
to
Stefan Reichör <stefan <at> xsteve.at>
:
bug acknowledged by developer.
(Mon, 29 May 2023 11:18:02 GMT)
Full text and
rfc822 format available.
Message #16 received at 48427-done <at> debbugs.gnu.org (full text, mbox):
Stefan Reichör <stefan <at> xsteve.at> writes:
> * gnu/packages/python-xyz.scm (python-aiostream): New variable.
[snip]
Superseded by commit ed18b697c4783f139e23731f5bd0b0ed197997bb, closing.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Mon, 26 Jun 2023 11:24:05 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 320 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.