Salome HOME
Rewinding to commit 37387bc for SALOME-8.5.0 Release
authorThomas Aubry <support-salome@cea.fr>
Thu, 21 Jun 2018 13:17:36 +0000 (15:17 +0200)
committerThomas Aubry <support-salome@cea.fr>
Thu, 21 Jun 2018 13:17:36 +0000 (15:17 +0200)
commands/prepare.py
src/ElementTree.py
src/options.py

index eb05265ec4685dcbe73a11a59724267e8ef1a909..687d051b6748649886755ce25ed7945a0a1f4500 100644 (file)
@@ -18,7 +18,6 @@
 
 import re
 import os
-import pprint as PP
 
 import src
 import src.debug as DBG
@@ -78,6 +77,34 @@ def get_products_list(options, cfg, logger):
         
     return products_infos
 
+def remove_products(arguments, l_products_info, logger):
+    '''function that removes the products in l_products_info from arguments list.
+    
+    :param arguments str: The arguments from which to remove products
+    :param l_products_info list: List of 
+                                 (str, Config) => (product_name, product_info)
+    :param logger Logger: The logger instance to use for the display and logging
+    :return: The updated arguments.
+    :rtype: str
+    '''
+    args = str(arguments) #copy of "--products ,XDATA,TESSCODE,cmake" for example
+    largs = args.split(',')
+    DBG.write("largs", largs)
+    toRemove = [name for name, xx in l_products_info]
+    DBG.write("remove_products", toRemove)
+    removed = []
+    notRemoved = []
+    for name in largs[1:]: # skip largs[0] as "--products "
+      if name in toRemove:
+        removed.append(name)
+      else:
+        notRemoved.append(name)
+    # DBG.write(removed, removed, True)
+    logger.write("  %s\n" % ",".join(removed), 1)
+    DBG.write("notRemoved", notRemoved)
+    res = largs[0] + ",".join(notRemoved)
+    return res
+
 def find_products_already_getted(l_products):
     '''function that returns the list of products that have an existing source 
        directory.
@@ -142,55 +169,58 @@ def run(args, runner, logger):
     products_infos = get_products_list(options, runner.cfg, logger)
 
     # Construct the arguments to pass to the clean, source and patch commands
-    args_appli = runner.cfg.VARS.application + " "  # useful whitespace
+    args_appli = runner.cfg.VARS.application + ' '
+    args_product_opt = '--products '
     if options.products:
-        listProd = list(options.products)
-    else: # no product interpeted as all products
-        listProd = [name for name, tmp in products_infos]
-
-    DBG.write("prepare products", sorted(listProd))
-    args_product_opt = '--products ' + ",".join(listProd)
-    do_source = (len(listProd) > 0)
-
+        for p_name in options.products:
+            args_product_opt += ',' + p_name
+    else:
+        for p_name, __ in products_infos:
+            args_product_opt += ',' + p_name
 
     ldev_products = [p for p in products_infos if src.product.product_is_dev(p[1])]
-    newList = listProd # default
+    args_product_opt_clean = args_product_opt
     if not options.force and len(ldev_products) > 0:
         l_products_not_getted = find_products_already_getted(ldev_products)
-        listNot = [i for i, tmp in l_products_not_getted]
-        newList, removedList = removeInList(listProd, listNot)
-        if len(removedList) > 0:
+        if len(l_products_not_getted) > 0:
             msg = _("""\
-Do not get the source of the following products in development mode.
+Do not get the source of the following products in development mode
 Use the --force option to overwrite it.
 """)
-            msg += "\n%s\n" % ",".join(removedList)
             logger.write(src.printcolors.printcWarning(msg), 1)
+            args_product_opt_clean = remove_products(args_product_opt_clean,
+                                                     l_products_not_getted,
+                                                     logger)
+            logger.write("\n", 1)
 
-    args_product_opt_clean = '--products ' + ",".join(newList)
-    do_clean = (len(newList) > 0)
     
-    newList = listProd # default
+    args_product_opt_patch = args_product_opt
     if not options.force_patch and len(ldev_products) > 0:
         l_products_with_patchs = find_products_with_patchs(ldev_products)
-        listNot = [i for i, tmp in l_products_with_patchs]
-        newList, removedList = removeInList(listProd, listNot)
-        if len(removedList) > 0:
+        if len(l_products_with_patchs) > 0:
             msg = _("""\
-Do not patch the following products in development mode.
+do not patch the following products in development mode
 Use the --force_patch option to overwrite it.
 """)
-            msg += "\n%s\n" % ",".join(removedList)
             logger.write(src.printcolors.printcWarning(msg), 1)
-                                                     
-    args_product_opt_patch = '--products ' + ",".join(newList)
-    do_patch = (len(newList) > 0)
-      
+            args_product_opt_patch = remove_products(args_product_opt_patch,
+                                                     l_products_with_patchs,
+                                                     logger)
+            logger.write("\n", 1)
+
     # Construct the final commands arguments
     args_clean = args_appli + args_product_opt_clean + " --sources"
     args_source = args_appli + args_product_opt  
     args_patch = args_appli + args_product_opt_patch
-      
+
+    # If there is no more any product in the command arguments,
+    # do not call the concerned command 
+    oExpr = re.compile("^--products *$")
+    do_clean = not(oExpr.search(args_product_opt_clean))
+    do_source = not(oExpr.search(args_product_opt))
+    do_patch = not(oExpr.search(args_product_opt_patch))
+    
+    
     # Initialize the results to a failing status
     res_clean = 1
     res_source = 1
@@ -201,7 +231,8 @@ Use the --force_patch option to overwrite it.
         msg = _("Clean the source directories ...")
         logger.write(msg, 3)
         logger.flush()
-        res_clean = runner.clean(args_clean, batch=True, verbose = 0, logger_add_link = logger)
+        res_clean = runner.clean(args_clean, batch=True, verbose = 0,
+                                    logger_add_link = logger)
         if res_clean == 0:
             logger.write('%s\n' % src.printcolors.printc(src.OK_STATUS), 3)
         else:
@@ -209,7 +240,8 @@ Use the --force_patch option to overwrite it.
     if do_source:
         msg = _("Get the sources of the products ...")
         logger.write(msg, 5)
-        res_source = runner.source(args_source, logger_add_link = logger)
+        res_source = runner.source(args_source,
+                                    logger_add_link = logger)
         if res_source == 0:
             logger.write('%s\n' % src.printcolors.printc(src.OK_STATUS), 5)
         else:
@@ -217,24 +249,11 @@ Use the --force_patch option to overwrite it.
     if do_patch:
         msg = _("Patch the product sources (if any) ...")
         logger.write(msg, 5)
-        res_patch = runner.patch(args_patch, logger_add_link = logger)
+        res_patch = runner.patch(args_patch,
+                                    logger_add_link = logger)
         if res_patch == 0:
             logger.write('%s\n' % src.printcolors.printc(src.OK_STATUS), 5)
         else:
             logger.write('%s\n' % src.printcolors.printc(src.KO_STATUS), 5)
     
     return res_clean + res_source + res_patch
-
-
-def removeInList(aList, removeList):
-    """Removes elements of removeList list from aList
-    
-    :param aList: (list) The list from which to remove elements
-    :param removeList: (list) The list which contains elements to remove
-    :return: (list, list) (list with elements removed, list of elements removed) 
-    """
-    res1 = [i for i in aList if i not in removeList]
-    res2 = [i for i in aList if i in removeList]
-    return (res1, res2)
-
-
index ef540ee63db795d244dd4c4bafca322d44d64a16..15d59ca164ba7d910891ff763e4d02a57ab5f637 100644 (file)
@@ -902,7 +902,7 @@ class iterparse:
     def __init__(self, source, events=None):
         if not hasattr(source, "read"):
             # OP TEST
-            print("iterparse.__init__ source = %s" % source)
+            print "iterparse.__init__ source = %s" %source
             source = open(source, "rb")
         self._file = source
         self._events = []
index 5822d39d519e3936fefb88c4dc17e5f6a15aa397..2cfeae0ae84246bbe33a8ffa966b5db2fbe801b3 100644 (file)
 #  You should have received a copy of the GNU Lesser General Public
 #  License along with this library; if not, write to the Free Software
 #  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
-
-"""
-The Options class that manages the access to all options passed as 
-parameters in salomeTools command lines
-"""
-
+'''The Options class that manages the access to all options passed as 
+   parameters in salomeTools command lines
+'''
 import getopt
 import sys
-import pprint as PP
-
 from . import printcolors
 
-import src
-import src.debug as DBG # Easy print stderr (for DEBUG only)
-
 class OptResult(object):
-    """
-    An instance of this class will be the object manipulated
-    in code of all salomeTools commands
-    The aim of this class is to have an elegant syntax to manipulate the options.
-    
-    | Example:        
-    | >> options, remainderArgs = command.parseArguments(args)
-    | >> print(options.output_verbose_level)
-    | >> 'INFO'
-    """
+    '''An instance of this class will be the object manipulated
+       in code of all salomeTools commands
+       The aim of this class is to have an elegant syntax 
+       to manipulate the options. 
+       ex: 
+       print(options.level)
+       5
+    '''
     def __init__(self):
-        """Initialization
-        """
+        '''Initialization
+        '''
         self.__dict__ = dict()
 
     def __getattr__(self, name):
-        """
-        Overwrite of the __getattr__ function 
-        to customize it for option usage
+        '''Overwrite of the __getattr__ function 
+           to customize it for option usage
         
