Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Readme: Adds note and link to yt-dlp #30613

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Jonta
Copy link

@Jonta Jonta commented Feb 7, 2022

Purpose

  • Improvement

Just getting a bold sketch out there

I still want this to include:

  • Mention of the break after the temporary takedown from GitHub
  • Short explanation of the change in management
  • Reasons for still supporting 2.7

Uncertain of how much of this is acceptable to @dirkf and the other maintainers, so getting the ball rolling for now. =)

Just getting a bold sketch out there

I still want this to include:

- Mention of the break after the temporary takedown from GitHub
- Short explanation of the change in management
- Reasons for still supporting 2.7

Uncertain of how much of this is acceptable to @dirkf and the other maintainers, so getting the ball rolling for now. =)
@Jonta Jonta mentioned this pull request Feb 7, 2022
@Jonta
Copy link
Author

Jonta commented Feb 7, 2022

yt-dlp having Readme-sections "New features" and "Differences in default behaviour" seems worth mentioning

@Jonta
Copy link
Author

Jonta commented Feb 7, 2022

Perhaps even more forward such as "youtube-dl - download videos from youtube.com or other video platforms on Python 2.7"

@garoto
Copy link

garoto commented Feb 7, 2022

Note: youtube-dl maintains compatibility with Python 2.7, which is EOL (End Of Life)

Hmm Ok, PR looking good.

If you can: Run yt-dlp instead. :)

No offense dude, but that's just lazy.

Perhaps adding a simple footnote to the README.md with "youtube-dl forks worth checking out:" might be enough.

@gamer191
Copy link

gamer191 commented Feb 8, 2022

May I suggest instead wording it as
NOTE: Youtube-DL is designed to maintain compatibility with legacy systems.
We recommend using yt-dlp instead, if you can.
Don't forget to move your config file, if you have one!
For a full list of differences between Youtube-DL and yt-dlp, see Differences in default behaviour (this is essential reading)
Not convinced yet? Check out a list of everything yt-dlp does that youtube-dl doesn't

@gamer191
Copy link

gamer191 commented Feb 8, 2022

*correct me if I'm wrong about having to move the config file

@Jonta
Copy link
Author

Jonta commented Feb 8, 2022

If you can: Run yt-dlp instead. :)

No offense dude, but that's just lazy.

It's not a result of laziness, but rather of me reading this entire thread in one sitting, and knowing myself well enough to get something down quickly, instead of perfect solution fallacying it to next October. =)

@gamer191 I like what you write

Thoughts on those 3 bullets I want to include?

@gamer191
Copy link

Thoughts on those 3 bullets I want to include?

I, personally, think they should be done as a new PR, because they seem off topic for this PR, but obviously that's not my decision

@dirkf
Copy link
Contributor

dirkf commented Feb 13, 2022

Personally, I was happy with the wording in #30568, though it could be expanded:

The project has a fork https://github.com/yt-dlp/yt-dlp that offers a lot of extra functions but demands an up-to-date Python version.

Let's not mention the current bug that only occurs with Python 3.8+.

As to these:

Mention of the break after the temporary takedown from GitHub

I think #27013 covers it but maybe something could be added to a new FAQ item concerning the legality of yt-dl; here I would defer to the project owner.

Short explanation of the change in management

I think #30568 covers it.

Reasons for still supporting 2.7

Should go in the FAQ, I think. Obviously the fact that 2.7 is a supported platform will continue to be in the first para of Description.

@gamer191
Copy link

How about:

The program is optimised for legacy systems. If you use a modern system (capable of running python 3.6 or above), please use https://github.com/yt-dlp/yt-dlp instead. If you have used youtube-dl before, please read yt-dlp's [differences in default behaviour](https://github.com/yt-dlp/yt-dlp#differences-in-default-behavior) before migrating.

@gamer191
Copy link

gamer191 commented Jun 13, 2022

Actually, before any note gets added to the readme, we should probably check with @pukkandan, because it would likely increase yt-dlp's popularity significantly and Pukkandan might be ready for yt-dlp to blow up so much

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants