Replace QPluginLoader with QLibrary, should allow to create statically linked plugins

cmake/functions.cmake

@@ -34,8 +34,12 @@ function(add_driver)
     set(disabled_drivers ${disabled_drivers} ${add_driver_NAME} CACHE INTERNAL "")
     return()
   endif()
+  configure_file(${CMAKE_CURRENT_SOURCE_DIR}/driver.json ${CMAKE_CURRENT_BINARY_DIR}/driver_${add_driver_NAME}.json)
   add_library(${add_driver_NAME} MODULE ${add_driver_SRCS})
   target_link_libraries(${add_driver_NAME} GuLinux_Qt_Commons GuLinux_c++_Commons drivers planetaryimager-commons ${add_driver_LINK} Qt5::Core Qt5::Qml ${OpenCV_LIBS})
+
+  set_target_properties(${add_driver_NAME} PROPERTIES PREFIX "driver_")
   install(TARGETS ${add_driver_NAME} LIBRARY DESTINATION ${drivers_destination})
+  install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${add_driver_NAME}.json DESTINATION ${drivers_destination})
   set(enabled_drivers ${enabled_drivers} ${add_driver_NAME} CACHE INTERNAL "")
 endfunction()

src/drivers/driver.h

@@ -38,10 +38,15 @@ class Driver : public QObject {
   virtual Cameras cameras() const = 0;
 };
 typedef QList<Driver::ptr> Drivers;
-#define DRIVER_IID "net.gulinux.planetaryimager.driver"
-Q_DECLARE_INTERFACE(Driver, DRIVER_IID)
-#define DECLARE_DRIVER_PLUGIN   \
-Q_INTERFACES(Driver) \
-Q_PLUGIN_METADATA(IID DRIVER_IID FILE "driver.json")
+typedef Driver *(*LoadDriverFunction)();
+
+#ifdef Q_OS_WIN
+#define EXPORT_DECL __declspec(dllexport)
+#else
+#define EXPORT_DECL
+#endif
+
+#define PLANETARY_IMAGER_DRIVER_LOAD_F "PlanetaryImager_loadDriver"
+#define DECLARE_DRIVER_PLUGIN_INIT(DriverClass) extern "C" EXPORT_DECL Driver *PlanetaryImager_loadDriver() { return new DriverClass(); }
 
 #endif

src/drivers/flycapture2/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "FlyCapture2",
   "description": "FLIR/Point Grey native API",
-  "version": "0.8"
+  "version": "${FULL_VERSION}"
 }

src/drivers/flycapture2/fc2_driver.cpp

@@ -108,3 +108,5 @@ FC2Driver::FC2Driver(): dptr()
 FC2Driver::~FC2Driver()
 {
 }
+
+DECLARE_DRIVER_PLUGIN_INIT(FC2Driver)

src/drivers/flycapture2/fc2_driver.h

@@ -28,8 +28,6 @@ class FC2Driver: public Driver
 
 public:
 
-    DECLARE_DRIVER_PLUGIN
-
     FC2Driver();
 
     ~FC2Driver();

src/drivers/iidc/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "IIDC",
   "description": "Instrumentation & Industrial Digital Camera, a standard for camera control over FireWire and USB",
-  "version": "0.8"
+  "version": "${FULL_VERSION}"
 }

src/drivers/iidc/iidc_driver.cpp

@@ -138,3 +138,6 @@ IIDCDriver::IIDCDriver(): dptr()
 }
 
 IIDCDriver::~IIDCDriver() { }
+
+
+DECLARE_DRIVER_PLUGIN_INIT(IIDCDriver)

src/drivers/iidc/iidc_driver.h

@@ -28,8 +28,6 @@ class IIDCDriver: public Driver
 
 public:
 
-    DECLARE_DRIVER_PLUGIN
-
     IIDCDriver();
 
     ~IIDCDriver();

src/drivers/qhy/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "QHY",
   "description": "QHYCCD Cameras driver",
-  "version": "0.7"
+  "version": "${FULL_VERSION}"
 }

src/drivers/qhy/qhydriver.cpp

@@ -82,3 +82,4 @@ Driver::Cameras QHYDriver::cameras() const
   return cameras;
 }
 
+DECLARE_DRIVER_PLUGIN_INIT(QHYDriver)

src/drivers/qhy/qhydriver.h

@@ -25,7 +25,6 @@ class QHYDriver : public Driver
 {
   Q_OBJECT
 public:
-  DECLARE_DRIVER_PLUGIN
     QHYDriver();
     virtual ~QHYDriver();
     Cameras cameras() const override;

src/drivers/simulator/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "Simulator",
   "description": "Various simulator imagers, useful mostly for debugging",
-  "version": "0.7"
+  "version": "${FULL_VERSION}"
 }

src/drivers/simulator/simulatordriver.cpp

@@ -65,4 +65,5 @@ SimulatorDriver::SimulatorDriver()
 
 
 
+DECLARE_DRIVER_PLUGIN_INIT(SimulatorDriver)
 #include "simulatordriver.moc"

src/drivers/simulator/simulatordriver.h

@@ -25,9 +25,9 @@ class SimulatorDriver : public Driver
 {
   Q_OBJECT
 public:
-  DECLARE_DRIVER_PLUGIN
   SimulatorDriver();
   virtual Driver::Cameras cameras() const;
 };
 
+
 #endif // SIMULATORDRIVER_H

src/drivers/supporteddrivers.cpp

@@ -17,7 +17,7 @@
  */
 
 #include "supporteddrivers.h"
-#include <QPluginLoader>
+#include <QLibrary>
 #include <QDebug>
 #include <QDirIterator>
 #include <QFile>
@@ -30,7 +30,8 @@ using namespace std;
 
 DPTR_IMPL(SupportedDrivers) {
   SupportedDrivers *q;
-  QList<shared_ptr<QPluginLoader>> drivers;
+  QList<shared_ptr<QLibrary>> drivers;
+
   void find_drivers(const QString &directory);
   void load_driver(const QString &filename);
   list<Driver*> instances() const;
@@ -57,18 +58,25 @@ void SupportedDrivers::Private::find_drivers(const QString& directory)
 
 void SupportedDrivers::Private::load_driver(const QString& filename)
 {
-  auto plugin = make_shared<QPluginLoader>(filename);
-  qDebug() << "trying " << filename << ": " << plugin->metaData();
-  auto getClassName = [](const auto &p) { return p->metaData().value("className"); };
-  if(plugin->metaData().value("IID").toString() == DRIVER_IID && 
-        find_if(drivers.begin(), drivers.end(), [&](const auto &p) { return getClassName(p) == getClassName(plugin); }) == drivers.end() ) {
-      if(plugin->load()) {
-        auto metadata = QJsonDocument{plugin->metaData()}.toVariant().toMap();
-        qInfo() << "driver " << plugin->fileName() << "loaded:" << metadata["className"].toString() << metadata["MetaData"].toMap()["description"].toString();
-        drivers.push_back(plugin);
-      } else {
-        qWarning() << "Error loading driver " << plugin->fileName() << ": " << plugin->errorString();
-      }
+  if(!filename.startsWith("driver_") && ! filename.endsWith(".json")) {
+    return;
+  }
+  QString library_name = filename;
+  library_name.remove(".json");
+
+  qDebug() << "trying " << filename << ": " << library_name;
+
+  auto driver = make_shared<QLibrary>(library_name);
+  if(driver->load()) {
+    qDebug() << "Driver " << library_name << " loaded successfully";
+    if(driver->resolve(PLANETARY_IMAGER_DRIVER_LOAD_F)) {
+        qDebug() << "PlanetaryImager_loadDriver resolved on " << library_name;
+        drivers.push_back(driver);
+    } else {
+        qWarning() << "Error resolving PlanetaryImager_loadDriver function on " << library_name;
+    }
+  } else {
+    qWarning() << "Error loading driver: " << library_name;
   }
 }
 
@@ -101,7 +109,8 @@ list<Driver *> SupportedDrivers::Private::instances() const
   transform(begin(drivers), end(drivers), back_inserter(instances), [](const auto &p) -> Driver* {
     qDebug() << "Initializing driver" << p->fileName();
     try {
-      return qobject_cast<Driver*>(p->instance());
+      auto driver_load_function = (LoadDriverFunction) p->resolve(PLANETARY_IMAGER_DRIVER_LOAD_F);
+      return qobject_cast<Driver*>(driver_load_function());
     } catch(const Imager::exception &e) {
       qWarning() << "Error loading driver: " << e.what();
       return nullptr;

src/drivers/v4l2/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "V4L2",
   "description": "V4L2 Devices driver (webcams, The Imaging Soure, Celestron Skyris",
-  "version": "0.7"
+  "version": "${FULL_VERSION}"
 }

src/drivers/v4l2/v4l2driver.cpp

@@ -66,3 +66,4 @@ Driver::Cameras V4L2Driver::cameras() const
 }
 
 
+DECLARE_DRIVER_PLUGIN_INIT(V4L2Driver)

src/drivers/v4l2/v4l2driver.h

@@ -26,7 +26,6 @@ class V4L2Driver : public Driver
 {
   Q_OBJECT
 public:
-  DECLARE_DRIVER_PLUGIN
 V4L2Driver();
 ~V4L2Driver();
 virtual Driver::Cameras cameras() const;

src/drivers/zwo_asi/driver.json

@@ -1,5 +1,5 @@
 {
   "name": "ASI",
   "description": "ZWO ASI CMOS Cameras driver",
-  "version": "0.7"
+  "version": "${FULL_VERSION}"
 }

src/drivers/zwo_asi/zwo_asi_driver.cpp

@@ -76,3 +76,5 @@ Driver::Cameras ZWO_ASI_Driver::cameras() const
   }
   return cameras;
 }
+
+DECLARE_DRIVER_PLUGIN_INIT(ZWO_ASI_Driver)

src/drivers/zwo_asi/zwo_asi_driver.h

@@ -27,7 +27,6 @@ class ZWO_ASI_Driver : public Driver
 {
   Q_OBJECT
 public:
-  DECLARE_DRIVER_PLUGIN
     ZWO_ASI_Driver();
     ~ZWO_ASI_Driver();
     virtual Driver::Cameras cameras() const;