From d006955bea26c16a357ef30b16e838de4dc1b0f3 Mon Sep 17 00:00:00 2001
From: YOANN AUDOUIN <B61570@dsp1062659>
Date: Wed, 13 Dec 2023 11:15:08 +0100
Subject: [PATCH] Better error handling for parallel mesh

---
 src/SMESH/SMESH_ParallelMesh.cxx  | 2 --
 src/SMESH_SWIG/mesher_launcher.py | 5 +++--
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/SMESH/SMESH_ParallelMesh.cxx b/src/SMESH/SMESH_ParallelMesh.cxx
index 2bce451f3..4b49486e0 100644
--- a/src/SMESH/SMESH_ParallelMesh.cxx
+++ b/src/SMESH/SMESH_ParallelMesh.cxx
@@ -61,7 +61,6 @@ SMESH_ParallelMesh::~SMESH_ParallelMesh()
 void SMESH_ParallelMesh::cleanup()
 {
   DeletePoolThreads();
-  std::cout << "Keeping tmp folder" << keepingTmpFolfer() << std::endl;
   if(!keepingTmpFolfer())
   {
     MESSAGE("Set SMESH_KEEP_TMP to > 0 to keep temporary folders")
@@ -78,7 +77,6 @@ void SMESH_ParallelMesh::cleanup()
 bool SMESH_ParallelMesh::keepingTmpFolfer()
 {
   const char* envVar = std::getenv("SMESH_KEEP_TMP");
-  std::cout << "smesh_keep_tmp: " << envVar << std::endl;
 
   if (envVar && (envVar[0] != '\0'))
   {
diff --git a/src/SMESH_SWIG/mesher_launcher.py b/src/SMESH_SWIG/mesher_launcher.py
index 65de14925..3794016df 100644
--- a/src/SMESH_SWIG/mesher_launcher.py
+++ b/src/SMESH_SWIG/mesher_launcher.py
@@ -244,11 +244,12 @@ def run_pylauncher(args):
     del_tmp_folder = True
     try:
        val = int(environ.get("SMESH_KEEP_TMP", "0"))
-       del_tmp_folder = val > 0
+       del_tmp_folder = val < 0
     except Exception as e:
         del_tmp_folder = True
 
-    launcher.clearJobWorkingDir(job_id)
+    if del_tmp_folder:
+        launcher.clearJobWorkingDir(job_id)
 
 def def_arg():
     """ Define and parse arguments for the script """