[Bindings.SofaGui] Fix compilation without compat layer (#331)

* fix config/compile w/o compat layer

* add cmake message when searching for guis

* remove useless dep sofa.gl

* Cosmectic change in bindings/SofaGui/src/SofaPython3/SofaGui/Module_SofaGui.cpp

Co-authored-by: Hugo <hugo.talbot@sofa-framework.org>

Author: fredroy

bindings/SofaGui/Bindings.SofaGuiConfig.cmake.in

@@ -5,9 +5,20 @@
 
 find_package(SofaPython3 QUIET REQUIRED COMPONENTS Plugin Bindings.Sofa)
 
-find_package(Sofa.GL QUIET REQUIRED)
 find_package(Sofa.Core QUIET REQUIRED)
-find_package(SofaGui QUIET REQUIRED)
+find_package(Sofa.Gui.Common QUIET REQUIRED)
+
+if(Sofa.GUI.Batch_FOUND)
+    find_package(Sofa.GUI.Batch QUIET REQUIRED)
+endif()
+
+if(Sofa.GUI.Qt_FOUND)
+    find_package(Sofa.GUI.Qt QUIET REQUIRED)
+endif()
+
+if(Sofa.GUI.HeadlessRecorder_FOUND)
+    find_package(Sofa.GUI.HeadlessRecorder QUIET REQUIRED)
+endif()
 
 # If we are importing this config file and the target is not yet there this is indicating that
 # target is an imported one. So we include it

bindings/SofaGui/CMakeLists.txt

@@ -13,16 +13,34 @@ set(HEADER_FILES
 )
 
 sofa_find_package(Sofa.Core REQUIRED)
-sofa_find_package(SofaGui REQUIRED)
-sofa_find_package(Sofa.GL REQUIRED)
+sofa_find_package(Sofa.GUI.Common REQUIRED) # to get the GUI mechanism
+
+sofa_find_package(Sofa.GUI.Batch QUIET)
+if(Sofa.GUI.Batch_FOUND)
+     list(APPEND SUPPORTED_GUIS Sofa.GUI.Batch)
+endif()
+sofa_find_package(Sofa.GUI.Qt QUIET)
+if(Sofa.GUI.Qt_FOUND)
+     list(APPEND SUPPORTED_GUIS Sofa.GUI.Qt)
+endif()
+sofa_find_package(Sofa.GUI.HeadlessRecorder QUIET)
+if(Sofa.GUI.HeadlessRecorder_FOUND)
+     list(APPEND SUPPORTED_GUIS Sofa.GUI.HeadlessRecorder)
+endif()
+
+if(SUPPORTED_GUIS)
+     message(STATUS "SofaPython3: Bindings.SofaGui will support: ${SUPPORTED_GUIS} .")
+else()
+     message(WARNING "SofaPython3: No GUIs detected.")
+endif()
 
 SP3_add_python_module(
     TARGET       ${PROJECT_NAME}
     MODULE       Gui
     DESTINATION  Sofa
     SOURCES      ${SOURCE_FILES}
     HEADERS      ${HEADER_FILES}
-    DEPENDS      Sofa.Core SofaGui SofaPython3::Plugin SofaPython3::Bindings.Sofa.Core
+    DEPENDS      Sofa.Core Sofa.GUI.Common SofaPython3::Plugin SofaPython3::Bindings.Sofa.Core ${SUPPORTED_GUIS}
 )
 
 sofa_create_component_in_package_with_targets(

bindings/SofaGui/src/SofaPython3/SofaGui/Module_SofaGui.cpp

@@ -20,16 +20,28 @@
 
 #include <pybind11/pybind11.h>
 
-#include <SofaGui/initSofaGui.h>
 #include <sofa/core/init.h>
 #include <sofa/helper/logging/Messaging.h>
 #include <sofa/helper/Utils.h>
 #include <sofa/helper/system/FileSystem.h>
 using sofa::helper::system::FileSystem;
 
-#if SOFAGUI_HAVE_SOFA_GUI_QT
+#if __has_include(<sofa/gui/batch/init.h>)
+#include <sofa/gui/batch/init.h>
+#define HAS_GUI_BATCH
+#endif
+
+#if __has_include(<sofa/gui/qt/init.h>)
+#include <sofa/gui/qt/init.h>
 #include <sofa/gui/qt/qt.conf.h>
-#endif // SOFAGUI_HAVE_SOFA_GUI_QT
+#define HAS_GUI_QT
+#endif
+
+#if __has_include(<sofa/gui/headlessrecorder/init.h>)
+#include <sofa/gui/headlessrecorder/init.h>
+#define HAS_GUI_HEADLESSRECORDER
+#endif
+
 
 #include "Binding_BaseGui.h"
 #include "Binding_GUIManager.h"
@@ -63,7 +75,7 @@ PYBIND11_MODULE(Gui, m) {
                     :members:
              )doc";
 
-#if SOFAGUI_HAVE_SOFA_GUI_QT
+#ifdef HAS_GUI_QT
     std::string sofaPrefixAbsolute = sofa::helper::Utils::getSofaPathPrefix();
     std::string inputFilepath = FileSystem::cleanPath(sofaPrefixAbsolute + "/bin/qt.conf");
     bool success = sofa::gui::qt::loadQtConfWithCustomPrefix(inputFilepath, sofaPrefixAbsolute);
@@ -82,11 +94,19 @@ PYBIND11_MODULE(Gui, m) {
         }
         std::cout << std::endl;
     }
-#endif // SOFAGUI_HAVE_SOFA_GUI_QT
+#endif // HAS_GUI_QT
+
+// forcefullly link libraries at compile-time
+#ifdef HAS_GUI_BATCH
+    sofa::gui::batch::init();
+#endif
+#ifdef HAS_GUI_QT
+    sofa::gui::qt::init();
+#endif
+#ifdef HAS_GUI_HEADLESSRECORDER
+    sofa::gui::headlessrecorder::init();
+#endif
 
-    // This is needed to make sure the GuiMain library (libSofaGuiMain.so) is correctly
-    // linked since the GUIs are statically created during the load of the library.
-    sofa::gui::initSofaGui();
     sofa::core::init();
 
     moduleAddBaseGui(m);