-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fetching formats failed #28
Comments
I also experience the same. does this only works on linux? I think this problem raise because we users use ytdlp, and at line 196-210 on .lua script is really depends on the youtube-dl, lead to broken downloading format. hmm.. Let me try to see if changing to yt-dlp.exe works EDIT: Yes it works if we replace youtube-dl to yt-dlp on line 164. Is there anyway to bind this config to .conf file? @jgreco ? as Youtube-dl already abandoned by the author itself.. and spin off as yt-dlp? |
@benyaminl I've changed it and I still get the same error |
Does yt-dlp is included on windows env PATH? If no the error will raise..
so please try to check it. Check also if yoy are using ytdl or ytdlp.
Regards,
Benyamin Limanto
sent from my Sony XPERIA XZ Premium
…On Tue, 1 Feb 2022, 22:25 RafeeDaBoy, ***@***.***> wrote:
@benyaminl <https://github.com/benyaminl> I've changed it and I still get
the same error
—
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAVP2NFNGEFGG2WYZXGHUKDUY73NDANCNFSM5NEXWIYA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Moving it from the mpv's install directory to ~\AppData\Roaming\mpv\scripts solved the issue, along with changing youtube-dl to yt-dlp on line 164. Thanks! |
@RafeeDaBoy I think yt-dlp path should be tested on cmd first, because I've yt-dlp and mpv on same folder and I put it on env path, then it works. Please remember that when you change env path, you need to restart, as lua/python can't detect automatically I think this issue should kept open as we need to adopt it as conf. ping @jgreco |
Yes, I had checked if yt-dlp and mpv was working in cmd and it was. And I've reopened the issue. |
Made this code to fix this error -- Function to check if a command is executable
function isExecutable(command)
local f = io.popen("which " .. command)
local path = f:read("*a")
f:close()
return string.len(path) > 0
end
-- Check if youtube-dl or yt-dlp is in PATH
if isExecutable("youtube-dl") then
yt_tool = "youtube-dl"
elseif isExecutable("yt-dlp") then
yt_tool = "yt-dlp"
end
local ytdl = {
path = yt_tool,
searched = false,
blacklisted = {}
}
|
@benyaminl done #32 |
where do i add this inside my lua script? at the end of file? |
@subhanshu1111 First, test if these commands are working: youtube-dl -F {URL} or yt-dlp -F {URL} Sometimes youtube-dl or yt-dlp don't work correctly. |
I am having this problem on openSUSE TW and I also removed the unmaintained youtube-dl and use yt-dl only which is in /usr/bin Here is the result of -F |
When pressing Ctrl + f gives out an error
The text was updated successfully, but these errors were encountered: