* [bug#72935] [PATCH] gnu: python-mpv: Update to 1.0.7.
@ 2024-09-01 15:46 Ian Eure
2024-09-01 17:05 ` bug#72935: " Tobias Geerinckx-Rice via Guix-patches via
0 siblings, 1 reply; 2+ messages in thread
From: Ian Eure @ 2024-09-01 15:46 UTC (permalink / raw)
To: 72935; +Cc: Ian Eure
* gnu/packages/python-xyz.scm (python-mpv): Update to 1.0.7. The recent
core-updates merge broke the previous version of this package, so I updated to
the most recent upstream while fixing the build. Upstream switched to
pyproject, so use that build-system and add packages the tests need. Upstream
added an environment variable to skip the test the previous package patched
out, so use that rather than changing the source.
Change-Id: Ia0e6c3341292fe6a40f63b484d70c6a68b6f7eb4
---
gnu/packages/python-xyz.scm | 72 +++++++++++++++++--------------------
1 file changed, 33 insertions(+), 39 deletions(-)
diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index a7688050e1..10908f185f 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -37071,7 +37071,7 @@ (define-public python-bsdiff4
(define-public python-mpv
(package
(name "python-mpv")
- (version "1.0.1")
+ (version "1.0.7")
(source
(origin
;; python-mpv from pypi does not include the tests directory.
@@ -37081,43 +37081,37 @@ (define-public python-mpv
(commit (string-append "v" version))))
(file-name (git-file-name name version))
(sha256
- (base32
- "10w6j3n62ap45sf6q487kz8z6g58sha37i14fa2hhng794z7a8jh"))
- (modules '((guix build utils)))
- (snippet
- #~(begin
- ;; One of the tests never completes, so neutering it using
- ;; early return allows other test to run without issue.
- (substitute* "tests/test_mpv.py"
- ;; Note the typo in "prooperty" - this was fixed later in
- ;; upstream but has no effect on whether the tests hangs or not.
- (("test_wait_for_prooperty_event_overflow.*" line)
- ;; The long whitespace between \n and return is to match the
- ;; identation level, which is significant in python.
- (string-append line "\n return\n")))))))
- (build-system python-build-system)
+ (base32 "102fajzrcgxapsanh0phlqmk9q2v95bvix6mrkg8rypv717idins"))
+ (modules '((guix build utils)))))
+ (build-system pyproject-build-system)
(arguments
- (list #:phases
- #~(modify-phases %standard-phases
- (add-before 'build 'patch-reference-to-mpv
- (lambda* (#:key inputs #:allow-other-keys)
- ;; Without an absolute path it is not able find and
- ;; load the libmpv library.
- (substitute* "mpv.py"
- (("sofile = .*")
- (string-append "sofile = \""
- (search-input-file inputs "/lib/libmpv.so")
- "\"\n")))))
- (add-before 'check 'prepare-for-tests
- (lambda _
- ;; Fontconfig throws errors when it has no cache dir to use.
- (setenv "XDG_CACHE_HOME" (getcwd))
- ;; Some tests fail without a writable and readable HOME.
- (setenv "HOME" (getcwd)))))))
- (native-inputs
- (list python-xvfbwrapper)) ; needed for tests only
+ (list
+ #:phases #~(modify-phases %standard-phases
+ (add-before 'build 'patch-reference-to-mpv
+ (lambda* (#:key inputs #:allow-other-keys)
+ ;; Without an absolute path it is not able find and
+ ;; load the libmpv library.
+ (substitute* "mpv.py"
+ (("sofile = .*")
+ (string-append "sofile = \""
+ (search-input-file inputs
+ "/lib/libmpv.so")
+ "\"\n")))))
+ (add-before 'check 'prepare-for-tests
+ (lambda _
+ ;; Fontconfig throws errors when it has no cache dir to use.
+ (setenv "XDG_CACHE_HOME"
+ (getcwd))
+ ;; Some tests fail without a writable and readable HOME.
+ (setenv "HOME"
+ (getcwd))
+ (setenv "PY_MPV_SKIP_TESTS"
+ "test_wait_for_property_event_overflow"))))))
+ (native-inputs (list
+ ;; needed for tests
+ python-pyvirtualdisplay python-xvfbwrapper python-pytest))
(inputs (list mpv))
- (propagated-inputs (list python-pillow)) ; for raw screenshots
+ (propagated-inputs (list python-pillow)) ;for raw screenshots
(home-page "https://github.com/jaseg/python-mpv")
(synopsis "Python interface to the mpv media player")
(description
@@ -37125,9 +37119,9 @@ (define-public python-mpv
It gives you more or less full control of all features of the player, just
as the lua interface does.")
;; From the project's README:
- ;; python-mpv inherits the underlying libmpv's license, which can be either
- ;; GPLv2 or later (default) or LGPLv2.1 or later. For details, see the mpv
- ;; copyright page.
+ ;; python-mpv inherits the underlying libmpv's license, which can be either
+ ;; GPLv2 or later (default) or LGPLv2.1 or later. For details, see the mpv
+ ;; copyright page.
(license license:gpl2+)))
(define-public python-biblib
--
2.45.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* bug#72935: [PATCH] gnu: python-mpv: Update to 1.0.7.
2024-09-01 15:46 [bug#72935] [PATCH] gnu: python-mpv: Update to 1.0.7 Ian Eure
@ 2024-09-01 17:05 ` Tobias Geerinckx-Rice via Guix-patches via
0 siblings, 0 replies; 2+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2024-09-01 17:05 UTC (permalink / raw)
To: Ian Eure; +Cc: 72935-done
[-- Attachment #1: Type: text/plain, Size: 2010 bytes --]
Hi Ian,
Ian Eure 写道:
> * gnu/packages/python-xyz.scm (python-mpv): Update to 1.0.7.
> The recent
> core-updates merge broke the previous version of this package,
> so I updated to
> the most recent upstream while fixing the build. Upstream
> switched to
> pyproject, so use that build-system and add packages the tests
> need. Upstream
> added an environment variable to skip the test the previous
> package patched
> out, so use that rather than changing the source.
Thanks! I rewrote this to follow our more formal change log
standards:
gnu: python-mpv: Update to 1.0.7.
This fixes the build after the core-updates merge.
* gnu/packages/python-xyz.scm (python-mpv): Update to 1.0.7.
[source]: Remove snippet obsoleted by PY_MPV_SKIP_TESTS.
[build-system]: Switch to pyproject.
[arguments]: Set newly-available PY_MPV_SKIP_TESTS variable.
[native-inputs]: Add python-pytest and
python-pyvirtualdisplay.
> + (modules '((guix build utils)))))
Origin [modules] are only ever used for snippets, so with the
snippet gone this served no purpose. Removed.
> (arguments
> - (list #:phases
> - #~(modify-phases %standard-phases
> + (list
> + #:phases #~(modify-phases %standard-phases
After our chat on IRC, I reverted this dubious ‘guix style’
indentation change. This makes your diff a breeze to read and
shortens some lines that had grown over 80 characters.
As things stand, I don't recommend running ‘guix style’ on
existing code. Maybe one day.
> + (native-inputs (list
> + ;; needed for tests
> + python-pyvirtualdisplay python-xvfbwrapper
> python-pytest))
I added the newlines that we both preferred.
Line comments (;;) are full sentences, so
;; Needed for tests.
Pushed, minus some other ‘guix style’ noise, as
a977900d76b7585c401222a3ce8bbb82e5f6da45.
Kind regards,
T G-R
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-09-01 17:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-01 15:46 [bug#72935] [PATCH] gnu: python-mpv: Update to 1.0.7 Ian Eure
2024-09-01 17:05 ` bug#72935: " Tobias Geerinckx-Rice via Guix-patches via
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.