GNU bug report logs -
#58072
[PATCH] gnu: Add python-textual.
Previous Next
Reported by: Brian Kubisiak <brian <at> kubisiak.com>
Date: Sun, 25 Sep 2022 14:01:02 UTC
Severity: normal
Tags: patch
Done: Ludovic Courtès <ludo <at> gnu.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 58072 in the body.
You can then email your comments to 58072 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#58072
; Package
guix-patches
.
(Sun, 25 Sep 2022 14:01:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Brian Kubisiak <brian <at> kubisiak.com>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Sun, 25 Sep 2022 14:01:02 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-textual): New variable.
---
gnu/packages/python-xyz.scm | 44 +++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index ffcb023d60..5fd7dad8e3 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -15973,6 +15973,50 @@ (define-public python-rich
syntax highlighting, markdown and more to the terminal.")
(license license:expat)))
+(define-public python-textual
+ (package
+ (name "python-textual")
+ (version "0.1.18")
+ (source (origin
+ (method url-fetch)
+ (uri (pypi-uri "textual" version))
+ (sha256
+ (base32
+ "08yg5a51hz1axfj5hx28hx31gq5apcj6vpkkmawmiplisa73z25j"))))
+ (build-system python-build-system)
+ (arguments
+ (list #:phases
+ #~(modify-phases %standard-phases
+ (replace 'check
+ (lambda* (#:key inputs tests? #:allow-other-keys)
+ (when tests?
+ (copy-recursively (string-append
+ (assoc-ref inputs "tests") "/tests")
+ "tests")
+ (invoke "python" "-m" "pytest" "-vv")))))))
+ (propagated-inputs
+ (list python-rich python-typing-extensions))
+ (native-inputs
+ `(("python-pytest" ,python-pytest)
+ ("tests"
+ ;; The release on pypi comes without tests. We can't build from this
+ ;; checkout, though, because installation requires an invocation of
+ ;; poetry.
+ ,(origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/Textualize/textual")
+ (commit (string-append "v" version))))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+ "0b3ycwqhp21mg9fvmadgxhgbvkwq6fd784l2xcmvy77rravrnnax"))))))
+ (home-page "https://github.com/Textualize/textual")
+ (synopsis "Build TUIs in python")
+ (description "Textual is a TUI (Text User Interface) framework for Python
+inspired by modern web development.")
+ (license license:expat)))
+
(define-public python-magic
(package
(name "python-magic")
--
2.37.3
Information forwarded
to
guix-patches <at> gnu.org
:
bug#58072
; Package
guix-patches
.
(Tue, 27 Sep 2022 21:10:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 58072 <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Hi Brian,
On Sun, 25 Sep 2022 05:59:24 -0700
Brian Kubisiak <brian <at> kubisiak.com> wrote:
> * gnu/packages/python-xyz.scm (python-textual): New variable.
> ---
> gnu/packages/python-xyz.scm | 44
[..]
> + (source (origin
> + (method url-fetch)
> + (uri (pypi-uri "textual" version))
> + (sha256
> + (base32
> +
> "08yg5a51hz1axfj5hx28hx31gq5apcj6vpkkmawmiplisa73z25j"))))
> + (build-system python-build-system)
> + (arguments
> + (list #:phases
> + #~(modify-phases %standard-phases
> + (replace 'check
> + (lambda* (#:key inputs tests? #:allow-other-keys)
> + (when tests?
> + (copy-recursively (string-append
> + (assoc-ref inputs "tests")
> "/tests")
> + "tests")
> + (invoke "python" "-m" "pytest" "-vv")))))))
> + (propagated-inputs
> + (list python-rich python-typing-extensions))
> + (native-inputs
> + `(("python-pytest" ,python-pytest)
> + ("tests"
> + ;; The release on pypi comes without tests. We can't build
> from this
> + ;; checkout, though, because installation requires an
> invocation of
> + ;; poetry.
> + ,(origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/Textualize/textual")
> + (commit (string-append "v" version))))
> + (file-name (git-file-name name version))
> + (sha256
> + (base32
> +
> "0b3ycwqhp21mg9fvmadgxhgbvkwq6fd784l2xcmvy77rravrnnax"))))))
You download the sources twice: Once from pypi, and another time from
GitHub as "test" input. I would suggest you use directly the GitHub
sources as main sources. Then you also don't need the modified 'check
phase.
As an example, I found python-identify, which also comments on the
sources that there are no tests in PyPi (please add that comment too,
it makes it clear why not to use PyPi).
Would you like to send an updated patch?
Thanks,
Björn
[Message part 2 (application/pgp-signature, inline)]
Information forwarded
to
guix-patches <at> gnu.org
:
bug#58072
; Package
guix-patches
.
(Wed, 28 Sep 2022 12:31:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 58072 <at> debbugs.gnu.org (full text, mbox):
> You download the sources twice: Once from pypi, and another time from
> GitHub as "test" input. I would suggest you use directly the GitHub
> sources as main sources. Then you also don't need the modified 'check
> phase.
The GitHub sources are built with poetry and do not have setup.py or
setup.cfg; in order to use the existing python-build-system, the
package must be built from the PyPi sources. The other option would be
to disable the tests completely instead of downloading them from
GitHub, but I think keeping the tests is a better option.
> As an example, I found python-identify, which also comments on the
> sources that there are no tests in PyPi
The patch I wrote is based on python-rich (which is a similar package
by the same author).
> (please add that comment too, it makes it clear why not to use
> PyPi).
I've already got that comment here:
> + ;; The release on pypi comes without tests. We can't build from this
> + ;; checkout, though, because installation requires an invocation of
> + ;; poetry.
Thanks,
Brian
Reply sent
to
Ludovic Courtès <ludo <at> gnu.org>
:
You have taken responsibility.
(Thu, 06 Oct 2022 20:43:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Brian Kubisiak <brian <at> kubisiak.com>
:
bug acknowledged by developer.
(Thu, 06 Oct 2022 20:43:02 GMT)
Full text and
rfc822 format available.
Message #16 received at 58072-done <at> debbugs.gnu.org (full text, mbox):
Hi,
Brian Kubisiak <brian <at> kubisiak.com> skribis:
> * gnu/packages/python-xyz.scm (python-textual): New variable.
Applied. Thank you and thanks Björn for taking a look!
I followed up with a change to remove input labels.
Ludo’.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Fri, 04 Nov 2022 11:24:11 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 174 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.