Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4.
@ 2023-03-08 19:44 icp1994
  2023-03-09  1:14 ` sgn
                   ` (8 more replies)
  0 siblings, 9 replies; 10+ messages in thread
From: icp1994 @ 2023-03-08 19:44 UTC (permalink / raw)
  To: ml

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

There is a new pull request by icp1994 against master on the void-packages repository

https://github.com/icp1994/void-packages mps-youtube
https://github.com/void-linux/void-packages/pull/42662

mps-youtube: rename to yewtube, update to 2.9.4.
#### Testing the changes
- I tested the changes in this PR: **WIP**

#### Local build testing
- I built this PR locally for my native architecture: **x86_64**

Seems like mps-youtube has been rebranded in upstream a while ago. There is a couple of hiccups while trying to test it though. The first one is that it complained yt-dlp requires brotli which I fixed in https://github.com/void-linux/void-packages/pull/42661. Now it says httpx requires rfc3986[idna2008]<2,>=1.3 which is also [true](https://github.com/encode/httpx/blob/0.23.3/pyproject.toml#L33) but the version in our repo is already 2.0 so I don't know how to fix that.

A patch file from https://github.com/void-linux/void-packages/pull/42662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mps-youtube-42662.patch --]
[-- Type: text/x-diff, Size: 2570 bytes --]

From 527d16b96709632dacc198baba43608fd4569c06 Mon Sep 17 00:00:00 2001
From: icp <pangolin@vivaldi.net>
Date: Thu, 9 Mar 2023 00:42:46 +0530
Subject: [PATCH] mps-youtube: rename to yewtube, update to 2.9.4.

---
 srcpkgs/mps-youtube          |  1 +
 srcpkgs/mps-youtube/template | 14 --------------
 srcpkgs/yewtube/template     | 21 +++++++++++++++++++++
 3 files changed, 22 insertions(+), 14 deletions(-)
 create mode 120000 srcpkgs/mps-youtube
 delete mode 100644 srcpkgs/mps-youtube/template
 create mode 100644 srcpkgs/yewtube/template

diff --git a/srcpkgs/mps-youtube b/srcpkgs/mps-youtube
new file mode 120000
index 000000000000..9223c39c3000
--- /dev/null
+++ b/srcpkgs/mps-youtube
@@ -0,0 +1 @@
+yewtube
\ No newline at end of file
diff --git a/srcpkgs/mps-youtube/template b/srcpkgs/mps-youtube/template
deleted file mode 100644
index 92b190949861..000000000000
--- a/srcpkgs/mps-youtube/template
+++ /dev/null
@@ -1,14 +0,0 @@
-# Template file for 'mps-youtube'
-pkgname=mps-youtube
-version=0.2.8
-revision=6
-build_style=python3-module
-pycompile_module="mps_youtube"
-hostmakedepends="python3-setuptools"
-depends="python3-pafy python3-setuptools"
-short_desc="Terminal based YouTube player and downloader"
-maintainer="Orphaned <orphan@voidlinux.org>"
-license="GPL-3.0-or-later"
-homepage="https://github.com/mps-youtube/mps-youtube/"
-distfiles="https://github.com/mps-youtube/${pkgname}/archive/v${version}.tar.gz"
-checksum=d5f2c4bc1f57f0566242c4a0a721a5ceaa6d6d407f9d6dd29009a714a0abec74
diff --git a/srcpkgs/yewtube/template b/srcpkgs/yewtube/template
new file mode 100644
index 000000000000..0beaa7575d4c
--- /dev/null
+++ b/srcpkgs/yewtube/template
@@ -0,0 +1,21 @@
+# Template file for 'yewtube'
+pkgname=yewtube
+version=2.9.4
+revision=1
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-requests python3-pyperclip python3-youtubesearch yt-dlp"
+checkdepends="${depends} python3-pip python3-dbus python3-gobject python3-pytest-xdist"
+short_desc="Terminal based YouTube player and downloader"
+maintainer="icp <pangolin@vivaldi.net>"
+license="GPL-3.0-or-later"
+homepage="https://github.com/mps-youtube/yewtube"
+changelog="https://raw.githubusercontent.com/mps-youtube/yewtube/master/CHANGELOG.md"
+distfiles="https://github.com/mps-youtube/yewtube/archive/refs/tags/v${version}.tar.gz"
+checksum=f31d43b6ea3a44234e7e0a5c31822e40fbae66362234fdaca0cf492f773bdee0
+
+mps-youtube_package() {
+	depends="${sourcepkg}>=${version}_${revision}"
+	build_style=meta
+	short_desc+=" (transitional dummy package)"
+}

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

