Howdy,
Might be useful to note that this was never sent to me, the BTS doesn't actually
send replies by default.
On Fri, 31 Dec 2021 20:18:14 +0100, Christoph Anton Mitterer wrote:
Hey Uni 193.
First, thanks for maintaining this.
Are you going to do that in https://salsa.debian.org/debian/yt-dlp ?
No. As the Vcs-* show, I don't have this on salsa.debian.org.
I'd have written an update to 0002-Disable-upstream-s-autoupdate-
mechanism.patch, which completely removes the run_update and
update_self functions... (better not have it in the code at all, so it
cannot be "accidentally" called by some other place in the code).
Bit hard to review a patch I can't see, but functionally how will this change
things? So far it sounds like it just enlarges it (thus increasing the
likelihood of conflicts) without much of a benefit.
If you wanted to adapt the patch, dropping the current __init__.py changes and
then only modify run_update to exit with the message (or one like it) that's in
parser.error now, that would seem to be about equal to the current changes but
fulfill your desire to neuter run_update completely.
~Unit 193
Unit193 @ Libera
Unit193 @ OFTC
