* [PR PATCH] MPV MPRIS plugin fix
@ 2020-08-15 0:09 harrisonthorne
2020-08-15 0:39 ` [PR REVIEW] " ericonr
` (9 more replies)
0 siblings, 10 replies; 11+ messages in thread
From: harrisonthorne @ 2020-08-15 0:09 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 590 bytes --]
There is a new pull request by harrisonthorne against master on the void-packages repository
https://github.com/harrisonthorne/void-packages mpv-mpris-fix
https://github.com/void-linux/void-packages/pull/24285
MPV MPRIS plugin fix
mpv and the mpv-mpris plugin weren't working together out-of-the-box for me, so these edits fix that.
I've confirmed on my end that these changes work :)
The changes made are based on the documentation for mpv-mpris [here](https://github.com/hoyon/mpv-mpris).
A patch file from https://github.com/void-linux/void-packages/pull/24285.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mpv-mpris-fix-24285.patch --]
[-- Type: text/x-diff, Size: 2748 bytes --]
From b097c4ceac589e0c99a603793a3e0666a1004c7e Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Fri, 14 Aug 2020 17:44:36 -0600
Subject: [PATCH 1/2] mpv-mpris: Change install directory
This change is part of an effort to fix integration between MPV and its
MPRIS plugin.
---
srcpkgs/mpv-mpris/template | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/srcpkgs/mpv-mpris/template b/srcpkgs/mpv-mpris/template
index 774c21c5965..1d956beb661 100644
--- a/srcpkgs/mpv-mpris/template
+++ b/srcpkgs/mpv-mpris/template
@@ -1,7 +1,7 @@
# Template file for 'mpv-mpris'
pkgname=mpv-mpris
version=0.5
-revision=1
+revision=2
build_style=gnu-makefile
make_build_target=mpris.so
hostmakedepends="pkg-config"
@@ -16,6 +16,6 @@ checksum=673aff031e7cc741edea68d7b4b0103d7b031d4a55755abb9e1be5dd4ec4e969
do_install() {
vlicense LICENSE
- vmkdir usr/lib/${pkgname}
- vinstall mpris.so 0644 usr/lib/${pkgname}
+ vmkdir etc/mpv/scripts
+ vinstall mpris.so 0644 etc/mpv/scripts/
}
From 3e92e83d35abb6c080209a76edec6a725dde9da0 Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Fri, 14 Aug 2020 17:47:42 -0600
Subject: [PATCH 2/2] mpv: Add --enable-cplugins config option
This change is part of an effort to fix integration between MPV and its
MPRIS plugin.
---
srcpkgs/mpv/template | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/srcpkgs/mpv/template b/srcpkgs/mpv/template
index 598624e8949..9535ecc048b 100644
--- a/srcpkgs/mpv/template
+++ b/srcpkgs/mpv/template
@@ -1,15 +1,15 @@
# Template file for 'mpv'
pkgname=mpv
version=0.32.0
-revision=3
+revision=4
build_style=waf3
configure_args="--confdir=/etc/mpv --docdir=/usr/share/examples/mpv
--enable-dvdnav --enable-cdda --enable-libmpv-shared
- --enable-dvbin --enable-libarchive $(vopt_enable alsa) $(vopt_enable caca)
- $(vopt_enable jack) $(vopt_enable lua) $(vopt_enable oss oss-audio)
- $(vopt_enable pulseaudio pulse) $(vopt_enable sdl2)
- $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
- $(vopt_enable vapoursynth) $(vopt_enable vdpau) $(vopt_enable wayland)
+ --enable-dvbin --enable-libarchive --enable-cplugins $(vopt_enable alsa)
+ $(vopt_enable caca) $(vopt_enable jack) $(vopt_enable lua)
+ $(vopt_enable oss oss-audio) $(vopt_enable pulseaudio pulse)
+ $(vopt_enable sdl2) $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
+ $(vopt_enable vapoursynth) $(vopt_enable vdpau) $(vopt_enable wayland)
$(vopt_enable x11) $(vopt_enable vulkan) $(vopt_enable vulkan shaderc)"
hostmakedepends="pkg-config python3-docutils perl $(vopt_if wayland wayland-devel)"
makedepends="MesaLib-devel ffmpeg-devel harfbuzz-devel lcms2-devel libXv-devel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
@ 2020-08-15 0:39 ` ericonr
2020-08-15 1:18 ` flexibeast
` (8 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2020-08-15 0:39 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 366 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r470915567
Comment:
Does this enable it by default? In my setup I have `~/.config/mpv/scripts/mpris.so -> /usr/lib/mpv-mpris/mpris.so`, and it just works. Having it enabled for everyone wouldn't be ideal for a multiuser setup or similar.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
2020-08-15 0:39 ` [PR REVIEW] " ericonr
@ 2020-08-15 1:18 ` flexibeast
2020-08-22 19:12 ` harrisonthorne
` (7 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: flexibeast @ 2020-08-15 1:18 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 224 bytes --]
New review comment by flexibeast on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r470920181
Comment:
Also, would this result in errors for people who aren't using a D-Bus bus?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
2020-08-15 0:39 ` [PR REVIEW] " ericonr
2020-08-15 1:18 ` flexibeast
@ 2020-08-22 19:12 ` harrisonthorne
2020-08-22 20:38 ` ericonr
` (6 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: harrisonthorne @ 2020-08-22 19:12 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 431 bytes --]
New review comment by harrisonthorne on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r475121415
Comment:
In either case @ericonr makes a good point. However, if we revert this change, I think it would be helpful to include a post-install message letting users know to `ln -s /usr/lib/mpv-mpris/mpris.so ~/.config/mpv/scripts/mpris.so`. At least for slightly noobish users like me ;)
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (2 preceding siblings ...)
2020-08-22 19:12 ` harrisonthorne
@ 2020-08-22 20:38 ` ericonr
2020-08-24 18:37 ` [PR PATCH] [Updated] " harrisonthorne
` (5 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2020-08-22 20:38 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 227 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r475136280
Comment:
What do you think about `vdoc`ing the README? https://github.com/hoyon/mpv-mpris
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Updated] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (3 preceding siblings ...)
2020-08-22 20:38 ` ericonr
@ 2020-08-24 18:37 ` harrisonthorne
2020-08-24 18:40 ` harrisonthorne
` (4 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: harrisonthorne @ 2020-08-24 18:37 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 595 bytes --]
There is an updated pull request by harrisonthorne against master on the void-packages repository
https://github.com/harrisonthorne/void-packages mpv-mpris-fix
https://github.com/void-linux/void-packages/pull/24285
MPV MPRIS plugin fix
mpv and the mpv-mpris plugin weren't working together out-of-the-box for me, so these edits fix that.
I've confirmed on my end that these changes work :)
The changes made are based on the documentation for mpv-mpris [here](https://github.com/hoyon/mpv-mpris).
A patch file from https://github.com/void-linux/void-packages/pull/24285.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mpv-mpris-fix-24285.patch --]
[-- Type: text/x-diff, Size: 2317 bytes --]
From 8042f9ad2ae5f19c5850dfeaa15f88c4532f1088 Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Fri, 14 Aug 2020 17:47:42 -0600
Subject: [PATCH 1/2] mpv: Add --enable-cplugins config option
This change is part of an effort to fix integration between MPV and its
MPRIS plugin.
---
srcpkgs/mpv/template | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/srcpkgs/mpv/template b/srcpkgs/mpv/template
index 598624e8949..9535ecc048b 100644
--- a/srcpkgs/mpv/template
+++ b/srcpkgs/mpv/template
@@ -1,15 +1,15 @@
# Template file for 'mpv'
pkgname=mpv
version=0.32.0
-revision=3
+revision=4
build_style=waf3
configure_args="--confdir=/etc/mpv --docdir=/usr/share/examples/mpv
--enable-dvdnav --enable-cdda --enable-libmpv-shared
- --enable-dvbin --enable-libarchive $(vopt_enable alsa) $(vopt_enable caca)
- $(vopt_enable jack) $(vopt_enable lua) $(vopt_enable oss oss-audio)
- $(vopt_enable pulseaudio pulse) $(vopt_enable sdl2)
- $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
- $(vopt_enable vapoursynth) $(vopt_enable vdpau) $(vopt_enable wayland)
+ --enable-dvbin --enable-libarchive --enable-cplugins $(vopt_enable alsa)
+ $(vopt_enable caca) $(vopt_enable jack) $(vopt_enable lua)
+ $(vopt_enable oss oss-audio) $(vopt_enable pulseaudio pulse)
+ $(vopt_enable sdl2) $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
+ $(vopt_enable vapoursynth) $(vopt_enable vdpau) $(vopt_enable wayland)
$(vopt_enable x11) $(vopt_enable vulkan) $(vopt_enable vulkan shaderc)"
hostmakedepends="pkg-config python3-docutils perl $(vopt_if wayland wayland-devel)"
makedepends="MesaLib-devel ffmpeg-devel harfbuzz-devel lcms2-devel libXv-devel
From 82f6dc269a6f28da770fda48d596e6940e03ebe0 Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Mon, 24 Aug 2020 12:37:10 -0600
Subject: [PATCH 2/2] mpv-mpris: vdoc README.md
---
srcpkgs/mpv-mpris/template | 1 +
1 file changed, 1 insertion(+)
diff --git a/srcpkgs/mpv-mpris/template b/srcpkgs/mpv-mpris/template
index 774c21c5965..edfa832fcce 100644
--- a/srcpkgs/mpv-mpris/template
+++ b/srcpkgs/mpv-mpris/template
@@ -18,4 +18,5 @@ do_install() {
vlicense LICENSE
vmkdir usr/lib/${pkgname}
vinstall mpris.so 0644 usr/lib/${pkgname}
+ vdoc README.md
}
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Updated] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (4 preceding siblings ...)
2020-08-24 18:37 ` [PR PATCH] [Updated] " harrisonthorne
@ 2020-08-24 18:40 ` harrisonthorne
2020-08-24 18:43 ` [PR REVIEW] " harrisonthorne
` (3 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: harrisonthorne @ 2020-08-24 18:40 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 595 bytes --]
There is an updated pull request by harrisonthorne against master on the void-packages repository
https://github.com/harrisonthorne/void-packages mpv-mpris-fix
https://github.com/void-linux/void-packages/pull/24285
MPV MPRIS plugin fix
mpv and the mpv-mpris plugin weren't working together out-of-the-box for me, so these edits fix that.
I've confirmed on my end that these changes work :)
The changes made are based on the documentation for mpv-mpris [here](https://github.com/hoyon/mpv-mpris).
A patch file from https://github.com/void-linux/void-packages/pull/24285.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mpv-mpris-fix-24285.patch --]
[-- Type: text/x-diff, Size: 2188 bytes --]
From 1041085d1502dbd8d03e8480596c9f912a76e028 Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Fri, 14 Aug 2020 17:47:42 -0600
Subject: [PATCH 1/2] mpv: Add --enable-cplugins config option
This change is part of an effort to fix integration between MPV and its
MPRIS plugin.
mpv: Remove trailing whitespace
---
srcpkgs/mpv/template | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/srcpkgs/mpv/template b/srcpkgs/mpv/template
index 598624e8949..f8dae99df85 100644
--- a/srcpkgs/mpv/template
+++ b/srcpkgs/mpv/template
@@ -1,14 +1,14 @@
# Template file for 'mpv'
pkgname=mpv
version=0.32.0
-revision=3
+revision=4
build_style=waf3
configure_args="--confdir=/etc/mpv --docdir=/usr/share/examples/mpv
--enable-dvdnav --enable-cdda --enable-libmpv-shared
- --enable-dvbin --enable-libarchive $(vopt_enable alsa) $(vopt_enable caca)
- $(vopt_enable jack) $(vopt_enable lua) $(vopt_enable oss oss-audio)
- $(vopt_enable pulseaudio pulse) $(vopt_enable sdl2)
- $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
+ --enable-dvbin --enable-libarchive --enable-cplugins $(vopt_enable alsa)
+ $(vopt_enable caca) $(vopt_enable jack) $(vopt_enable lua)
+ $(vopt_enable oss oss-audio) $(vopt_enable pulseaudio pulse)
+ $(vopt_enable sdl2) $(vopt_enable smb libsmbclient) $(vopt_enable sndio)
$(vopt_enable vapoursynth) $(vopt_enable vdpau) $(vopt_enable wayland)
$(vopt_enable x11) $(vopt_enable vulkan) $(vopt_enable vulkan shaderc)"
hostmakedepends="pkg-config python3-docutils perl $(vopt_if wayland wayland-devel)"
From 8fb3e1627640e4c92ac8233d5e382f8a3953af61 Mon Sep 17 00:00:00 2001
From: Harrison Thorne <harrisonthorne@protonmail.com>
Date: Mon, 24 Aug 2020 12:37:10 -0600
Subject: [PATCH 2/2] mpv-mpris: vdoc README.md
---
srcpkgs/mpv-mpris/template | 1 +
1 file changed, 1 insertion(+)
diff --git a/srcpkgs/mpv-mpris/template b/srcpkgs/mpv-mpris/template
index 774c21c5965..e82e3f70222 100644
--- a/srcpkgs/mpv-mpris/template
+++ b/srcpkgs/mpv-mpris/template
@@ -18,4 +18,5 @@ do_install() {
vlicense LICENSE
vmkdir usr/lib/${pkgname}
vinstall mpris.so 0644 usr/lib/${pkgname}
+ vdoc README.md
}
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (5 preceding siblings ...)
2020-08-24 18:40 ` harrisonthorne
@ 2020-08-24 18:43 ` harrisonthorne
2020-08-24 19:23 ` ericonr
` (2 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: harrisonthorne @ 2020-08-24 18:43 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 272 bytes --]
New review comment by harrisonthorne on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r475820796
Comment:
What does that do exactly (I added vdoc anyways)? I was thinking something more along the lines of using `INSTALL.msg`
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (6 preceding siblings ...)
2020-08-24 18:43 ` [PR REVIEW] " harrisonthorne
@ 2020-08-24 19:23 ` ericonr
2021-02-11 2:05 ` ericonr
2021-02-11 2:05 ` [PR PATCH] [Closed]: " ericonr
9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2020-08-24 19:23 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 252 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#discussion_r475841572
Comment:
It adds the README to the package, so people can hunt for how to enable it in `/usr/share/doc/mpv-mpris`.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (7 preceding siblings ...)
2020-08-24 19:23 ` ericonr
@ 2021-02-11 2:05 ` ericonr
2021-02-11 2:05 ` [PR PATCH] [Closed]: " ericonr
9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-02-11 2:05 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 207 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/24285#issuecomment-777161980
Comment:
Closed in c26a75635468cb20c6d6fafc7bb66095e9f58975 and its parent.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Closed]: MPV MPRIS plugin fix
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
` (8 preceding siblings ...)
2021-02-11 2:05 ` ericonr
@ 2021-02-11 2:05 ` ericonr
9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-02-11 2:05 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 419 bytes --]
There's a closed pull request on the void-packages repository
MPV MPRIS plugin fix
https://github.com/void-linux/void-packages/pull/24285
Description:
mpv and the mpv-mpris plugin weren't working together out-of-the-box for me, so these edits fix that.
I've confirmed on my end that these changes work :)
The changes made are based on the documentation for mpv-mpris [here](https://github.com/hoyon/mpv-mpris).
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2021-02-11 2:05 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-15 0:09 [PR PATCH] MPV MPRIS plugin fix harrisonthorne
2020-08-15 0:39 ` [PR REVIEW] " ericonr
2020-08-15 1:18 ` flexibeast
2020-08-22 19:12 ` harrisonthorne
2020-08-22 20:38 ` ericonr
2020-08-24 18:37 ` [PR PATCH] [Updated] " harrisonthorne
2020-08-24 18:40 ` harrisonthorne
2020-08-24 18:43 ` [PR REVIEW] " harrisonthorne
2020-08-24 19:23 ` ericonr
2021-02-11 2:05 ` ericonr
2021-02-11 2:05 ` [PR PATCH] [Closed]: " ericonr
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 NNTP newsgroup(s).