Deploy Qt plugins with default deployStandardQtPlugins() function

dantti · dantti · commit b195c7ccd13d · 2024-06-19T09:15:48.000-03:00
diff --git a/src/deployers/BasicPluginsDeployer.cpp b/src/deployers/BasicPluginsDeployer.cpp
@@ -29,21 +29,23 @@ BasicPluginsDeployer::BasicPluginsDeployer(std::string moduleName,
                                                                   qtDataPath(std::move(qtDataPath)) {}
 
 bool BasicPluginsDeployer::deploy() {
-    for (const auto &pluginName : qtPluginsToBeDeployed()) {
-        ldLog() << "Deploying" << pluginName << "plugins" << std::endl;
+    // currently this is a no-op, but we might add more functionality later on, such as some kinds of default
+    // attempts to copy data based on the moduleName
+    return doDeploy();
+}
+
+bool BasicPluginsDeployer::deployStandardQtPlugins(const std::vector<std::string>& plugins)
+{
+    for (const auto &pluginName : plugins) {
+        ldLog() << "Deploying Qt" << pluginName << "plugins" << std::endl;
         for (fs::directory_iterator i(qtPluginsPath / pluginName); i != fs::directory_iterator(); ++i) {
             if (!appDir.deployLibrary(*i, appDir.path() / "usr/plugins" / pluginName))
                 return false;
         }
     }
-
-    return customDeploy();
-}
-
-bool BasicPluginsDeployer::customDeploy() {
     return true;
 }
 
-std::vector<std::string> BasicPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {};
+bool BasicPluginsDeployer::doDeploy() {
+    return true;
 }
diff --git a/src/deployers/BasicPluginsDeployer.h b/src/deployers/BasicPluginsDeployer.h
@@ -61,13 +61,13 @@ namespace linuxdeploy {
                  * name and path scheme, but some modules are special so
                  * they should write custom deployment code.
                  */
-                virtual bool customDeploy();
+                virtual bool doDeploy();
 
                 /**
                  * Returns a list of Qt plugin names that should be deployed and
                  * follow the default name and path scheme.
                  */
-                virtual std::vector<std::string> qtPluginsToBeDeployed() const;
+                bool deployStandardQtPlugins(const std::vector<std::string>& plugins);
             };
         }
     }
diff --git a/src/deployers/BearerPluginsDeployer.cpp b/src/deployers/BearerPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "BearerPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> BearerPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"bearer"};
+bool BearerPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"bearer"});
 }
diff --git a/src/deployers/BearerPluginsDeployer.h b/src/deployers/BearerPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/GamepadPluginsDeployer.cpp b/src/deployers/GamepadPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "GamepadPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> GamepadPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"gamepads"};
+bool GamepadPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"gamepads"});
 }
diff --git a/src/deployers/GamepadPluginsDeployer.h b/src/deployers/GamepadPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/LocationPluginsDeployer.cpp b/src/deployers/LocationPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "LocationPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> LocationPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"geoservices"};
+bool LocationPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"geoservices"});
 }
diff --git a/src/deployers/LocationPluginsDeployer.h b/src/deployers/LocationPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/Multimedia5PluginsDeployer.cpp b/src/deployers/Multimedia5PluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "Multimedia5PluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> Multimedia5PluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"mediaservice", "audio"};
+bool Multimedia5PluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"mediaservice", "audio"});
 }
diff --git a/src/deployers/Multimedia5PluginsDeployer.h b/src/deployers/Multimedia5PluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/Multimedia6PluginsDeployer.cpp b/src/deployers/Multimedia6PluginsDeployer.cpp
@@ -12,11 +12,11 @@ using namespace linuxdeploy::core::log;
 
 namespace fs = std::filesystem;
 
-std::vector<std::string> Multimedia6PluginsDeployer::qtPluginsToBeDeployed() const {
+bool Multimedia6PluginsDeployer::doDeploy() {
     if (fs::exists(qtPluginsPath / "multimedia")) {
-        return {"multimedia"};
+        return deployStandardQtPlugins({"multimedia"});
     } else {
         ldLog() << LD_WARNING << "Missing Qt 6 multimedia plugins, skipping." << std::endl;
-        return {};
+        return true;
     }
 }
diff --git a/src/deployers/Multimedia6PluginsDeployer.h b/src/deployers/Multimedia6PluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/PlatformPluginsDeployer.cpp b/src/deployers/PlatformPluginsDeployer.cpp
@@ -13,7 +13,7 @@ using namespace linuxdeploy::core::log;
 
 namespace fs = std::filesystem;
 
-bool PlatformPluginsDeployer::customDeploy() {
+bool PlatformPluginsDeployer::doDeploy() {
     ldLog() << "Deploying platform plugins" << std::endl;
 
     // always deploy default platform
@@ -30,6 +30,10 @@ bool PlatformPluginsDeployer::customDeploy() {
         }
     }
 
+    if (!deployStandardQtPlugins({"platforminputcontexts", "imageformats"})) {
+        return false;
+    }
+
     // TODO: platform themes -- https://github.com/probonopd/linuxdeployqt/issues/236
 
     const fs::path platformThemesPath = qtPluginsPath / "platformthemes";
@@ -68,7 +72,3 @@ bool PlatformPluginsDeployer::customDeploy() {
 
     return true;
 }
-
-std::vector<std::string> PlatformPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"platforminputcontexts", "imageformats"};
-}
diff --git a/src/deployers/PlatformPluginsDeployer.h b/src/deployers/PlatformPluginsDeployer.h
@@ -10,8 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                bool customDeploy() override;
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/PositioningPluginsDeployer.cpp b/src/deployers/PositioningPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "PositioningPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> PositioningPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"position"};
+bool PositioningPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"position"});
 }
