File 0001-switch-to-yt-dlp.patch of Package QMPlay2

Overview Repositories Revisions Requests Users Attributes Meta

File 0001-switch-to-yt-dlp.patch of Package QMPlay2 (Revision 616b42b59f210465f911e8798c4361e5)

Currently displaying revision 616b42b59f210465f911e8798c4361e5 , Show latest

From 60c0a63c35afc5a8c95f0b94292500644abcfe41 Wed 13 10 08:50:32 2021
From: Simon Vogl <simon.vogl@gmx.net>
Date: Wed, 13 Oct 2021 08:50:32 UTC
Subject: [PATCH] Make QMPlay2 use yt-dlp instead of youtube-dl

This patch makes QMPlay2 use yt-dlp instead of youtube-dl.
The switch adresses several issues such as buffering not working correctly.

--- a/src/qmplay2/YouTubeDL.cpp
+++ b/src/qmplay2/YouTubeDL.cpp
@@ -22,6 +22,7 @@
 #include <QMPlay2Core.hpp>
 #include <Functions.hpp>
 
+#include <QRegularExpression>
 #include <QStandardPaths>
 #include <QJsonDocument>
 #include <QJsonObject>
@@ -36,9 +37,9 @@ static QMutex g_mutex(QMutex::Recursive);
 
 QString YouTubeDL::getFilePath()
 {
-    return QMPlay2Core.getSettingsDir() + "youtube-dl"
+    return QMPlay2Core.getSettingsDir() + "yt-dlp"
 #ifdef Q_OS_WIN
-    ".exe"
+    "_x86.exe"
 #endif
     ;
 }
@@ -305,9 +306,9 @@ bool YouTubeDL::download()
 {
     // Mutex must be locked here
 
-    const QString downloadUrl = "https://yt-dl.org/downloads/latest/youtube-dl"
+    const QString downloadUrl = "https://github.com/yt-dlp/yt-dlp/releases/latest/download/yt-dlp"
 #ifdef Q_OS_WIN
-    ".exe"
+    "_x86.exe"
 #endif
     ;
 
@@ -389,29 +390,11 @@ bool YouTubeDL::update()
         {
             qCritical() << "youtube-dl update failed:" << updateOutput;
         }
-        else if (m_process.exitCode() == 0 && !updateOutput.contains("up-to-date"))
+        else if (m_process.exitCode() == 0 && !updateOutput.contains(QRegularExpression(R"(up\Wto\Wdate)")))
         {
-#ifdef Q_OS_WIN
-            const QString updatedFile = m_ytDlPath + ".new";
-            QFile::remove(Functions::filePath(m_ytDlPath) + "youtube-dl-updater.bat");
-            if (QFile::exists(updatedFile))
-            {
-                Functions::s_wait(0.2); // Wait 200 ms to be sure that file is closed
-                QFile::remove(m_ytDlPath);
-                if (QFile::rename(updatedFile, m_ytDlPath))
-                {
-#endif
-                    QMPlay2Core.setWorking(false);
-                    emit QMPlay2Core.sendMessage(tr("\"youtube-dl\" has been successfully updated!"), g_name);
-                    return true;
-#ifdef Q_OS_WIN
-                }
-            }
-            else
-            {
-                qDebug() << "Updated youtube-dl file:" + updatedFile + "not found!";
-            }
-#endif
+            QMPlay2Core.setWorking(false);
+            emit QMPlay2Core.sendMessage(tr("\"youtube-dl\" has been successfully updated!"), g_name);
+            return true;
         }
     }
     else if (updating && m_aborted)

Places

File 0001-switch-to-yt-dlp.patch of Package QMPlay2 (Revision 616b42b59f210465f911e8798c4361e5)

Places