Hotfix: enclose executable paths in quotes

Render/renderers/Appleseed.py

@@ -1392,6 +1392,18 @@ def set_denoiser(root):
         )
         return root
 
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     # Here starts render
 
     # Make various adjustments on file:
@@ -1466,7 +1478,7 @@ def set_denoiser(root):
     with open(input_file, "wb") as f:
         f.write(template)
 
-    # Prepare command line parameters
+    # Prepare command line parameters and rpath
     params = App.ParamGet("User parameter:BaseApp/Preferences/Mod/Render")
     if not batch:
         # GUI
@@ -1489,6 +1501,7 @@ def set_denoiser(root):
             "Edit -> Preferences -> Render\n"
         )
         return None, None
+    rpath = enclose_rpath(rpath)
     if args:
         args += " "
 

Render/renderers/Cycles.py

@@ -954,6 +954,19 @@ def render(
         The command to run renderer (string)
         A path to output image file (string)
     """
+
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     # Denoise
     if denoise:
         tree = et.parse(input_file)
@@ -984,6 +997,7 @@ def render(
             "Edit -> Preferences -> Render\n"
         )
         return None, None
+    rpath = enclose_rpath(rpath)
     args += " --width " + str(width)
     args += " --height " + str(height)
     filepath = f'"{input_file}"'

Render/renderers/Luxcore.py

@@ -649,6 +649,18 @@ def export_section(section, prefix, suffix):
             output.write("\n".join(result))
         return f_path
 
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     # LuxCore requires 2 files:
     # - a configuration file, with rendering parameters (engine, sampler...)
     # - a scene file, with the scene objects (camera, lights, meshes...)
@@ -735,7 +747,7 @@ def export_section(section, prefix, suffix):
     scene = pageresult["Scene"]
     scn_path = export_section(scene, project.Name, "scn")
 
-    # Get rendering parameters
+    # Get rendering parameters and rpath
     params = App.ParamGet("User parameter:BaseApp/Preferences/Mod/Render")
     args = params.GetString("LuxCoreParameters", "")
     rpath = params.GetString(
@@ -748,6 +760,7 @@ def export_section(section, prefix, suffix):
         )
         App.Console.PrintError(msg)
         return None, None
+    rpath = enclose_rpath(rpath)
 
     # Prepare command line and return
     cmd = f"""{prefix}{rpath} {args} -o "{cfg_path}" -f "{scn_path}"\n"""

Render/renderers/Ospray.py

@@ -1041,6 +1041,19 @@ def render(
         The command to run renderer (string)
         A path to output image file (string)
     """
+
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     # Read scene_graph (json)
     with open(input_file, "r", encoding="utf8") as f:
         scene_graph = json.load(f)
@@ -1107,6 +1120,7 @@ def render(
             "Edit -> Preferences -> Render\n"
         )
         return None, None
+    rpath = enclose_rpath(rpath)
 
     cmd = prefix + rpath + " " + args + " " + f'"{input_file}"'
 

Render/renderers/Pbrt.py

@@ -652,6 +652,19 @@ def render(
         The command to run renderer (string)
         A path to output image file (string)
     """
+
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     # Make various adjustments on file:
     # Reorder camera declarations and set width/height
     with open(input_file, "r", encoding="utf-8") as f:
@@ -703,6 +716,7 @@ def render(
             "Edit -> Preferences -> Render\n"
         )
         return None, None
+    rpath = enclose_rpath(rpath)
 
     filepath = f'"{input_file}"'
 

Render/renderers/Povray.py

@@ -831,6 +831,19 @@ def render(
         The command to run renderer (string)
         A path to output image file (string)
     """
+
+    def enclose_rpath(rpath):
+        """Enclose rpath in quotes, if needed."""
+        if not rpath:
+            return ""
+        if rpath[0] == rpath[-1] == '"':
+            # Already enclosed (double quotes)
+            return rpath
+        if rpath[0] == rpath[-1] == "'":
+            # Already enclosed (simple quotes)
+            return rpath
+        return f'"{rpath}"'
+
     params = App.ParamGet("User parameter:BaseApp/Preferences/Mod/Render")
 
     prefix = params.GetString("Prefix", "")
@@ -845,6 +858,7 @@ def render(
             "Edit -> Preferences -> Render\n"
         )
         return None, None
+    rpath = enclose_rpath(rpath)
 
     # Prepare command line parameters
     args = params.GetString("PovRayParameters", "")