unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#60171] Fix mpv on X11
@ 2022-12-18  0:21 phodina via Guix-patches via
  2022-12-18 12:17 ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: phodina via Guix-patches via @ 2022-12-18  0:21 UTC (permalink / raw)
  To: 60171; +Cc: Ludovic Courtès, Tobias Geerinckx-Rice, andreas@enge.fr


[-- Attachment #1.1: Type: text/plain, Size: 594 bytes --]

Hi,

for some reason the mpv player is broken on X11 and does not play movies. Enabling support for X11 solves the problem.

$ mpv movie.mkv
(+) Video --vid=1 (*) (h264 1280x544 23.976fps)
(+) Audio --aid=1 (*) (aac 2ch 48000Hz)
[vo/gpu] VT_GETMODE failed: Inappropriate ioctl for device
[vo/gpu/opengl] Failed to set up VT switcher. Terminal switching will be unavailable.
[vo/gpu] Failed to commit ModeSetting atomic request (-13)
[vo/gpu/opengl] Failed to set CRTC for connector 95: Permission denied
Error opening/initializing the selected video_out (--vo) device.Video: no video

----
Petr

[-- Attachment #1.2: Type: text/html, Size: 969 bytes --]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-mpv-Enable-x11-video.patch --]
[-- Type: text/x-patch; name=0001-gnu-mpv-Enable-x11-video.patch, Size: 718 bytes --]

From 80111b265152cec985adf862fc077e81c0b2b931 Mon Sep 17 00:00:00 2001
From: Petr Hodina <phodina@protonmail.com>
Date: Sun, 18 Dec 2022 01:01:17 +0100
Subject: [PATCH] gnu: mpv: Enable x11 video.

* gnu/packages/video.scm (mpv)[arguments]: Enable x11 video in configure-flags.

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index a15c86ee07..1cbe07f1bf 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -2214,6 +2214,7 @@ (define-public mpv
       #~(list "--enable-libmpv-shared"
               "--enable-cdda"
               "--enable-dvdnav"
+              "--enable-x11"
               "--disable-build-date")
       ;; No check function defined.
       #:tests? #f))
-- 
2.38.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#60171] Fix mpv on X11
  2022-12-18  0:21 [bug#60171] Fix mpv on X11 phodina via Guix-patches via
@ 2022-12-18 12:17 ` Tobias Geerinckx-Rice via Guix-patches via
  2022-12-19  5:29   ` phodina via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2022-12-18 12:17 UTC (permalink / raw)
  To: phodina; +Cc: 60171

[-- Attachment #1: Type: text/plain, Size: 439 bytes --]

Hi!

phodina 写道:
> Enabling support for X11 solves the problem.

Are you sure your Guix & mpv installation are up to date?

See

  https://issues.guix.gnu.org/59508 (patch)
  https://issues.guix.gnu.org/59279

and specifically

  https://issues.guix.gnu.org/59378#2 and linked
  https://issues.guix.gnu.org/59339

which mentions your exact error message as being fixed back in 
November.

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#60171] Fix mpv on X11
  2022-12-18 12:17 ` Tobias Geerinckx-Rice via Guix-patches via
@ 2022-12-19  5:29   ` phodina via Guix-patches via
  2022-12-25 18:05     ` phodina via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: phodina via Guix-patches via @ 2022-12-19  5:29 UTC (permalink / raw)
  To: Tobias Geerinckx-Rice; +Cc: 60171






Sent with Proton Mail secure email.

------- Original Message -------
On Sunday, December 18th, 2022 at 1:17 PM, Tobias Geerinckx-Rice <me@tobias.gr> wrote:


> Hi!
> 
> phodina 写道:
> 
> > Enabling support for X11 solves the problem.
> 
> 
> Are you sure your Guix & mpv installation are up to date?
> 
> See
> 
> https://issues.guix.gnu.org/59508 (patch)
> https://issues.guix.gnu.org/59279
> 
> and specifically
> 
> https://issues.guix.gnu.org/59378#2 and linked
> https://issues.guix.gnu.org/59339
> 
> which mentions your exact error message as being fixed back in
> November.
> 
> Kind regards,
> 
> T G-R

Hi Tobias,

I checked the patch is applied.

I use the following setup SDDM running under wayland and GNOME on wayland - here it works ok. 

However, I have a second environment where I spend half the time and it runs i3. Here the `mpv` gives me an error.

$ guix describe
Generation 11	Dec 18 2022 00:32:54	(current)
  guix f28ca24
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: f28ca2447c5e2eef1ba6a3a11587380a665b0e26


----
Petr




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#60171] Fix mpv on X11
  2022-12-19  5:29   ` phodina via Guix-patches via
@ 2022-12-25 18:05     ` phodina via Guix-patches via
  0 siblings, 0 replies; 4+ messages in thread
From: phodina via Guix-patches via @ 2022-12-25 18:05 UTC (permalink / raw)
  To: Tobias Geerinckx-Rice; +Cc: 60171

Hi Tobias,

you were right. There's no need for this patch as the mpv installed in my home profile was causing the trouble. 

I've found out when comparing the paths to the mpv.

This ticket can be closed now.

----
Petr




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-12-25 18:06 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-18  0:21 [bug#60171] Fix mpv on X11 phodina via Guix-patches via
2022-12-18 12:17 ` Tobias Geerinckx-Rice via Guix-patches via
2022-12-19  5:29   ` phodina via Guix-patches via
2022-12-25 18:05     ` phodina via Guix-patches via

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).