-        :param name: (str) The attribute to get the value.
-        :return: (str int list boolean level)
-          the value corresponding to the attribute.
-        """
+        :param name str: The attribute to get the value.
+        :return: the value corresponding to the attribute.
+        :rtype: str,int,list,boolean
+        '''
         if name in self.__dict__:
             return self.__dict__[name]
         else:
-            raise AttributeError("--" + name + _(u" is not a valid option"))
+            raise AttributeError(name + _(u" is not a valid option"))
 
     def __setattr__(self, name, value):
-        """
-        Overwrite of the __setattr__ function 
-        to customize it for option usage
+        '''Overwrite of the __setattr__ function 
+           to customize it for option usage
         
-        :param name: (str) The attribute to set.
-        :param value: (str) The value  corresponding to the attribute.
-        :return: None
-        """
-        object.__setattr__(self, name, value)
-
-    def __repr__(self):
-        aStr = PP.pformat(self.__dict__)
-        res = "%s(\n %s\n)" % (self.__class__.__name__, aStr[1:-1])
-        return res
-
-class Options(object):
-    """
-    Class to manage all salomeTools options
-    """
+        :param name str: The attribute to set.
+        :param value str: The value  corresponding to the attribute.
+        :return: Nothing.
+        :rtype: N\A
+        '''
+        object.__setattr__(self,name,value)
+
+class Options:
+    '''Class to manage all salomeTools options
+    '''
     def __init__(self):
