commit:     595b9a5ed61563222ae8a861ac67baaae7acc969
Author:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
AuthorDate: Thu Nov 13 11:36:44 2025 +0000
Commit:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
CommitDate: Thu Nov 13 11:42:30 2025 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=595b9a5e

dev-python/yt-dlp-ejs: re-arrange comments

Noticed the .js files were in github's assets and considered using them
rather than the wheel, but 1) they're ~3x bigger than the wheel due to
no compression 2) wheel spares us from having to figure out what to
include in the event that it changes given just using a wildcard 3)
would need to rename them both in SRC_URI and the ebuild to include
then remove ${PV}. The one advantage may be being able to skip the
unzip dep but well.

Just keep a short note about these.

Signed-off-by: Ionen Wolkens <ionen <AT> gentoo.org>

 dev-python/yt-dlp-ejs/yt-dlp-ejs-0.3.1.ebuild | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/dev-python/yt-dlp-ejs/yt-dlp-ejs-0.3.1.ebuild 
b/dev-python/yt-dlp-ejs/yt-dlp-ejs-0.3.1.ebuild
index a915c54d984d..ad0b85d31488 100644
--- a/dev-python/yt-dlp-ejs/yt-dlp-ejs-0.3.1.ebuild
+++ b/dev-python/yt-dlp-ejs/yt-dlp-ejs-0.3.1.ebuild
@@ -9,10 +9,11 @@ inherit distutils-r1 pypi
 
 DESCRIPTION="External JavaScript for yt-dlp supporting many runtimes"
 HOMEPAGE="https://github.com/yt-dlp/ejs/";
+# wheel for .js files, github's assets also has them but uncompressed
 SRC_URI+=" $(pypi_wheel_url --unpack)"
 
 LICENSE="Unlicense"
-LICENSE+=" ISC MIT" # .js dependencies from the wheel
+LICENSE+=" ISC MIT" # .js dependencies
 SLOT="0"
 # bumps should typically be done straight-to-stable like yt-dlp itself
 KEYWORDS="amd64 arm arm64 ~hppa ppc ppc64 ~riscv x86 ~arm64-macos ~x64-macos"
@@ -28,10 +29,9 @@ distutils_enable_tests unittest
 src_prepare() {
        distutils-r1_src_prepare
 
-       # drop deno/npm calls and use the pre-generated .js from the .whl
-       # instead, this both prevents network use and ensures no hash
-       # mismatch given yt-dlp checks the sha512sum of the .js files
-       # (makes generating our own meaningless given can't be patched)
+       # drop deno/npm calls and use pre-generated .js instead, this
+       # both prevents network use nad ensures no hash mismatch given
+       # yt-dlp checks the sha512sum of the .js files
        sed -i '/wheel.hooks.custom/d' pyproject.toml || die
        mv ../yt_dlp_ejs/yt/solver/*.js yt_dlp_ejs/yt/solver/ || die
 }

Reply via email to