]> SALOME platform Git repositories - tools/sat.git/commitdiff
Salome HOME
feat(all): fix common issues (exept Exception, missing import for gettext.gettext...
authorGbkng <guillaume.brooking@gmail.com>
Wed, 14 Feb 2024 22:40:56 +0000 (23:40 +0100)
committerGbkng <guillaume.brooking@gmail.com>
Wed, 14 Feb 2024 22:43:15 +0000 (23:43 +0100)
54 files changed:
AllTestLauncherSat.py
commands/application.py
commands/check.py
commands/clean.py
commands/compile.py
commands/config.py
commands/configure.py
commands/doc.py
commands/environ.py
commands/find_duplicates.py
commands/generate.py
commands/init.py
commands/install.py
commands/job.py
commands/jobs.py
commands/launcher.py
commands/log.py
commands/make.py
commands/makeinstall.py
commands/package.py
commands/patch.py
commands/prepare.py
commands/run.py
commands/script.py
commands/shell.py
commands/source.py
commands/template.py
commands/test.py
data/local.pyconf
data/templates/Application/config/splash_generator/splash2.py
data/templates/PythonComponent/src/PYCMP/PYCMP_utils.py
data/templates/PythonComponent8/src/PYCMP/PYCMP_utils.py
src/ElementTree.py
src/ElementTreePython2.py
src/ElementTreePython3.py
src/architecture.py
src/callerName.py
src/compilation.py
src/debug.py
src/environment.py
src/fileEnviron.py
src/fork.py
src/i18n/i18nTest.py
src/logger.py
src/loggingSimple.py
src/options.py
src/product.py
src/pyconf.py
src/returnCode.py
src/salomeTools.py
src/system.py
src/test_module.py
src/versionMinorMajorPatch.py
src/xmlManager.py

index eaa1a967b5054a52b30297dfd3095c4ea38cc061..cf8319d96a0eebbc99c3801f22fe165dc404bf74 100755 (executable)
@@ -86,7 +86,7 @@ def errPrint(aStr):
 
 try:
   import unittestpy.HTMLTestRunner as HTST
-except:
+except Exception:
   HTST = None
   errPrint("""
 WARNING: no HTML output available.
@@ -96,7 +96,7 @@ WARNING: no HTML output available.
 
 try:
   import xmlrunner as XTST
-except:
+except Exception:
   XTST = None
   errPrint("""
 WARNING: no XML output available for unittest.
index 025baf3077a79443c5f4face376680e2bceaeb55..1620b64db44b840f09935f03e12d8cefc6bc845c 100644 (file)
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
-import stat
 import sys
 import shutil
 import subprocess
 import getpass
+from gettext import gettext as _
 
 from src import ElementTree as etree
 import src
@@ -155,7 +155,7 @@ def customize_app(config, appli_dir, logger):
         if text is not None:
             try:
                 n.text = text.strip("\n\t").decode("UTF-8")
-            except:
+            except Exception:
                 sys.stderr.write("################ %s %s\n" % (node_name, text))
                 n.text = "?"
         parent.append(n)
index 366a1371e05b7e68591235f1c66a0467eb4e9d55..2cfd29f700e63af77e60fc7533c77e63d4131a3a 100644 (file)
@@ -17,8 +17,8 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
-
 import src
+from gettext import gettext as _
 
 # Define all possible option for the check command :  sat check <options>
 parser = src.options.Options()
index 070b87517cbed510c3d216b23ed1ca82005c51a5..4cd38d298eca01f90429f1c5f1c5c2b4fccf8ed6 100644 (file)
 #  License along with this library; if not, write to the Free Software
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
-import re
 import os
-
 import src
+from gettext import gettext as _
 
 # Compatibility python 2/3 for input function
 # input stays input for python 3 and input = raw_input for python 2
index ded2297cc312acf04b14d97123096ed75968d20a..4308bdd45aafb7ce200e84f3a2c425ed91ac382d 100644 (file)
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
-import re
 import subprocess
 import src
 import src.debug as DBG
+from gettext import gettext as _
 
 # Compatibility python 2/3 for input function
 # input stays input for python 3 and input = raw_input for python 2
@@ -229,7 +229,7 @@ def compile_all_products(sat, config, options, products_infos, all_products_dict
                                   batch=True,
                                   verbose=0,
                                   logger_add_link = logger)
-                except:
+                except Exception:
                     pass
 
     if check_salome_configuration:
@@ -546,7 +546,7 @@ def compile_product_pip(sat,
                                env=build_environ.environ.environ,
                                stderr=subprocess.STDOUT).strip()
         pip_build_options=int(res_pip_version.split('.')[0]) < 21
-    except:
+    except Exception:
         pip_build_options= True
     # d- install (in python or in separate product directory)
     if src.appli_test_property(config,"pip_install_dir", "python"):
index a5997bb7814fcc265dbeef9d3df3413f44d0eb58..4e8836b4f64b89eeaf8d7827d7057b19f81ad9c0 100644 (file)
@@ -23,10 +23,11 @@ import datetime
 import shutil
 import gettext
 import pprint as PP
+from gettext import gettext as _
 
 import src
 import src.logger as LOG
-import src.debug as DBG
+import src.debug as debug
 import src.callerName as CALN
 
 logger = LOG.getDefaultLogger()
@@ -1068,7 +1069,7 @@ def get_config_children(config, args):
                 if dir(a).__contains__('keys'):
                     vals = map(lambda x: head + '.' + x,
                                [m for m in a.keys() if m.startswith(tail)])
-            except:
+            except Exception:
                 pass
 
     for v in sorted(vals):
@@ -1110,7 +1111,7 @@ def run(args, runner, logger):
     if options.debug:
         if options.debug == ".":
             # if argument is ".", print all the config
-            res = DBG.indent(DBG.getStrConfigDbg(runner.cfg))
+            res = debug.indent(debug.getStrConfigDbg(runner.cfg))
             logger.write("\nConfig of application %s:\n\n%s\n" % (runner.cfg.VARS.application, res))
         else:
             if options.debug[0] == ".": # accept ".PRODUCT.etc" as "PRODUCT.etc"
@@ -1124,7 +1125,7 @@ def run(args, runner, logger):
               exec(aCode, globals(), aDict)
               # DBG.write("globals()", globals(), True)
               # DBG.write("aDict", aDict, True)
-              res = DBG.indent(DBG.getStrConfigDbg(aDict["a"]))
+              res = debug.indent(debug.getStrConfigDbg(aDict["a"]))
               logger.write("\nConfig.%s of application %s:\n\n%s\n" % (od, runner.cfg.VARS.application, res))
             except Exception as e:
               msg = "\nConfig.%s of application %s: Unknown pyconf key\n" % (od, runner.cfg.VARS.application)
@@ -1160,7 +1161,7 @@ def run(args, runner, logger):
       # DBG.write("products", sorted(runner.cfg.APPLICATION.products.keys()), True)
       src.check_config_has_application(runner.cfg)
       taggedProducts = src.getProductNames(runner.cfg, options.info, logger)
-      DBG.write("tagged products", sorted(taggedProducts))
+      debug.write("tagged products", sorted(taggedProducts))
       for prod in sorted(taggedProducts):
         if prod in runner.cfg.APPLICATION.products:
           try:
index 876e928f189cc0777c3dd7d2fdc83d7cf80ec1e6..ac1ac260ef8bf87efaeb11fdaa2cd1b596a92da9 100644 (file)
@@ -17,8 +17,8 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
-
 import src
+from gettext import gettext as _
 
 # Define all possible option for configure command :  sat configure <options>
 parser = src.options.Options()
index 0c9b44881b807698f208719607bd97816135b85d..a776785721b990363ca53ad9c0c499b0820cdcbe 100644 (file)
@@ -18,6 +18,7 @@
 
 import os
 import src
+from gettext import gettext as _
 
 # Define all possible option for log command :  sat doc <options>
 parser = src.options.Options()
index 7b2676354df69d4307767c47b49a5bfd55cbdbd9..41f8817459a6e30c804fecc4a05a20cad97e0c53 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 
index 56e7143135c2284c43020c2dcdfba717d1948aed..e271f5407c6bff43b702360b6d38ac90be19a1ee 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 
index fbd08aa5e57271f44df9b46c69f58eab60b02141..bbea5f7fd17bc8a99675fb6358a46c26bef9a7b6 100644 (file)
@@ -21,10 +21,11 @@ import sys
 import shutil
 import imp
 import subprocess
+from gettext import gettext as _
 
 import src
 from  src.versionMinorMajorPatch import MinorMajorPatch as MMP
-import src.debug as DBG
+import src.debug as debug
 
 parser = src.options.Options()
 parser.add_option('p', 'products', 'list2', 'products',
@@ -147,7 +148,7 @@ def generate_component(config, compo, product_name, product_info, context, heade
             # get files to build a template GUI
             try: # try new yacsgen api
                 gui_files = salome_compo.getGUIfilesTemplate(compo)
-            except:  # use old yacsgen api
+            except Exception:  # use old yacsgen api
                 gui_files = salome_compo.getGUIfilesTemplate()
         else:
             gui_files = None
index 592843a5163a208d0f754ec4266f64efb84f6e15..7cea38ac9b5752419b9f45bfab5cdee778a1e241 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 
index 2811a6af8c4dd7648c2f48034a48f26335f17930..ce056c63f026c201f40ae26e3bfca79c75000c5a 100644 (file)
 
 import os
 import shutil
-import re
-import subprocess
+from gettext import gettext as _
 
 import src
-import prepare
-import src.debug as DBG
+import src.debug as debug
 
 PACKAGE_EXT=".tar.gz" # the extension we use for the packages
 
@@ -219,7 +217,7 @@ def description():
 def run(args, runner, logger):
     '''method that is called when salomeTools is called with install parameter.
     '''
-    DBG.write("install.run()", args)
+    debug.write("install.run()", args)
     # Parse the options
     (options, args) = parser.parse_args(args)
     
@@ -240,7 +238,7 @@ def run(args, runner, logger):
     from compile import get_dependencies_graph,depth_search_graph
     all_products_graph=get_dependencies_graph(all_products_infos)
     #logger.write("Dependency graph of all application products : %s\n" % all_products_graph, 6)
-    DBG.write("Dependency graph of all application products : ", all_products_graph)
+    debug.write("Dependency graph of all application products : ", all_products_graph)
 
     products_infos=[]
     if options.products is None:
index f5805feaa557dcaf65007cdf09ed75c8a5f1ef6d..27980ec2d340c2b5735d0508c9601bd84590472b 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 import src.salomeTools
index c043d7048e094d8463f6f40b6f9f26b9a6c3c7dc..1e7ec65895f8bca4d9faee459fc60dc0f8b20f38 100644 (file)
@@ -26,11 +26,12 @@ import csv
 import shutil
 import itertools
 import re
+from gettext import gettext as _
 
 # generate problem
 try:
   import paramiko
-except:
+except Exception:
   paramiko = "import paramiko impossible"
   pass
 
@@ -114,7 +115,7 @@ class Machine(object):
         except paramiko.SSHException:
             message = ( _("SSHException error connecting or "
                           "establishing an SSH session"))            
-        except:
+        except Exception:
             message = ( _("Error connecting or establishing an SSH session"))
         else:
             self._connection_successful = True
@@ -215,7 +216,7 @@ class Machine(object):
                             ": the server failed to execute the command\n")
             logger.write( src.printcolors.printcError(message))
             return (None, None, None)
-        except:
+        except Exception:
             logger.write( src.printcolors.printcError(src.KO_STATUS + '\n'))
             return (None, None, None)
         else:
@@ -327,7 +328,7 @@ class Job(object):
         '''
         try:
             pids = self.get_pids()
-        except:
+        except Exception:
             return ("Unable to get the pid of the command.", "")
             
         cmd_kill = " ; ".join([("kill -2 " + pid) for pid in pids])
index ea7cc82136738a343d63a2ecde9c1becc34f72f4..1dac9a4c41bad94d04ecd082e00ff06d3324bfd9 100644 (file)
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
-import platform
 import shutil
 import getpass
 import subprocess
 import stat
+from gettext import gettext as _
 
 import src
-import src.debug as DBG
 
 parser = src.options.Options()
 
index 22a788570fc8fa85895c4ea82891a55e1b865ba6..0fe9785830dd59effcdfdcfcd5b212b39639edf6 100644 (file)
@@ -22,6 +22,7 @@ import re
 import glob
 import datetime
 import stat
+from gettext import gettext as _
 
 # Compatibility python 2/3 for input function
 # input stays input for python 3 and input = raw_input for python 2
@@ -181,7 +182,7 @@ def ask_value(nb):
             x = int(rep)
             if x > nb:
                 x = -1
-    except:
+    except Exception:
         x = -1
     
     return x
@@ -269,7 +270,7 @@ def run(args, runner, logger):
       src.ensure_path_exists(os.path.join(logDir, "TEST"))
       shutil.copy(xsltest, os.path.join(logDir, "TEST"))
       shutil.copy(imgLogo, logDir)
-    except:
+    except Exception:
       # we are here  if an user make sat log in jenkins LOGS without write rights
       # Make a warning and do nothing
       logger.warning("problem for writing in directory '%s', may be not owner." % logDir)
@@ -338,7 +339,7 @@ def run(args, runner, logger):
                               notShownCommands = notShownCommands)
 
       logger.write(src.printcolors.printc("OK"), 3)
-    except:
+    except Exception:
       logger.write(src.printcolors.printc("KO"), 3)
       logger.write(" problem update hat.xml", 3)
 
index 39f562a100e94d500bdf68572b9a93e8f8d2b762..c1fa8b30ce9a3908c9b8da5b4d337454407a5dbf 100644 (file)
@@ -18,6 +18,7 @@
 
 import os
 import re
+from gettext import gettext as _
 
 import src
 
index eea3f85b7615712108646a33b69652c3e7777c53..69efa09460f1c46ecee705fac375e351433c0472 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 
index f91476e2d6b6b87c7027b2a4ba37431b0af6de62..2bfbef3615f921300536b34e8aadd3469da4d0db 100644 (file)
@@ -24,13 +24,15 @@ import tarfile
 import codecs
 import string
 import glob
-import pprint as PP
 import sys
+from gettext import gettext as _
+
+import pprint as PP
 import src
 
-from application import get_SALOME_modules
 from  src.versionMinorMajorPatch import MinorMajorPatch as MMP
-import src.debug as DBG
+import src.debug as debug
+from commands.application import get_SALOME_modules
 
 old_python = sys.version_info[0] == 2 and sys.version_info[1] <= 6
 
@@ -166,7 +168,7 @@ def add_files(tar, name_archive, d_content, logger, f_exclude=None):
     success = 0
     # loop over each directory or file stored in the d_content dictionary
     names = sorted(d_content.keys())
-    DBG.write("add tar names", names)
+    debug.write("add tar names", names)
 
     # used to avoid duplications (for pip install in python, or single_install_dir cases)
     already_added=set()
@@ -385,7 +387,7 @@ def hack_for_distene_licence(filepath, licence_file):
             import imp
             distene = imp.load_source('distene_licence', distene_licence_file)
         distene.set_distene_variables(context)
-    except:
+    except Exception:
         pass\n"""  % licence_file
     text.insert(num_line + 1, text_to_insert)
     for line in text:
@@ -948,7 +950,7 @@ def source_package(sat, config, logger, options, tmp_working_dir):
     if not src.architecture.is_windows():
         try:
             t = os.getcwd()
-        except:
+        except Exception:
             # In the jobs, os.getcwd() can fail
             t = config.LOCAL.workdir
         os.chdir(tmp_working_dir)
@@ -1326,7 +1328,7 @@ def find_product_scripts_and_pyconf(p_name,
             sections = [p_info.section]
         for section in sections:
             if section in product_pyconf_cfg and "get_source" in product_pyconf_cfg[section]:
-                DBG.write("sat package set archive mode to archive for product %s and section %s" %\
+                debug.write("sat package set archive mode to archive for product %s and section %s" %\
                           (p_name,section))
                 product_pyconf_cfg[section].get_source = "archive"
                 if not "archive_info" in product_pyconf_cfg[section]:
@@ -1447,7 +1449,7 @@ def project_package(config, name_project, project_file_path, ftp_mode, tmp_worki
 
     try:
       project_pyconf_cfg = config.PROJECTS.projects.__getattr__(name_project)
-    except:
+    except Exception:
       logger.write("""
 WARNING: inexisting config.PROJECTS.projects.%s, try to read now from:\n%s\n""" % (name_project, project_file_path))
       project_pyconf_cfg = src.pyconf.Config(project_file_path)
@@ -1846,7 +1848,7 @@ Please add it in file:
                                   options, logger))
 
     if options.project:
-        DBG.write("config for package %s" % options.project, runner.cfg)
+        debug.write("config for package %s" % options.project, runner.cfg)
         d_files_to_add.update(project_package(runner.cfg, options.project, options.project_file_path, options.ftp, tmp_working_dir, options.sat, logger))
 
     if not(d_files_to_add):
@@ -1899,7 +1901,7 @@ Please add it in file:
     # case if no application, only package sat as 'sat package -t'
     try:
         app = runner.cfg.APPLICATION
-    except:
+    except Exception:
         app = None
 
     # unconditionaly remove the tmp_local_working_dir
@@ -1909,7 +1911,7 @@ Please add it in file:
             shutil.rmtree(tmp_local_working_dir)
 
     # remove the tmp directory, unless user has registered as developer
-    if os.path.isdir(tmp_working_dir) and (not DBG.isDeveloper()):
+    if os.path.isdir(tmp_working_dir) and (not debug.isDeveloper()):
         shutil.rmtree(tmp_working_dir)
 
     # Print again the path of the package
index ec438655fd61df9be55ada8452628d251ee8fc97..a42a71889e4e9304e03bc82ca45724563108296b 100644 (file)
 
 import os
 import subprocess
-import re
+from gettext import gettext as _
 
 import src
-import prepare
 
 # Define all possible option for patch command :  sat patch <options>
 parser = src.options.Options()
index 71a44a805b935d1dac409f7f2f6256675dd37a09..60f7f71c975877ec0f1d3eca5f16c9c3a781f60b 100644 (file)
 #  License along with this library; if not, write to the Free Software
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
-import re
 import os
-import pprint as PP
+from gettext import gettext as _
 
 import src
-import src.debug as DBG
 
 
 # Define all possible option for prepare command :  sat prepare <options>
index c6138e5087e18db34ab8fb1fbb3021dc15102e87..c1bd847728e85a91290866233179becfdabc0f6c 100644 (file)
@@ -18,6 +18,7 @@
 
 import os
 import subprocess
+from gettext import gettext as _
 
 import src
 
index 824a1331b20fc139ee661b600cf6fb15edbf6523..a2b4bdab7362a532e282f2d81e5c7f301cf88912 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import os
+from gettext import gettext as _
 
 import src
 
index c2139755c3b373d66b2a1ab64b1a4be4cc03dee2..f8257d224741098e60170c39bab622ad88aab311 100644 (file)
@@ -17,6 +17,7 @@
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
 import subprocess
+from gettext import gettext as _
 
 import src
 
index 832e484f0f032653e3522caa83bbfe2e58b87ef7..b5584220dcc4a690dcd8c94d72953b1782378d00 100644 (file)
 
 import os
 import shutil
-import re
-import subprocess
+from gettext import gettext as _
 
 import src
-import prepare
-import src.debug as DBG
+import src.debug as debug
 
 # Define all possible option for patch command :  sat patch <options>
 parser = src.options.Options()
@@ -554,7 +552,7 @@ def description():
 def run(args, runner, logger):
     '''method that is called when salomeTools is called with source parameter.
     '''
-    DBG.write("source.run()", args)
+    debug.write("source.run()", args)
     # Parse the options
     (options, args) = parser.parse_args(args)
     
index e4ddc59677672ec65b8130cd0f57eff046c6248d..88831858ba5cb641eb36cb8de7786a860c2a9230 100644 (file)
@@ -22,6 +22,7 @@ import shutil
 import subprocess
 import fnmatch
 import re
+from gettext import gettext as _
 
 import src
 
@@ -35,7 +36,7 @@ except NameError:
 # Python 2/3 compatibility for execfile function
 try:
     execfile
-except:
+except Exception:
     def execfile(somefile, global_vars, local_vars):
         with open(somefile) as f:
             code = compile(f.read(), somefile, 'exec')
index 62fd7fef5cc6d55e793a4955b2dc6bde2f4ebc72..1085482c427397df3208764e8123ec62edacdc3a 100644 (file)
@@ -22,6 +22,7 @@ import shutil
 import subprocess
 import datetime
 import gzip
+from gettext import gettext as _
 
 # Compatibility python 2/3 for input function
 # input stays input for python 3 and input = raw_input for python 2
@@ -174,7 +175,7 @@ def move_test_results(in_dir, what, out_dir, logger):
                 #shutil.rmtree(finalPath)
                 os.makedirs(finalPath)
             pathIsOk = True
-        except:
+        except Exception:
             logger.error(_("%s cannot be created.") % finalPath)
             finalPath = ask_a_path()
 
@@ -664,7 +665,7 @@ def run(args, runner, logger):
     if os.access(tmp_dir, os.F_OK):
         try:
             shutil.rmtree(tmp_dir)
-        except:
+        except Exception:
             logger.error(_("error removing TT_TMP_RESULT %s\n") 
                                 % tmp_dir)
 
index 24ee9ebec4a3ac1a969c9399cd81270e5efed655..3a68b587cd8ebf2476cb9d4645ef1ed672a594b7 100644 (file)
@@ -1,4 +1,3 @@
-
   LOCAL :
   {
     base : 'default'
@@ -12,5 +11,6 @@
   {
     project_file_paths :
     [
+      '/home/catA/gb262689/cea-projects/salome/sat_salome_fork_gb262689/salome.pyconf'
     ]
   }
index 1521075b8d14de1e4e9c23ebb3eb1df0240426c8..8c9e3cdc91e9cb6f01bc306ab75e7667d3fabe5e 100755 (executable)
@@ -9,7 +9,7 @@ import os, sys
 
 try:
   __LANG__ = os.environ["LANG"] # original locale
-except:
+except Exception:
   __LANG__ = "en_US.utf8" #default
 
 __FR__={
@@ -27,7 +27,7 @@ def _loc(text):
   if "FR" in __LANG__:
     try:
       return __FR__[text]
-    except:
+    except Exception:
       return text
   return text
 
index 56e155787b66ba1cd434368deb74a3edd722aa5f..da4f92b2a50df5d2b79a81703409d99c3af3a95d 100755 (executable)
@@ -88,7 +88,7 @@ def verbose():
     if __verbose__ is None:
         try:
             __verbose__ = int( os.getenv( 'SALOME_VERBOSE', 0 ) )
-        except:
+        except Exception:
             __verbose__ = 0
             pass
         pass
@@ -177,7 +177,7 @@ def findOrCreateComponent( study ):
         try:
             builder.DefineComponentInstance( father, getEngine() )
             pass
-        except:
+        except Exception:
             pass
         pass
     return father
index 56e155787b66ba1cd434368deb74a3edd722aa5f..da4f92b2a50df5d2b79a81703409d99c3af3a95d 100755 (executable)
@@ -88,7 +88,7 @@ def verbose():
     if __verbose__ is None:
         try:
             __verbose__ = int( os.getenv( 'SALOME_VERBOSE', 0 ) )
-        except:
+        except Exception:
             __verbose__ = 0
             pass
         pass
@@ -177,7 +177,7 @@ def findOrCreateComponent( study ):
         try:
             builder.DefineComponentInstance( father, getEngine() )
             pass
-        except:
+        except Exception:
             pass
         pass
     return father
index b871200f6461694aa6ff92d2fa83ab6bf8576130..d13441debaf19bec00fb880d3a77ce768a56bf41 100644 (file)
@@ -6,21 +6,21 @@ appending method tostring serialize 'pretty_xml'
 """
 
 import sys
-import debug as DBG
+import src.debug as debug
 
 _versionPython = sys.version_info[0]
 
 if _versionPython < 3:
   # python2 previous historic mode
   import src.ElementTreePython2 as etree
-  DBG.write("ElementTree Python2 %s" % etree.VERSION, etree.__file__, DBG.isDeveloper())
+  debug.write("ElementTree Python2 %s" % etree.VERSION, etree.__file__, debug.isDeveloper())
   tostring = etree.tostring
 
 else:
   # python3 mode
   # import xml.etree.ElementTree as etree # native version
   import src.ElementTreePython3 as etree # VERSION 1.3.0 plus _serialize 'pretty_xml'
-  DBG.write("ElementTree Python3 %s" % etree.VERSION, etree.__file__, DBG.isDeveloper())
+  debug.write("ElementTree Python3 %s" % etree.VERSION, etree.__file__, debug.isDeveloper())
 
   def tostring(node, encoding='utf-8'):
     """
@@ -30,7 +30,7 @@ else:
     """
     try:
       aStr = etree.tostring(node, encoding='unicode', method="pretty_xml")
-    except:
+    except Exception:
       print("*****************************\n problem node", node)
       # try no pretty
       aStr = etree.tostring(node, encoding='unicode')
index d1ba909381f358c50b07e5852dde728708176723..4b31874cd939a618ff8ec61060b1c02b1a674b9e 100644 (file)
@@ -700,7 +700,7 @@ class ElementTree:
             if items or xmlns_items:
                 try:
                     items = sorted(items) # lexical order
-                except:
+                except Exception:
                     print("*** problem sorting items", items)
                 for k, v in items:
                     try:
@@ -1086,7 +1086,7 @@ class TreeBuilder:
                 for item in self._data:
                     try:
                         text += item
-                    except:
+                    except Exception:
                         text += item.decode()
                 if self._tail:
                     assert self._last.tail is None, "internal error (tail)"
@@ -1292,7 +1292,7 @@ class XMLTreeBuilder:
         """
         try:
             self._parser.Parse(data, 0)
-        except:
+        except Exception:
             print("*** problem feed:\n%s" % data.decode('utf-8'))
 
     ##
index 68635ce6d819df15a5a0f14af2f1164947dbab72..185828a8532bd4372c2291a2088b98dd027b55f3 100644 (file)
@@ -97,7 +97,7 @@ import warnings
 import io
 import contextlib
 
-import ElementPath
+from src import ElementPath
 
 
 class ParseError(SyntaxError):
@@ -1266,7 +1266,7 @@ def iterparse(source, events=None, parser=None):
         close_source = True
     try:
         return _IterParseIterator(source, events, parser, close_source)
-    except:
+    except Exception:
         if close_source:
             source.close()
         raise
@@ -1365,7 +1365,7 @@ class _IterParseIterator:
                 else:
                     self._root = self._parser._close_and_return_root()
             self.root = self._root
-        except:
+        except Exception:
             if self._close_file:
                 self._file.close()
             raise
index 6e83d0beeea8dcb950ea3f9cd0c98dba03cd7e8d..0d61421155b71b1d2ee5d22cebdccade628a0a2b 100644 (file)
@@ -21,23 +21,19 @@ In this file : all the stuff that can change with the architecture
 on which SAT is running
 '''
 
-import os, sys
+import os
+import sys
+from gettext import gettext as _
 from platform import system,python_version,release
 
-# linux_distribution is removed from platform module in python 3.8+
-# we have to use distro module, which is not standard. 
-# write an error message if distro is not installed
 try:
-    from platform import linux_distribution
-except:
-    try:
-        from distro import linux_distribution
-    except:
-        print ("\nError :\n"
-               "  linux_distribution was removed from platform module in Python 3.8+\n"
-               "  For python 3.8+ sat requires distro module to get information on linux distribution.\n"
-               "  Please install distro module with : pip install distro")
-        sys.exit(-1)
+    from distro import distro
+except Exception:
+    print ("\nError :\n"
+            "  linux_distribution was removed from platform module in Python 3.8+\n"
+            "  For python 3.8+ sat requires distro module to get information on linux distribution.\n"
+            "  Please install distro module with : pip install distro")
+    sys.exit(1)
 
 
 def is_windows():
@@ -76,7 +72,7 @@ def get_distribution(codes):
         return "W"
 
     # else get linux distribution description from platform, and encode it with code
-    lin_distrib = linux_distribution()[0].lower()
+    lin_distrib = distro.id().lower()
     distrib="not found"
     for dist in codes:
         if dist in lin_distrib:
@@ -95,7 +91,7 @@ def get_version_XY():
     from a CentOS example, returns '7.6'
     extracted from platform.linux_distribution()
     """
-    dist_version=linux_distribution()[1].split('.')
+    dist_version=distro.version().split('.')
     if len(dist_version)==1:
         version = dist_version[0]
     else:
@@ -119,7 +115,7 @@ def get_distrib_version(distrib):
         return release()
 
     # get version from platform
-    dist_version=linux_distribution()[1].split('.')
+    dist_version=distro.version().split('.')
 
     # encode it (conform to src/internal_config/distrib.pyconf VERSIONS dist
     if distrib == "CO":
index 9fd0308f8088e21f9afb4a4ac65476a27e174db9..439de552e4bd25392b241b75854a0cdb80f24c76 100644 (file)
@@ -8,8 +8,6 @@
 import os
 import sys
 import inspect
-import logging
-import pprint as PP
 
 
 ##############################################################################
@@ -116,7 +114,7 @@ def caller_name_stack(skip=1):
       if namesrc == "__main__":
         namesrc = os.path.basename(fr.f_globals["__file__"])
       lineno.insert(0, (namesrc + "[%s]" % fr.f_lineno))
-    except:
+    except Exception:
       lineno.insert(0, ("??", fr.f_lineno))
 
   if codename != '<module>':  # top level usually
index 751fe73b911891c94f4de45f3fa9d46768d169be..da7978920f5ce6f2a6eb197722342c0bae1bec82 100644 (file)
@@ -21,6 +21,7 @@ import subprocess
 import sys
 import shutil
 import glob
+from gettext import gettext as _
 
 import src
 
@@ -440,7 +441,7 @@ CC=\\"hack_libtool\\"%g" libtool'''
             pymodule = imp.load_source(product + "_compile_script", script)
             self.nb_proc = nb_proc
             retcode = pymodule.compil(self.config, self, self.logger)
-        except:
+        except Exception:
             __, exceptionValue, exceptionTraceback = sys.exc_info()
             self.logger.write(str(exceptionValue), 1)
             import traceback
index 330fb6df37c33468237c5c49f03f4e04528161b4..a5292d289e51811b811ca8550328eeaa891ec453 100755 (executable)
@@ -56,12 +56,11 @@ import sys
 import traceback
 import pprint as PP
 import inspect
-import src
 
 # Compatibility python 2/3 for unicode
 try:
     _test = unicode
-except:
+except Exception:
     unicode = str
 
 # Compatibility python 2/3 for StringIO
@@ -272,7 +271,7 @@ def _saveConfigRecursiveDbg(config, aStream, indent, path, nb):
     try: #type config, mapping
       order = object.__getattribute__(config, 'order')
       data = object.__getattribute__(config, 'data')
-    except:
+    except Exception:
       aStream.write("%s%s : '%s'\n" % (indstr, path, str(config)))
       return     
     for key in sorted(data): #order): # data as sort alphabetical, order as initial order
index a706d5deab31149f876575522860b92198d41b46..49c54b848c0a9623cbfe62602b0a07d0fe189d75 100644 (file)
@@ -23,7 +23,7 @@ import sys
 import copy
 
 import src
-import src.debug as DBG
+import src.debug as debug
 import pprint as PP
 
 
@@ -321,7 +321,7 @@ class SalomeEnviron:
         for k in self.environ.environ.keys():
             try:
                 value = self.get(k)
-            except:
+            except Exception:
                 value = "?"
             out.write("%s=%s\n" % (k, value))
 
@@ -411,7 +411,7 @@ class SalomeEnviron:
         :param product_info Config: The product description
         :param logger Logger: The logger instance to display messages        
         """
-        DBG.write("set_salome_minimal_product_env", product_info)
+        debug.write("set_salome_minimal_product_env", product_info)
 
         # set root dir
         root_dir = product_info.name + "_ROOT_DIR"
@@ -743,7 +743,7 @@ class SalomeEnviron:
                 # not mandatory, if set_nativ_env not defined, we do nothing
                 if "set_nativ_env" in dir(pyproduct):
                     pyproduct.set_nativ_env(self)
-        except:
+        except Exception:
             __, exceptionValue, exceptionTraceback = sys.exc_info()
             print(exceptionValue)
             import traceback
@@ -779,7 +779,7 @@ class SalomeEnviron:
         :param logger Logger: The logger instance to display messages
         :param env_info list: the list of products
         """
-        DBG.write("set_full_environ for", env_info)
+        debug.write("set_full_environ for", env_info)
         # DBG.write("set_full_environ config", self.cfg.APPLICATION.environ, True)
         # set product environ
         self.set_application_env(logger)
index 9e98b3fd938d79e9273e731114810e0bbcc1df01..438a2ef2f33c3e2640dba640b4dc666389d9f580 100644 (file)
@@ -18,7 +18,7 @@
 
 import os
 import pprint as PP
-import src.debug as DBG
+import src.debug as debug
 import src.architecture
 import src.environment
 
index aa239de9a05581797fa8d08ea04959a210e59e7f..e4492f04cdb1b5edf5a7f36591f69a2608313e84 100644 (file)
@@ -131,7 +131,7 @@ def batch_salome(cmd, logger, cwd, args, getTmpDir,
             try:
                 statinfo = os.stat(os.path.join(tmp_dir, file_name))
                 currentTime = statinfo.st_mtime
-            except:
+            except Exception:
                 pass
 
             if currentTime and currentTime > beginTime:
index cf467dac8e81bdc3d405a68928546591285b33df..7a2205dad4e635e3268742e9909416a311c877d3 100755 (executable)
@@ -25,6 +25,8 @@ import os
 import gettext
 import unittest
 
+_ = gettext.gettext
+
 verbose = False
 
 class TestCase(unittest.TestCase):
index 545ac0b8dfbe368f7d2333b23129455ed653dffa..fe232c0361c5328c83f54bf582e51381b0e88109 100755 (executable)
@@ -27,12 +27,12 @@ import datetime
 import re
 import tempfile
 import shutil
+from gettext import gettext as _
 
 import src
-import printcolors
-import xmlManager
-
-import src.debug as DBG
+import src.xmlManager as xmlManager
+import src.printcolors as printcolors
+import src.debug as debug
 
 log_macro_command_file_expression = "^[0-9]{8}_+[0-9]{6}_+.*\.xml$"
 log_all_command_file_expression = "^.*[0-9]{8}_+[0-9]{6}_+.*\.xml$"
@@ -54,7 +54,7 @@ class Logger(object):
         :param silent_sysstd boolean: if True, do not write anything
                                       in terminal.
         """
-        DBG.write("src.logger.Logger", id(self))
+        debug.write("src.logger.Logger", id(self))
         self.config = config
         self.default_level = 3
         self.silentSysStd = silent_sysstd
@@ -538,9 +538,8 @@ def setCurrentLogger(logger):
     logger.debug("set current logger as %s" % logger.name)
   else:
     if _currentLogger[0].name != logger.name:
-      # logger.debug("quit current logger as default %s" % _currentLogger[0].name)
       _currentLogger[0] = logger
-      logger.warning("change current logger as %s" % logger.name)
+      logger.warning("change current logger as %s" % logger.name)
   return _currentLogger[0]
 
 def isCurrentLoggerUnittest():
index 177b3426f2bf67dea058be8d2933518fc663c6dd..4d20a2c81ca2809fc5cb615696b7feb2620b0050 100755 (executable)
@@ -23,7 +23,7 @@ import logging as LOGI
 from logging.handlers import BufferingHandler
 import pprint as PP
 
-import src.debug as DBG  # Easy print stderr (for DEBUG only)
+import src.debug as debug  # Easy print stderr (for DEBUG only)
 
 _verbose = False
 _name = "loggingSimple"
@@ -377,7 +377,7 @@ class StreamHandlerSimple(LOGI.StreamHandler, object): # object force new-style
       self.flush()
     except (KeyboardInterrupt, SystemExit):
       raise
-    except:
+    except Exception:
       self.handleError(record)
 
 
index bbcf8936866588aa8290797ac2df6176f11546d5..71885f43a493627171b6908a20f53e189c872f54 100755 (executable)
@@ -25,11 +25,11 @@ import getopt
 import sys
 import re
 import pprint as PP
+from gettext import gettext as _
 
 from . import printcolors
 
-import src
-import src.debug as DBG # Easy print stderr (for DEBUG only)
+import src.debug as debug
 
 class OptResult(object):
     """
@@ -196,7 +196,7 @@ class Options(object):
         if argList is None:
             argList = sys.argv[1:]
         
-        DBG.write("parse_args", argList)
+        debug.write("parse_args", argList)
         # DBG.write("options", self.options)
         # format shortNameOption and longNameOption 
         # to make right arguments to getopt.getopt function
@@ -260,19 +260,19 @@ class Options(object):
             option['result'] = None
 
         self.results = {"optlist": optlist, "optResult": optResult, "args": args, "argList": argList}
-        DBG.write("results", self.results)
+        debug.write("results", self.results)
         return optResult, args
         
     def filterLevel(self, aLevel):
       """filter level logging values"""
-      import src.loggingSimple as LOG
+      import src.loggingSimple as log
       aLev = aLevel.upper()
-      knownLevels = LOG._knownLevels
+      knownLevels = log._knownLevels
       maxLen = max([len(i) for i in knownLevels])
       for i in range(maxLen):
         for lev in knownLevels:
           if aLev == lev[:i]:
-            DBG.write("filterLevel", "%s -> %s" % (aLevel, lev)) 
+            debug.write("filterLevel", "%s -> %s" % (aLevel, lev)) 
             return lev
       msg = "Unknown level '%s', accepted are:\n%s" % (aLev, ", ".join(knownLevels))
       raise Exception(msg)
@@ -323,6 +323,6 @@ class Options(object):
       return res
         
     def debug_write(self):
-      DBG.write("options and results", self, True)
+      debug.write("options and results", self, True)
 
 
index a679e44817d9d767140fbeb15f83076b0ed1fe5c..9ef9528f962ad147c96ceecaac8ae2a3e959f953 100644 (file)
@@ -26,7 +26,7 @@ import re
 import pprint as PP
 
 import src
-import src.debug as DBG
+import src.debug as debug
 import src.versionMinorMajorPatch as VMMP
 
 AVAILABLE_VCS = ['git', 'svn', 'cvs']
@@ -304,14 +304,14 @@ Please provide a 'compil_script' key in its definition.""") % product_name
             script_path = src.find_file_in_lpath(script_name, config.PATHS.PRODUCTPATH, "compil_scripts")
             if not script_path:
                 msg = _("Compilation script %s not found in") % script_name
-                DBG.tofix(msg, config.PATHS.PRODUCTPATH, True) # say where searched
+                debug.tofix(msg, config.PATHS.PRODUCTPATH, True) # say where searched
                 script_path = "%s_(Not_Found_by_Sat!!)" % script_name
             prod_info.compil_script = script_path
 
        
         # Check that the script is executable
         if os.path.exists(prod_info.compil_script) and not os.access(prod_info.compil_script, os.X_OK):
-            DBG.tofix("Compilation script  file is not in 'execute mode'", prod_info.compil_script, True)
+            debug.tofix("Compilation script  file is not in 'execute mode'", prod_info.compil_script, True)
     
     # If the product has a post install script, check the script existence
     # and if it is executable
@@ -328,14 +328,14 @@ Please provide a 'compil_script' key in its definition.""") % product_name
             script_path = src.find_file_in_lpath(script_name, config.PATHS.PRODUCTPATH, "post_scripts")
             if not script_path:
                 msg = _("Post install script %s not found in") % script_name
-                DBG.tofix(msg, config.PATHS.PRODUCTPATH, True) # say where searched
+                debug.tofix(msg, config.PATHS.PRODUCTPATH, True) # say where searched
                 script_path = "%s_(Not_Found_by_Sat!!)" % script_name
             prod_info.post_script = script_path
 
        
         # Check that the script is executable
         if os.path.exists(prod_info.post_script) and not os.access(prod_info.post_script, os.X_OK):
-            DBG.tofix("Post install script file is not in 'execute mode'", prod_info.post_script, True)
+            debug.tofix("Post install script file is not in 'execute mode'", prod_info.post_script, True)
 
     # Get the full paths of all the patches
     if product_has_patches(prod_info):
@@ -355,8 +355,8 @@ Please provide a 'compil_script' key in its definition.""") % product_name
                                        "prod_name" : prod_info.name}) 
                       raise src.SatException(msg)
               patches.append(patch_path)