-        """Initialization
-        """
+        '''Initialization
+        '''
         # The options field stocks all options of a command 
         # in a list that contains dicts
         self.options = []
         # The list of available option type
-        self.availableOptions = "noboolean boolean string int float long list list2 level".split()
-        self.noArgOptions = "noboolean boolean".split()
+        self.availableOptions = ["boolean", "string", "int", "float",
+                                  "long", "list", "list2"]
         self.default = None
-        self.results = {}
 
-    def add_option(self, shortName, longName, optionType, destName, helpString="", default=None):
-        """
-        Add an option to a command. It gets all attributes
-        of an option and append it in the options field
+    def add_option(self, shortName, longName,
+                    optionType, destName, helpString="", default = None):
+        '''Method to add an option to a command. It gets all attributes
+           of an option and append it in the options field
         
-        :param shortName: (str) 
-          The short name of the option (as '-l' for level option).
-        :param longName: (str) 
-          The long name of the option (as '--level' for level option).
-        :param optionType: (str) The type of the option (ex "int").
-        :param destName: (str) The name that will be used in the code.
-        :param helpString: (str) 
-          The text to display when user ask for help on a command.     
-        :return: None
-        """
-        tmp = [o['shortName'] for o in self.options if o['shortName'] != '']
-        if shortName in tmp: 
-          raise Exception("option '-%s' existing yet" % shortName)
-        tmp = [o['longName'] for o in self.options if o['longName'] != '']
-        if longName in tmp: 
-          raise Exception("option '--%s' existing yet" % longName)
-
+        :param shortName str: The short name of the option
+                              (ex "l" for level option).
+        :param longName str: The long name of the option 
+                             (ex "level" for level option).
+        :param optionType str: The type of the option (ex "int").
+        :param destName str: The name that will be used in the code.
+        :param helpString str: The text to display 
+                               when user ask for help on a command.     
+        :return: Nothing.
+        :rtype: N\A
+        '''
         option = dict()
         option['shortName'] = shortName
         option['longName'] = longName
 
         if optionType not in self.availableOptions:
-          raise Exception("error optionType '%s' not available." % optionType)
+            print("error optionType", optionType, "not available.")
+            sys.exit(-1)
 
         option['optionType'] = optionType
         option['destName'] = destName
         option['helpString'] = helpString
         option['result'] = default
-        
         self.options.append(option)
-        
-    def getDetailOption(self, option):
-        """
-        for convenience 
-        
-        :return: (tuple) 4-elements (shortName, longName, optionType, helpString)
-        """
-        oos = option['shortName']
-        ool = option['longName']
-        oot = option['optionType']
-        ooh = option['helpString']
-        return (oos, ool, oot, ooh)
 
-    def get_help(self):
-        """
-        Returns all options stored in self.options 
-        as help message colored string
+    def print_help(self):
+        '''Method that display all options stored in self.options and there help
         
