Fix of space in process path

(references #1291)

(co-authored-by: Armel Thöni)

Author: DorianGerardin

CMake.in.cmake

@@ -23,7 +23,7 @@ SET(OFBUILD_CUSTOM_CMAKE_VERSION "${CMAKE_MAJOR_VERSION}.${CMAKE_MINOR_VERSION}.
 SET(OPENFLUID_VERSION_MAJOR 2)
 SET(OPENFLUID_VERSION_MINOR 2)
 SET(OPENFLUID_VERSION_PATCH 0)
-SET(OPENFLUID_VERSION_STATUS "beta3") # example: SET(OPENFLUID_VERSION_STATUS "rc1")
+SET(OPENFLUID_VERSION_STATUS "beta4") # example: SET(OPENFLUID_VERSION_STATUS "rc1")
 
 SET(OPENFLUID_VERSION_FULL "${OPENFLUID_VERSION_MAJOR}.${OPENFLUID_VERSION_MINOR}.${OPENFLUID_VERSION_PATCH}")
 

src/openfluid/fluidx/SpatialDomainDescriptor.cpp

@@ -312,7 +312,7 @@ void SpatialDomainDescriptor::addUnit(const openfluid::fluidx::SpatialUnitDescri
   {
     throw openfluid::base::FrameworkException(OPENFLUID_CODE_LOCATION,
                                               "unable to add spatial unit " +
-                                              openfluid::tools::classIDToString(ClassName,ID) + "that already exists");
+                                              openfluid::tools::classIDToString(ClassName,ID) + " that already exists");
   }
 
   auto itInserted = (m_SpatialUnits[ClassName].insert(std::make_pair(ID, UnitDesc))).first;

src/openfluid/utils/Process.cpp

@@ -169,7 +169,13 @@ bool Process::run()
     {
         WorkDir = m_Cmd.WorkDir;
     }
-    boost::process::child BPC(boost::process::exe = m_Cmd.Program,
+    std::string Program = m_Cmd.Program;
+    // Process with space must be surrounded by quotes (works for windows and linux)
+    if ((Program.find(' ') != std::string::npos) && (Program[0] != '"'))
+    {
+        Program = "\""+Program+"\"";
+    }
+    boost::process::child BPC(boost::process::exe = Program,
                               boost::process::args = m_Cmd.Args,
                               boost::process::start_dir = WorkDir,
                               boost::process::std_out > StdOutStr, boost::process::std_err > StdErrStr,