guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#61117] Update svtplay-dl to 4.18


From: Jessica Tallon
Subject: [bug#61117] Update svtplay-dl to 4.18
Date: Sun, 5 Mar 2023 11:39:10 +0100

Hello,

Sorry for the long delay on addressing this feedback.

Tobias Geerinckx-Rice <me@tobias.gr> writes:
> Jessica,
>
> Please avoid propagation whenever possible; it breaks all kinds of
> nice things.
>
> Propagating A from B means that Guix will effectively ‘install’ A
> alongside B when the user installs only B.  Or how classical package
> managers bluntly handle ‘dependencies’.  Not good!
>

Thanks for the tip, I've included a patch which does it this way.

>
>> it's needed while the program is running to merge two files, not
>> while building.
>
> So 4.17 did not need or use ffmpeg this way?
>
> Otherwise, put each unrelated changes into a separate commit: one to
> improve the ffmpeg situation, and one to update svtplay-dl. You can
> send multiple related commits as one patch series to one bug number,
> though.

4.17 did indeed need ffmpeg, I've included two patches one which
resolves the ffmpeg issue and the other which updates it to 4.18.

Thanks,
Jessica.

Attachment: 0000-cover-letter.patch
Description: Text Data

Attachment: 0001-gnu-fix-svtplay-dl-wrapping-to-use-ffmpeg.patch
Description: Text Data

Attachment: 0002-gnu-svtplay-dl-update-to-4.18.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]