-        :return: (str) colored string
-        """
-        msg = ""
+        :return: Nothing.
+        :rtype: N\A
+        '''
         # Do nothing if there are no options
         if len(self.options) == 0:
-            return _("No available options.")
+            return
 
-        # for all options, gets its values. 
-        # "shortname" is an mandatory field of the options, could be '' 
-        msg += printcolors.printcHeader(_("Available options are:"))
+        # for all options, print its values. 
+        # "shortname" is an optional field of the options 
+        print(printcolors.printcHeader(_("Available options are:")))
         for option in self.options:
-            oos, ool, oot, ooh = self.getDetailOption(option)
-            if len(oos) > 0:
-                msg += "\n -%1s, --%s (%s)\n" % (oos, ool, oot)
+            if 'shortName' in option and len(option['shortName']) > 0:
+                print(" -%(shortName)1s, --%(longName)s"
+                      " (%(optionType)s)\n\t%(helpString)s\n" % option)
             else:
-                msg += "\n --%s (%s)\n" % (ool, oot)
-                
-            msg += "%s\n" % self.indent(ooh, 10)
-        return msg
+                print(" --%(longName)s (%(optionType)s)\n\t%(helpString)s\n"
+                       % option)
 
-    def print_help(self):
-        """
-        Method that display all options stored in self.options and there help
-        
-        :return: None
-        """
-        print(self.get_help())
-        return
-
-    def indent(self, text, amount, car=" "):
-        """indent multi lines message"""
-        padding = amount * car
-        return ''.join(padding + line for line in text.splitlines(True))
-               
     def parse_args(self, argList=None):
-        """
-        Instantiates the class OptResult 
-        that gives access to all options in the code
+        '''Method that instantiates the class OptResult 
+           that gives access to all options in the code
         
-        :param argList: (list) the raw list of arguments that were passed
-        :return: (OptResult, list) as (optResult, args) 
-          optResult is the option instance to manipulate in the code. 
-          args is the full raw list of passed options 
-        """
-        # see https://pymotw.com/2/getopt/
+        :param argList list: the raw list of arguments that were passed
+        :return: optResult, args : optResult is the option instance 
+                                   to manipulate in the code. args 
+                                   is the full raw list of passed options 
+        :rtype: (class 'common.options.OptResult',list)
+        '''
         if argList is None:
             argList = sys.argv[1:]
         