diff --git a/src/deployers/PositioningPluginsDeployer.h b/src/deployers/PositioningPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/PrintSupportPluginsDeployer.cpp b/src/deployers/PrintSupportPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "PrintSupportPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> PrintSupportPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"printsupport"};
+bool PrintSupportPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"printsupport"});
 }
diff --git a/src/deployers/PrintSupportPluginsDeployer.h b/src/deployers/PrintSupportPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/QmlPluginsDeployer.cpp b/src/deployers/QmlPluginsDeployer.cpp
@@ -8,7 +8,7 @@ using namespace linuxdeploy::plugin::qt;
 
 namespace fs = std::filesystem;
 
-bool QmlPluginsDeployer::customDeploy() {
+bool QmlPluginsDeployer::doDeploy() {
     try {
         deployQml(appDir, qtInstallQmlPath);
     } catch (const QmlImportScannerError &) {
diff --git a/src/deployers/QmlPluginsDeployer.h b/src/deployers/QmlPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                bool customDeploy() override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/Qt3DPluginsDeployer.cpp b/src/deployers/Qt3DPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "Qt3DPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> Qt3DPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"geometryloaders", "sceneparsers"};
+bool Qt3DPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"geometryloaders", "sceneparsers"});
 }
diff --git a/src/deployers/Qt3DPluginsDeployer.h b/src/deployers/Qt3DPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/SqlPluginsDeployer.cpp b/src/deployers/SqlPluginsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "SqlPluginsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> SqlPluginsDeployer::qtPluginsToBeDeployed() const {
-    return {"sqldrivers"};
+bool SqlPluginsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"sqldrivers"});
 }
diff --git a/src/deployers/SqlPluginsDeployer.h b/src/deployers/SqlPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                std::vector<std::string> qtPluginsToBeDeployed() const override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/SvgPluginsDeployer.cpp b/src/deployers/SvgPluginsDeployer.cpp
@@ -12,7 +12,7 @@ using namespace linuxdeploy::core::log;
 
 namespace fs = std::filesystem;
 
-bool SvgPluginsDeployer::customDeploy() {
+bool SvgPluginsDeployer::doDeploy() {
     ldLog() << "Deploying svg icon engine" << std::endl;
 
     if (!appDir.deployLibrary(qtPluginsPath / "iconengines/libqsvgicon.so", appDir.path() / "usr/plugins/iconengines/"))
diff --git a/src/deployers/SvgPluginsDeployer.h b/src/deployers/SvgPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                bool customDeploy() override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/TextToSpeechPluginsDeployer.cpp b/src/deployers/TextToSpeechPluginsDeployer.cpp
@@ -12,7 +12,7 @@ using namespace linuxdeploy::core::log;
 
 namespace fs = std::filesystem;
 
-bool TextToSpeechPluginsDeployer::customDeploy() {
+bool TextToSpeechPluginsDeployer::doDeploy() {
     // calling the default code is optional, but it won't hurt for now
     if (!BasicPluginsDeployer::deploy())
         return false;
diff --git a/src/deployers/TextToSpeechPluginsDeployer.h b/src/deployers/TextToSpeechPluginsDeployer.h
@@ -10,7 +10,7 @@ namespace linuxdeploy {
                 // we can just use the base class's constructor
                 using BasicPluginsDeployer::BasicPluginsDeployer;
 
-                bool customDeploy() override;
+                bool doDeploy() override;
             };
         }
     }
diff --git a/src/deployers/TlsBackendsDeployer.cpp b/src/deployers/TlsBackendsDeployer.cpp
@@ -1,15 +1,11 @@
 // system headers
 #include <filesystem>
 
-// library headers
-#include <linuxdeploy/core/log.h>
-
 // local headers
 #include "TlsBackendsDeployer.h"
 
 using namespace linuxdeploy::plugin::qt;
-using namespace linuxdeploy::core::log;
 
-std::vector<std::string> TlsBackendsDeployer::qtPluginsToBeDeployed() const {
-    return {"tls"};
+bool TlsBackendsDeployer::doDeploy() {
+    return deployStandardQtPlugins({"tls"});
 }
diff --git a/src/deployers/TlsBackendsDeployer.h b/src/deployers/TlsBackendsDeployer.h
diff --git a/src/deployers/WaylandcompositorPluginsDeployer.cpp b/src/deployers/WaylandcompositorPluginsDeployer.cpp
diff --git a/src/deployers/WaylandcompositorPluginsDeployer.h b/src/deployers/WaylandcompositorPluginsDeployer.h
diff --git a/src/deployers/WebEnginePluginsDeployer.cpp b/src/deployers/WebEnginePluginsDeployer.cpp
diff --git a/src/deployers/WebEnginePluginsDeployer.h b/src/deployers/WebEnginePluginsDeployer.h
diff --git a/src/deployers/XcbglIntegrationPluginsDeployer.cpp b/src/deployers/XcbglIntegrationPluginsDeployer.cpp
diff --git a/src/deployers/XcbglIntegrationPluginsDeployer.h b/src/deployers/XcbglIntegrationPluginsDeployer.h

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ namespace linuxdeploy {`
`10`	`10`	`// we can just use the base class's constructor`
`11`	`11`	`using BasicPluginsDeployer::BasicPluginsDeployer;`
`12`	`12`
`13`		`- std::vector<std::string> qtPluginsToBeDeployed() const override;`
	`13`	`+ bool doDeploy() override;`
`14`	`14`	`};`
`15`	`15`	`}`
`16`	`16`	`}`