-        except:
-          DBG.tofix("problem in prod_info.patches", prod_info)
+        except Exception:
+          debug.tofix("problem in prod_info.patches", prod_info)
         prod_info.patches = patches
 
     # Get the full paths of the environment scripts
@@ -425,7 +425,7 @@ def get_product_section(config, product_name, version, section=None):
     # decode version number
     try:
       versionMMP = VMMP.MinorMajorPatch(version)
-    except: # example setuptools raise "minor in major_minor_patch is not integer: '0_6c11'"
+    except Exception: # example setuptools raise "minor in major_minor_patch is not integer: '0_6c11'"
       versionMMP = None
 
     # if a section is explicitely specified we select it
@@ -461,7 +461,7 @@ def get_product_section(config, product_name, version, section=None):
               tagged.append((name, [vmin, vmax]))
 
         if len(tagged) > 1:
-          DBG.write("multiple version ranges tagged for '%s', fix it" % version,
+          debug.write("multiple version ranges tagged for '%s', fix it" % version,
                          PP.pformat(tagged))
           pi=None
         elif len(tagged) == 1: # ok
@@ -666,10 +666,10 @@ def add_compile_config_file(p_info, config):
     try:
       with open(aFile, 'w') as f:
         p_info.__save__(f, evaluated=True) # evaluated expressions mode
-    except:
+    except Exception:
       # sometime some information cannot be evaluated.
       # for example, in the context of non VCS archives, information on git server is not available.
-      DBG.write("Warning : sat was not able to evaluate and write down some information in file %s" % aFile)
+      debug.write("Warning : sat was not able to evaluate and write down some information in file %s" % aFile)
   
 
 def check_config_exists(config, prod_dir, prod_info, verbose=False):
@@ -694,8 +694,8 @@ def check_config_exists(config, prod_dir, prod_info, verbose=False):
     else:
       raise Exception("Inexisting directory '%s'" % prod_dir)
 
-    DBG.write("check_config_exists 000",  (prod_dir, l_dir_and_files), verbose)
-    DBG.write("check_config_exists 111",  prod_info, verbose)
+    debug.write("check_config_exists 000",  (prod_dir, l_dir_and_files), verbose)
+    debug.write("check_config_exists 111",  prod_info, verbose)
 
     depend_all=[]
     if "depend" in prod_info:
@@ -714,7 +714,7 @@ def check_config_exists(config, prod_dir, prod_info, verbose=False):
         # directory    
         afilename = CONFIG_FILENAME + prod_info.name + ".pyconf"
         config_file = os.path.join(prod_dir, dir_or_file, afilename)
-        DBG.write("check_config_exists 222", config_file, verbose)
+        debug.write("check_config_exists 222", config_file, verbose)
         if not os.path.exists(config_file):
             continue
         
@@ -741,7 +741,7 @@ def check_config_exists(config, prod_dir, prod_info, verbose=False):
             # assume new compatibility with prod_name in sat-config.pyconf files
             if prod_name == prod_info.name:
               if prod_info.version == compile_cfg[prod_name]:
-                DBG.write("check_config_exists OK 333", compile_cfg, verbose)
+                debug.write("check_config_exists OK 333", compile_cfg, verbose)
                 pass
               else: # no correspondence with newer with prod_name sat-config.pyconf files
                 config_corresponds = False
@@ -754,7 +754,7 @@ def check_config_exists(config, prod_dir, prod_info, verbose=False):
                 break
         
         if config_corresponds: # returns (and stops) at first correspondence found
-            DBG.write("check_config_exists OK 444", dir_or_file, verbose)
+            debug.write("check_config_exists OK 444", dir_or_file, verbose)
             return True, os.path.join(prod_dir, dir_or_file)
 
     # no correspondence found
@@ -829,7 +829,7 @@ def get_products_list(options, cfg, logger):
               else:
                 res.append((p_name, p_info))
                 ok.append(p_name)
-            except:
+            except Exception:
               res.append((p_name, p_info))
               ok.append(p_name)
       else:
@@ -840,7 +840,7 @@ def get_products_list(options, cfg, logger):
                 ok.append(p_name)
               else:
                 ko.append(p_name)
-            except:
+            except Exception:
               ko.append(p_name)
 
       if len(ok) != len(resAll):
@@ -1003,7 +1003,7 @@ def product_is_dev(product_info):
     """
     dev = product_info.dev
     res = (dev.lower() == 'yes')
-    DBG.write('product_is_dev %s' % product_info.name, res)
+    debug.write('product_is_dev %s' % product_info.name, res)
     # if product_info.name == "XDATA": return True #test #10569
     return res
 
index 5957d6ddcf8a12cbe2d5d971bf3eac266ca50ada..d19b5776c421d7c92ef1828598c1816e195f7e5e 100644 (file)
@@ -155,7 +155,7 @@ else:
 
 try:
     has_utf32 = True
-except:
+except Exception:
     has_utf32 = False
 
 class ConfigInputStream(object):
@@ -951,7 +951,7 @@ class Reference(object):
                         rv = eval(str(self)[1:-1], vars(ns))
                         found = True
                         break
-                    except:
+                    except Exception:
                         pass
                 if found:
                     break
@@ -963,7 +963,7 @@ class Reference(object):
                         key = item[1]
                         rv = rv[key]
                     break
-                except:
+                except Exception:
                     rv = None
                     pass
             current = object.__getattribute__(current, 'parent')
@@ -1632,13 +1632,13 @@ class ConfigMerger(object):
                             continue
                         try:
                             exec( 'map1.' + key + " = " + repr(overwrite_instruction[key]))
-                        except:
+                        except Exception:
                             exec('map1.' + key + " = " + str(overwrite_instruction[key]))
             else:
                 for key in overwrite_instruction.keys():
                     try:
                         exec('map1.' + key + " = " + repr(overwrite_instruction[key]))
-                    except:
+                    except Exception:
                         exec('map1.' + key + " = " + str(overwrite_instruction[key]))
 
     def mergeMapping(self, map1, map2):
index 1af9116677d72a9969a3b197376751e66d925140..240eb6332ce797a73404b5dfa011da95225723d5 100644 (file)
@@ -148,7 +148,7 @@ class ReturnCode(object):
     """return system return code as bash or bat"""
     try:
       return self._TOSYS[self._status]
-    except:
+    except Exception:
       return self._TOSYS[self.NA_STATUS]
 
   def toXmlPassed(self):
index a49524b24c465199efc051015dc06bed564dd2f7..04b7deab9f0688a734fc64dad75f7419d74ae285 100755 (executable)
 #  License along with this library; if not, write to the Free Software
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
 
+import sys
+from gettext import gettext as _
+import os
+import re
+import tempfile
+import imp
+import types
+import gettext
+
+import src
+import src.debug as debug # Easy print stderr (for DEBUG only)
+import src.returnCode as RCO # Easy (ok/ko, why) return methods code
+import src.utilsSat as UTS
+
 """
 This file is the main API file for salomeTools
 
@@ -25,8 +39,6 @@ This file is the main API file for salomeTools
 | Usage: see file ../sat
 """
 
-import sys
-
 # exit OKSYS and KOSYS seems equal on linux or windows
 _OKSYS = 0  # OK
 _KOSYS = 1  # KO
@@ -41,19 +53,6 @@ ERROR: 'salomeTools.py' is not main command entry (CLI) for salomeTools.
     sys.stderr.write(msg)
     sys.exit(_KOSYS)
 
-# python imports
-import os
-import re
-import tempfile
-import imp
-import types
-import gettext
-import traceback
-
-import src
-import src.debug as DBG # Easy print stderr (for DEBUG only)
-import src.returnCode as RCO # Easy (ok/ko, why) return methods code
-import src.utilsSat as UTS
 
 # get path to salomeTools sources
 satdir  = os.path.dirname(os.path.dirname(os.path.realpath(__file__)))
@@ -67,7 +66,7 @@ gettext.install("salomeTools", os.path.join(srcdir, "i18n"))
 
 try:
   _LANG = os.environ["LANG"] # original locale
-except:
+except Exception:
   _LANG = "en_US.utf8" #default
 
 # The possible hooks : 
@@ -149,7 +148,7 @@ def setNotLocale():
     """force english at any moment"""
     os.environ["LANG"] = ''
     gettext.install("salomeTools", os.path.join(srcdir, "i18n"))
-    DBG.write("setNotLocale", os.environ["LANG"])
+    debug.write("setNotLocale", os.environ["LANG"])
     
 def setLocale():
     """
@@ -159,7 +158,7 @@ def setLocale():
     """
     os.environ["LANG"] = _LANG
     gettext.install("salomeTools", os.path.join(srcdir, "i18n"))
-    DBG.write("setLocale", os.environ["LANG"])
+    debug.write("setLocale", os.environ["LANG"])
     
 def getVersion():
     """get version number as string"""
@@ -235,13 +234,13 @@ class Sat(object):
         """set the commands by calling the dedicated function etc..."""
         options, remaindersArgs = parser.parse_args(opt)
         if options.debug_mode:
-            DBG.push_debug(True)
+            debug.push_debug(True)
         self.arguments = opt
         self.options = options # the generic options passed to salomeTools
         self.remaindersArgs = remaindersArgs  # the command and their options
         self.datadir = datadir # default value will be <salomeTools root>/data
         self._setCommands(cmdsdir)
-        DBG.write("Sat.options", self.options, self.options.debug_mode)
+        debug.write("Sat.options", self.options, self.options.debug_mode)
 
     def getConfig(self):
         return self.cfg
@@ -264,8 +263,8 @@ class Sat(object):
           self.mainLogger.info(get_help())
           return RCO.ReturnCode("OK", "help done")
 
-        DBG.write("options", self.options)
-        DBG.write("remaindersArgs", self.remaindersArgs)
+        debug.write("options", self.options)
+        debug.write("remaindersArgs", self.remaindersArgs)
 
         if len(self.remaindersArgs) == 0:
           return RCO.ReturnCode("KO", "Nothing to do")
@@ -285,7 +284,8 @@ class Sat(object):
         # Run the command using the arguments
         code = fun_command(self.remaindersArgs[1:])
 
-        if code is None: code = 0 # what?! do not know why so respect history
+        if code is None: 
+            code = 0
 
         # return salomeTools command with the right message
         # code (0 if no errors, else 1)
@@ -356,13 +356,14 @@ class Sat(object):
                     sys.stderr = ff
                     import paramiko
                     sys.stderr = saveout
-                except:
+                except Exception:
                     sys.stderr = saveout
                     continue
 
             # load the module that has name nameCmd in dirPath
             (file_, pathname, description) = imp.find_module(nameCmd, [dirPath])
             module = imp.load_module(nameCmd, file_, pathname, description)
+            file_.close()
             
             def run_command(args='',
                             options=None,
@@ -488,7 +489,7 @@ class Sat(object):
                     logger_command.write(" *****\n", 1)
                     print(e.message)
                     if self.options.debug_mode:
-                        logger_command.write("\n" + DBG.format_exception("") + "\n", 1)
+                        logger_command.write("\n" + debug.format_exception("") + "\n", 1)
 
                 except Exception as e:
                     # here we print the stack in addition
@@ -496,7 +497,7 @@ class Sat(object):
                     logger_command.write(src.printcolors.printcError(
                             "salomeTools ERROR: sat %s" % __nameCmd__), 1)
 
-                    logger_command.write("\n" + DBG.format_exception("") + "\n", 1)
+                    logger_command.write("\n" + debug.format_exception("") + "\n", 1)
 
 
                 finally:
@@ -678,6 +679,7 @@ class Sat(object):
         # load the module
         (file_, pathname, description) = imp.find_module(module, [cmdsdir])
         module = imp.load_module(module, file_, pathname, description)
+        file_.close()
         return module
 
 ##################################################################
index acb6feaef88bb8002125fb989b86899c77f38aad..544e6c71201eb0feadd21e4db6a2a224d93335e8 100644 (file)
@@ -25,13 +25,12 @@ import os
 import subprocess as SP
 import time
 import tarfile
-import time
+from gettext import gettext as _
  
-
-import debug as DBG
-import utilsSat as UTS
 import src
 
+from . import utilsSat as UTS
+from . import debug
 from . import printcolors
 
 def show_in_editor(editor, filePath, logger):
@@ -53,7 +52,7 @@ def show_in_editor(editor, filePath, logger):
         logger.write('Launched command:\n' + cmd + '\n', 5)
         p = SP.Popen(cmd, shell=True)
         p.communicate()
-    except:
+    except Exception:
         logger.write(printcolors.printcError(_("Unable to edit file %s\n") 
                                              % filePath), 1)
 
@@ -105,7 +104,7 @@ def git_describe(repo_path):
     '''Use git describe --tags command to return tag description of the git repository"
     :param repo_path str: The git repository to describe
     '''
-    git_cmd="cd %s;git describe --tags" % repo_path
+    git_cmd=f"cd {repo_path};git describe --tags"
     p = SP.Popen(git_cmd, shell=True, stdin=SP.PIPE, stdout=SP.PIPE, stderr=SP.PIPE)
     p.wait()
     if p.returncode != 0:
@@ -130,7 +129,7 @@ def git_extract(from_what, tag, git_options, where, logger, environment=None):
   :return: True if the extraction is successful
   :rtype: boolean
   '''
-  DBG.write("git_extract", [from_what, tag, str(where)])
+  debug.write("git_extract", [from_what, tag, str(where)])
   if not where.exists():
     where.make()
   where_git = os.path.join(str(where), ".git")
@@ -178,7 +177,7 @@ exit $res
   logger.logTxtFile.write("\n" + cmd + "\n")
   logger.logTxtFile.flush()
 
-  DBG.write("cmd", cmd)
+  debug.write("cmd", cmd)
   # git commands may fail sometimes for various raisons 
   # (big module, network troubles, tuleap maintenance)
   # therefore we give several tries
@@ -227,7 +226,7 @@ def git_extract_sub_dir(from_what, tag, git_options, where, sub_dir, logger, env
            'parentWhere': parentWhere,
            'tmpWhere': tmpWhere,
            }
-  DBG.write("git_extract_sub_dir", aDict)
+  debug.write("git_extract_sub_dir", aDict)
   if not src.architecture.is_windows():
     cmd = r"""
 set -x
@@ -253,7 +252,7 @@ git log -1 > %(where)s/README_git_log.txt && \
 rm -rf $tmpDir
 """ % aDict
 
-  DBG.write("cmd", cmd)
+  debug.write("cmd", cmd)
 
   for nbtry in range(0,3): # retries case of network problem
     rc = UTS.Popen(cmd, cwd=parentWhere, env=environment.environ.environ, logger=logger)
@@ -444,7 +443,7 @@ def check_system_pkg(check_cmd,pkg):
         try:
             output = SP.check_output(['grep', pkg], stdin=p.stdout)
             msg_status=src.printcolors.printcSuccess("OK")
-        except:
+        except Exception:
             msg_status=src.printcolors.printcError("KO")
             msg_status+=" (package is not installed!)\n"
     elif check_cmd[0] == "dpkg-query":
index 03c7b5810943500cefa86a2b459fed78860e2f35..218ba182e1073ff4afb26afbefe335ae6da7fc34 100644 (file)
@@ -19,7 +19,7 @@
 # Python 2/3 compatibility for execfile function
 try:
     execfile
-except:
+except Exception:
     def execfile(somefile, global_vars, local_vars):
         with open(somefile) as f:
             code = compile(f.read(), somefile, 'exec')
@@ -354,12 +354,12 @@ class Test:
                   if 'time' in ldic:
                       try:
                           exec_time = float(ldic['time'])
-                      except:
+                      except Exception:
                           pass
 
                   results[test] = [status, exec_time, callback, expected]
 
-                except:
+                except Exception:
                   results[test] = ["?", -1, "", []]
                   # results[test] = [src.O_STATUS, -1, open(resfile, 'r').read(), []]
 
@@ -484,7 +484,7 @@ echo -e 'import os\nprint(os.environ[\"KERNEL_ROOT_DIR\"])' > tmpscript.py
         try:
             grid = imp.load_module(sal_uts, file_, pathname, description)
             return grid.getLogDir
-        except:
+        except Exception:
             grid = imp.load_module(sal_uts, file_, pathname, description)
             return grid.getTmpDir
         finally:
index a8df9545c1f84dd954fb3da96b5b89660b9daed9..e7d87319c4eacbc2cb096ad10a1632fa5ae32131 100755 (executable)
@@ -90,7 +90,7 @@ def toList_majorMinorPatch(aStr, verbose=False):
 
   try:
     ii = int(res[0])
-  except:
+  except Exception:
     msg = "major in major_minor_patch is not integer: '%s'" % aStr
     raise Exception(msg)
   if ii < 0:
@@ -99,7 +99,7 @@ def toList_majorMinorPatch(aStr, verbose=False):
 
   try:
     ii = int(res[1])
-  except:
+  except Exception:
     msg = "minor in major_minor_patch is not integer: '%s'" % aStr
     raise Exception(msg)
   if ii < 0:
@@ -108,7 +108,7 @@ def toList_majorMinorPatch(aStr, verbose=False):
 
   try:
     ii = int(res[2])
-  except:
+  except Exception:
     msg = "patch in major_minor_patch is not integer: '%s'" % aStr
     raise Exception(msg)
   if ii < 0:
@@ -186,7 +186,7 @@ def getRange_majorMinorPatch(aStr, verbose=False):
   try:
     rMin = MinorMajorPatch(aMin)
     rMax = MinorMajorPatch(aMax)
-  except:
+  except Exception:
     msg = "problem version range in '%s'" % aStr
     raise Exception(msg)
     """if verbose:
index 6a4bb660eebf4ec231cedc40f0b753724116654a..8f7f28e250aa8839ff43ff5e5ffd16f1a8d64e29 100644 (file)
@@ -23,7 +23,7 @@ try: # For python2
     import sys
     reload(sys)  
     sys.setdefaultencoding('utf8')
-except:
+except Exception:
     pass
 
 import src