-        DBG.write("parse_args", argList)
-        # DBG.write("options", self.options)
         # format shortNameOption and longNameOption 
         # to make right arguments to getopt.getopt function
         shortNameOption = ""
         longNameOption = []
         for option in self.options:
             shortNameOption = shortNameOption + option['shortName']
-            if option['shortName'] != "" and option['optionType'] not in self.noArgOptions:
+            if option['shortName'] != "" and option['optionType'] != "boolean":
                 shortNameOption = shortNameOption + ":"
 
             if option['longName'] != "":
-                if option['optionType'] not in self.noArgOptions:
+                if option['optionType'] != "boolean":
                     longNameOption.append(option['longName'] + "=")
                 else:
                     longNameOption.append(option['longName'])
 
         # call to getopt.getopt function to get the option 
         # passed in the command regarding the available options
-        try:
-          optlist, args = getopt.getopt(argList, shortNameOption, longNameOption)
-        except Exception as e:
-          msg = str(e) + " on '%s'\n\n" % " ".join(argList) + self.get_help()
-          raise Exception(msg)
-
+        optlist, args = getopt.getopt(argList, shortNameOption, longNameOption)
+        
         # instantiate and completing the optResult that will be returned
         optResult = OptResult()
         for option in self.options:
@@ -230,8 +169,6 @@ class Options(object):
                         option['result'] = opt[1]
                     elif optionType == "boolean":
                         option['result'] = True
-                    elif optionType == "noboolean":
-                        option['result'] = False
                     elif optionType == "int":
                         option['result'] = int(opt[1])
                     elif optionType == "float":
@@ -242,67 +179,18 @@ class Options(object):
                         if option['result'] is None:
                             option['result'] = list()
                         option['result'].append(opt[1])
-                    elif optionType == "level": #logger logging levels
-                        option['result'] = self.filterLevel(opt[1])
                     elif optionType == "list2":
                         if option['result'] is None:
                             option['result'] = list()
-                        option['result'] = self.filterList2(opt[1])
+                        if opt[1].find(",") == -1:
+                            option['result'].append(opt[1])
+                        else:
+                            elts = filter(lambda l: len(l) > 0, opt[1].split(","))
+                            option['result'].extend(elts)
 
             optResult.__setattr__(option['destName'], option['result'])
             # free the option in order to be able to make 
             # a new free call of options (API case)
             option['result'] = None
-
-        self.results = {"optlist": optlist, "optResult": optResult, "args": args, "argList": argList}
-        DBG.write("results", self.results)
         return optResult, args
-        
-    def filterLevel(self, aLevel):
-      """filter level logging values"""
-      import src.loggingSat as LOG
-      aLev = aLevel.upper()
-      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)) 
-            return lev
-      msg = "Unknown level '%s', accepted are:\n%s" % (aLev, ", ".join(knownLevels))
-      raise Exception(msg)
-      
-    def filterList2(self, aStr):
-      """filter a list as 'KERNEL,YACS,etc.'"""
-      aList = aStr.strip().split(",")
-      # fix list leading ',' as ',KERNEL,...'
-      aList = [i for i in aList if i != ""] # split old list leadin "," as ",KERNEL,ETC..."
-      return aList
-      
-
-    def __repr__(self): 
-        """
-        repr for only self.options and self.results (if present)
-        """
-        aDict = {'options': self.options, 'results': self.results}
-        aStr = PP.pformat(aDict)
-        res = "%s(\n %s\n)" % (self.__class__.__name__, aStr[1:-1])
-        return res
-        
-    def __str__(self): 
-        """
-        str for only resume expected self.options
-        """
-        #aDict = [(k["longName"], k["shortName", k["helpString"]) for k in self.options}
-        #aList = [(k, self.options[k]) for k in sorted(self.options.keys())]
-        aDict = {}
-        for o in self.options:
-          aDict[o["longName"]] = (o["shortName"], o["helpString"])
-        aStr = PP.pformat(aDict)
-        res = "%s(\n %s)" % (self.__class__.__name__, aStr[1:-1])
-        return res
-        
-    def debug_write(self):
-        DBG.write("options and results", self, True)
-