Salome HOME
fix sat compile from detar problem file sat-product.pyconf evaluation
[tools/sat.git] / commands / clean.py
index 805fa9a3a10a69d9c00b532e3f6dade36a796cd0..1077297675980897a07e8c4cccf78ab6358fa329 100644 (file)
@@ -27,17 +27,12 @@ try:
 except NameError: 
     pass
 
-PROPERTY_EXPRESSION = "^.+:.+$"
 
 # Define all possible option for the clean command :  sat clean <options>
 parser = src.options.Options()
 parser.add_option('p', 'products', 'list2', 'products',
-    _('Optional: Products to clean. This option can be'
-    ' passed several time to clean several products.'))
-parser.add_option('', 'properties', 'string', 'properties',
-    _('Optional: Filter the products by their properties.\n\tSyntax: '
-      '--properties <property>:<value>'))
-parser.add_option('s', 'sources', 'boolean', 'sources', 
+    _('Optional: Products to clean. This option accepts a comma separated list.'))
+parser.add_option('s', 'sources', 'boolean', 'sources',
     _("Optional: Clean the product source directories."))
 parser.add_option('b', 'build', 'boolean', 'build', 
     _("Optional: Clean the product build directories."))
@@ -48,57 +43,19 @@ parser.add_option('a', 'all', 'boolean', 'all',
 parser.add_option('', 'sources_without_dev', 'boolean', 'sources_without_dev', 
     _("Optional: do not clean the products in development mode."))
 
-def get_products_list(options, cfg, logger):
-    '''method that gives the product list with their informations from 
-       configuration regarding the passed options.
-    
-    :param options Options: The Options instance that stores the commands 
-                            arguments
-    :param config Config: The global configuration
-    :param logger Logger: The logger instance to use for the display and logging
-    :return: The list of (product name, product_informations).
-    :rtype: List
-    '''
-    # Get the products to be prepared, regarding the options
-    if options.products is None:
-        # No options, get all products sources
-        products = cfg.APPLICATION.products
-    else:
-        # if option --products, check that all products of the command line
-        # are present in the application.
-        products = options.products
-        for p in products:
-            if p not in cfg.APPLICATION.products:
-                raise src.SatException(_("Product %(product)s "
-                            "not defined in application %(application)s") %
-                        { 'product': p, 'application': cfg.VARS.application} )
-    
-    # Construct the list of tuple containing 
-    # the products name and their definition
-    products_infos = src.product.get_products_infos(products, cfg)
-    
-    # if the property option was passed, filter the list
-    if options.properties:
-        [prop, value] = options.properties.split(":")
-        products_infos = [(p_name, p_info) for 
-                          (p_name, p_info) in products_infos 
-                          if "properties" in p_info and 
-                          prop in p_info.properties and 
-                          p_info.properties[prop] == value]
-        
-    return products_infos
 
 def get_source_directories(products_infos, without_dev):
-    '''Returns the list of directory source paths corresponding to the list of 
-       product information given as input. If without_dev (bool), then
-       the dev products are ignored.
+    """\
+    Returns the list of directory source paths corresponding 
+    to the list of product information given as input.
+    If without_dev (bool) the dev products are ignored.
     
     :param products_infos list: The list of (name, config) corresponding to one
                                 product.
     :param without_dev boolean: If True, then ignore the dev products.
     :return: the list of source paths.
     :rtype: list
-    '''
+    """
     l_dir_source = []
     for __, product_info in products_infos:
         if product_has_dir(product_info, without_dev):
@@ -106,14 +63,15 @@ def get_source_directories(products_infos, without_dev):
     return l_dir_source
 
 def get_build_directories(products_infos):
-    '''Returns the list of directory build paths corresponding to the list of 
-       product information given as input.
+    """\
+    Returns the list of directory build paths corresponding to the list of 
+    product information given as input.
     
     :param products_infos list: The list of (name, config) corresponding to one
                                 product.
     :return: the list of build paths.
     :rtype: list
-    '''
+    """
     l_dir_build = []
     for __, product_info in products_infos:
         if product_has_dir(product_info):
@@ -122,14 +80,14 @@ def get_build_directories(products_infos):
     return l_dir_build
 
 def get_install_directories(products_infos):
-    '''Returns the list of directory install paths corresponding to the list of 
-       product information given as input.
+    """\
+    Returns the list of directory install paths corresponding to the list of 
+    product information given as input.
     
-    :param products_infos list: The list of (name, config) corresponding to one
-                                product.
+    :param products_infos list: The list of (name, config) corresponding to one product.
     :return: the list of install paths.
     :rtype: list
-    '''
+    """
     l_dir_install = []
     for __, product_info in products_infos:
         if product_has_dir(product_info):
@@ -137,14 +95,14 @@ def get_install_directories(products_infos):
     return l_dir_install
 
 def product_has_dir(product_info, without_dev=False):
-    '''Returns a boolean at True if there is a source, build and install
-       directory corresponding to the product described by product_info.
+    """\
+    Returns a boolean at True if there is a source, build and install
+    directory corresponding to the product described by product_info.
     
     :param products_info Config: The config corresponding to the product.
-    :return: True if there is a source, build and install
-             directory corresponding to the product described by product_info.
+    :return: True if there is a source, build and install directory corresponding to the product described by product_info.
     :rtype: boolean
-    '''
+    """
     if (src.product.product_is_native(product_info) or 
                             src.product.product_is_fixed(product_info)):
         return False
@@ -154,12 +112,11 @@ def product_has_dir(product_info, without_dev=False):
     return True
     
 def suppress_directories(l_paths, logger):
-    '''Suppress the paths given in the list in l_paths.
+    """Suppress the paths given in the list in l_paths.
     
     :param l_paths list: The list of Path to be suppressed
-    :param logger Logger: The logger instance to use for the display and 
-                          logging
-    '''    
+    :param logger Logger: The logger instance to use for the display and logging
+    """    
     for path in l_paths:
         if not path.isdir():
             msg = _("Warning: the path %s does not "
@@ -171,20 +128,23 @@ def suppress_directories(l_paths, logger):
             logger.write('%s\n' % src.printcolors.printc(src.OK_STATUS), 3)
 
 def description():
-    '''method that is called when salomeTools is called with --help option.
+    """method called when salomeTools is called with --help option.
     
     :return: The text to display for the clean command description.
     :rtype: str
-    '''
-    return _("The clean command suppress the source, build, or install "
-             "directories of the application products.\nUse the options to"
-             " define what directories you want to suppress and to reduce "
-             "the list of products\n\nexample:\nsat clean SALOME-master "
-             "--build --install --properties is_salome_module:yes")
+    """
+    return _("""\
+The clean command suppress the SOURCES, BUILD or INSTALL directories of the application products.
+Use the options to define what directories you want to suppress and to set the list of products
+
+example:
+>> sat clean SALOME-xx --build --install --properties is_salome_module:yes
+""")
   
 def run(args, runner, logger):
-    '''method that is called when salomeTools is called with clean parameter.
-    '''
+    """\
+    method called when salomeTools is called with clean parameter.
+    """
     
     # Parse the options
     (options, args) = parser.parse_args(args)
@@ -192,19 +152,9 @@ def run(args, runner, logger):
     # check that the command has been called with an application
     src.check_config_has_application( runner.cfg )
 
-    # Verify the --properties option
-    if options.properties:
-        oExpr = re.compile(PROPERTY_EXPRESSION)
-        if not oExpr.search(options.properties):
-            msg = _('WARNING: the "--properties" options must have the '
-                    'following syntax:\n--properties <property>:<value>')
-            logger.write(src.printcolors.printcWarning(msg), 1)
-            logger.write("\n", 1)
-            options.properties = None
-            
 
     # Get the list of products to threat
-    products_infos = get_products_list(options, runner.cfg, logger)
+    products_infos = src.product.get_products_list(options, runner.cfg, logger)
 
     # Construct the list of directories to suppress
     l_dir_to_suppress = []