* Re: mps-youtube: rename to yewtube, update to 2.9.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
@ 2023-03-09  1:14 ` sgn
  2023-03-09  1:14 ` sgn
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: sgn @ 2023-03-09  1:14 UTC (permalink / raw)
  To: ml

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1461114637

Comment:
It seems like httpx can use rfc3986 2.0, please patch it and run unittest.

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

* Re: mps-youtube: rename to yewtube, update to 2.9.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
  2023-03-09  1:14 ` sgn
@ 2023-03-09  1:14 ` sgn
  2023-03-09 20:14 ` icp1994
                   ` (6 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: sgn @ 2023-03-09  1:14 UTC (permalink / raw)
  To: ml

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1461114637

Comment:
It seems like httpx can use rfc3986 2.0, please patch it and run unittest.
https://github.com/encode/httpx/issues/2169

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

* Re: mps-youtube: rename to yewtube, update to 2.9.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
  2023-03-09  1:14 ` sgn
  2023-03-09  1:14 ` sgn
@ 2023-03-09 20:14 ` icp1994
  2023-05-09  3:37 ` sgn
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: icp1994 @ 2023-03-09 20:14 UTC (permalink / raw)
  To: ml

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

New comment by icp1994 on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1462707414

Comment:
https://github.com/void-linux/void-packages/pull/42681 for httpx but unfortunately yewtube still not working with unrelated issue(s).

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

* Re: mps-youtube: rename to yewtube, update to 2.9.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (2 preceding siblings ...)
  2023-03-09 20:14 ` icp1994
@ 2023-05-09  3:37 ` sgn
  2023-05-09  7:53 ` [PR PATCH] [Updated] " icp1994
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: sgn @ 2023-05-09  3:37 UTC (permalink / raw)
  To: ml

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1539339355

Comment:
?

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

* Re: [PR PATCH] [Updated] mps-youtube: rename to yewtube, update to 2.9.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (3 preceding siblings ...)
  2023-05-09  3:37 ` sgn
@ 2023-05-09  7:53 ` icp1994
  2023-06-27 22:38 ` mps-youtube: rename to yewtube, update to 2.10.2 classabbyamp
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: icp1994 @ 2023-05-09  7:53 UTC (permalink / raw)
  To: ml

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

There is an updated pull request by icp1994 against master on the void-packages repository

https://github.com/icp1994/void-packages mps-youtube
https://github.com/void-linux/void-packages/pull/42662

mps-youtube: rename to yewtube, update to 2.9.4.
#### Testing the changes
- I tested the changes in this PR: **WIP**

#### Local build testing
- I built this PR locally for my native architecture: **x86_64**

Seems like mps-youtube has been rebranded in upstream a while ago. There is a couple of hiccups while trying to test it though. The first one is that it complained yt-dlp requires brotli which I fixed in https://github.com/void-linux/void-packages/pull/42661. Now it says httpx requires rfc3986[idna2008]<2,>=1.3 which is also [true](https://github.com/encode/httpx/blob/0.23.3/pyproject.toml#L33) but the version in our repo is already 2.0 so I don't know how to fix that.

A patch file from https://github.com/void-linux/void-packages/pull/42662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mps-youtube-42662.patch --]
[-- Type: text/x-diff, Size: 2715 bytes --]

From d9aa47cab2c3254c5d58a75c6381781d825a628d Mon Sep 17 00:00:00 2001
From: icp <pangolin@vivaldi.net>
Date: Thu, 9 Mar 2023 00:42:46 +0530
Subject: [PATCH] mps-youtube: rename to yewtube, update to 2.10.2.

---
 srcpkgs/mps-youtube          |  1 +
 srcpkgs/mps-youtube/template | 14 --------------
 srcpkgs/yewtube/template     | 26 ++++++++++++++++++++++++++
 3 files changed, 27 insertions(+), 14 deletions(-)
 create mode 120000 srcpkgs/mps-youtube
 delete mode 100644 srcpkgs/mps-youtube/template
 create mode 100644 srcpkgs/yewtube/template

diff --git a/srcpkgs/mps-youtube b/srcpkgs/mps-youtube
new file mode 120000
index 000000000000..9223c39c3000
--- /dev/null
+++ b/srcpkgs/mps-youtube
@@ -0,0 +1 @@
+yewtube
\ No newline at end of file
diff --git a/srcpkgs/mps-youtube/template b/srcpkgs/mps-youtube/template
deleted file mode 100644
index 92b190949861..000000000000
--- a/srcpkgs/mps-youtube/template
+++ /dev/null
@@ -1,14 +0,0 @@
-# Template file for 'mps-youtube'
-pkgname=mps-youtube
-version=0.2.8
-revision=6
-build_style=python3-module
-pycompile_module="mps_youtube"
-hostmakedepends="python3-setuptools"
-depends="python3-pafy python3-setuptools"
-short_desc="Terminal based YouTube player and downloader"
-maintainer="Orphaned <orphan@voidlinux.org>"
-license="GPL-3.0-or-later"
-homepage="https://github.com/mps-youtube/mps-youtube/"
-distfiles="https://github.com/mps-youtube/${pkgname}/archive/v${version}.tar.gz"
-checksum=d5f2c4bc1f57f0566242c4a0a721a5ceaa6d6d407f9d6dd29009a714a0abec74
diff --git a/srcpkgs/yewtube/template b/srcpkgs/yewtube/template
new file mode 100644
index 000000000000..91ed1e768f2a
--- /dev/null
+++ b/srcpkgs/yewtube/template
@@ -0,0 +1,26 @@
+# Template file for 'yewtube'
+pkgname=yewtube
+version=2.10.2
+revision=1
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-requests python3-pyperclip python3-youtubesearch python3-pylast yt-dlp
+ python3-mutagen python3-pycryptodomex python3-websockets python3-Brotli python3-certifi"
+checkdepends="${depends} python3-pip python3-dbus python3-gobject python3-pytest-xdist"
+short_desc="Terminal based YouTube player and downloader"
+maintainer="icp <pangolin@vivaldi.net>"
+license="GPL-3.0-or-later"
+homepage="https://github.com/mps-youtube/yewtube"
+changelog="https://raw.githubusercontent.com/mps-youtube/yewtube/master/CHANGELOG.md"
+distfiles="https://github.com/mps-youtube/yewtube/archive/refs/tags/v${version}.tar.gz"
+checksum=418607c8eb243c5d48a97089847f24fc475aa302b764e262a7fcc375c6992c63
+
+post_install() {
+	vbin yt
+}
+
+mps-youtube_package() {
+	depends="${sourcepkg}>=${version}_${revision}"
+	build_style=meta
+	short_desc+=" (transitional dummy package)"
+}

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

* Re: mps-youtube: rename to yewtube, update to 2.10.2.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (4 preceding siblings ...)
  2023-05-09  7:53 ` [PR PATCH] [Updated] " icp1994
@ 2023-06-27 22:38 ` classabbyamp
  2023-06-28  6:41 ` [PR PATCH] [Updated] " icp1994
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: classabbyamp @ 2023-06-27 22:38 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1610313653

Comment:
does it work now?

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

* Re: [PR PATCH] [Updated] mps-youtube: rename to yewtube, update to 2.10.2.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (5 preceding siblings ...)
  2023-06-27 22:38 ` mps-youtube: rename to yewtube, update to 2.10.2 classabbyamp
@ 2023-06-28  6:41 ` icp1994
  2023-06-28  6:41 ` icp1994
  2023-06-28  8:12 ` [PR PATCH] [Merged]: mps-youtube: rename to yewtube, update to 2.10.4 classabbyamp
  8 siblings, 0 replies; 10+ messages in thread
From: icp1994 @ 2023-06-28  6:41 UTC (permalink / raw)
  To: ml

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

There is an updated pull request by icp1994 against master on the void-packages repository

https://github.com/icp1994/void-packages mps-youtube
https://github.com/void-linux/void-packages/pull/42662

mps-youtube: rename to yewtube, update to 2.10.2.
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture: **x86_64**

Seems like mps-youtube has been rebranded in upstream a while ago. There is a couple of hiccups while trying to test it though. The first one is that it complained yt-dlp requires brotli which I fixed in https://github.com/void-linux/void-packages/pull/42661. Now it says httpx requires rfc3986[idna2008]<2,>=1.3 which is also [true](https://github.com/encode/httpx/blob/0.23.3/pyproject.toml#L33) but the version in our repo is already 2.0 so I don't know how to fix that.

A patch file from https://github.com/void-linux/void-packages/pull/42662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-mps-youtube-42662.patch --]
[-- Type: text/x-diff, Size: 2715 bytes --]

From 08f50d90f5605e9d850be47109c82208593cbb34 Mon Sep 17 00:00:00 2001
From: icp <pangolin@vivaldi.net>
Date: Thu, 9 Mar 2023 00:42:46 +0530
Subject: [PATCH] mps-youtube: rename to yewtube, update to 2.10.4.

---
 srcpkgs/mps-youtube          |  1 +
 srcpkgs/mps-youtube/template | 14 --------------
 srcpkgs/yewtube/template     | 26 ++++++++++++++++++++++++++
 3 files changed, 27 insertions(+), 14 deletions(-)
 create mode 120000 srcpkgs/mps-youtube
 delete mode 100644 srcpkgs/mps-youtube/template
 create mode 100644 srcpkgs/yewtube/template

diff --git a/srcpkgs/mps-youtube b/srcpkgs/mps-youtube
new file mode 120000
index 000000000000..9223c39c3000
--- /dev/null
+++ b/srcpkgs/mps-youtube
@@ -0,0 +1 @@
+yewtube
\ No newline at end of file
diff --git a/srcpkgs/mps-youtube/template b/srcpkgs/mps-youtube/template
deleted file mode 100644
index 92b190949861..000000000000
--- a/srcpkgs/mps-youtube/template
+++ /dev/null
@@ -1,14 +0,0 @@
-# Template file for 'mps-youtube'
-pkgname=mps-youtube
-version=0.2.8
-revision=6
-build_style=python3-module
-pycompile_module="mps_youtube"
-hostmakedepends="python3-setuptools"
-depends="python3-pafy python3-setuptools"
-short_desc="Terminal based YouTube player and downloader"
-maintainer="Orphaned <orphan@voidlinux.org>"
-license="GPL-3.0-or-later"
-homepage="https://github.com/mps-youtube/mps-youtube/"
-distfiles="https://github.com/mps-youtube/${pkgname}/archive/v${version}.tar.gz"
-checksum=d5f2c4bc1f57f0566242c4a0a721a5ceaa6d6d407f9d6dd29009a714a0abec74
diff --git a/srcpkgs/yewtube/template b/srcpkgs/yewtube/template
new file mode 100644
index 000000000000..b8d240443863
--- /dev/null
+++ b/srcpkgs/yewtube/template
@@ -0,0 +1,26 @@
+# Template file for 'yewtube'
+pkgname=yewtube
+version=2.10.4
+revision=1
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-requests python3-pyperclip python3-youtubesearch python3-pylast yt-dlp
+ python3-mutagen python3-pycryptodomex python3-websockets python3-Brotli python3-certifi"
+checkdepends="${depends} python3-pip python3-dbus python3-gobject python3-pytest-xdist"
+short_desc="Terminal based YouTube player and downloader"
+maintainer="icp <pangolin@vivaldi.net>"
+license="GPL-3.0-or-later"
+homepage="https://github.com/mps-youtube/yewtube"
+changelog="https://raw.githubusercontent.com/mps-youtube/yewtube/master/CHANGELOG.md"
+distfiles="https://github.com/mps-youtube/yewtube/archive/refs/tags/v${version}.tar.gz"
+checksum=e7b93f3929f9266ecb32317d0c23acdd8966626f83015e361ef44cff1089fa32
+
+post_install() {
+	vbin yt
+}
+
+mps-youtube_package() {
+	depends="${sourcepkg}>=${version}_${revision}"
+	build_style=meta
+	short_desc+=" (transitional dummy package)"
+}

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

* Re: mps-youtube: rename to yewtube, update to 2.10.2.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (6 preceding siblings ...)
  2023-06-28  6:41 ` [PR PATCH] [Updated] " icp1994
@ 2023-06-28  6:41 ` icp1994
  2023-06-28  8:12 ` [PR PATCH] [Merged]: mps-youtube: rename to yewtube, update to 2.10.4 classabbyamp
  8 siblings, 0 replies; 10+ messages in thread
From: icp1994 @ 2023-06-28  6:41 UTC (permalink / raw)
  To: ml

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

New comment by icp1994 on void-packages repository

https://github.com/void-linux/void-packages/pull/42662#issuecomment-1610853405

Comment:
Yes, have been using it for months now. Also pushed the latest patches.

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

* Re: [PR PATCH] [Merged]: mps-youtube: rename to yewtube, update to 2.10.4.
  2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
                   ` (7 preceding siblings ...)
  2023-06-28  6:41 ` icp1994
@ 2023-06-28  8:12 ` classabbyamp
  8 siblings, 0 replies; 10+ messages in thread
From: classabbyamp @ 2023-06-28  8:12 UTC (permalink / raw)
  To: ml

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

There's a merged pull request on the void-packages repository

mps-youtube: rename to yewtube, update to 2.10.4.
https://github.com/void-linux/void-packages/pull/42662

Description:
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture: **x86_64**

Seems like mps-youtube has been rebranded in upstream a while ago. There is a couple of hiccups while trying to test it though. The first one is that it complained yt-dlp requires brotli which I fixed in https://github.com/void-linux/void-packages/pull/42661. Now it says httpx requires rfc3986[idna2008]<2,>=1.3 which is also [true](https://github.com/encode/httpx/blob/0.23.3/pyproject.toml#L33) but the version in our repo is already 2.0 so I don't know how to fix that.

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

end of thread, other threads:[~2023-06-28  8:12 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-08 19:44 [PR PATCH] mps-youtube: rename to yewtube, update to 2.9.4 icp1994
2023-03-09  1:14 ` sgn
2023-03-09  1:14 ` sgn
2023-03-09 20:14 ` icp1994
2023-05-09  3:37 ` sgn
2023-05-09  7:53 ` [PR PATCH] [Updated] " icp1994
2023-06-27 22:38 ` mps-youtube: rename to yewtube, update to 2.10.2 classabbyamp
2023-06-28  6:41 ` [PR PATCH] [Updated] " icp1994
2023-06-28  6:41 ` icp1994
2023-06-28  8:12 ` [PR PATCH] [Merged]: mps-youtube: rename to yewtube, update to 2.10.4 classabbyamp

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).