src/rougail/annotator.py

@@ -75,7 +75,6 @@ KEY_TYPE = {'variable': 'symlink',
 TYPE_PARAM_CHECK = ('string', 'python', 'eole', 'variable')
 TYPE_PARAM_CONDITION = ('string', 'python', 'number', 'eole', 'variable')
 TYPE_PARAM_FILL = ('string', 'eole', 'number', 'context', 'variable')
-CONVERSION = {'number': int}
 
 ERASED_FAMILY_ACTION_ATTRIBUTES = ('index', 'action')
 
@@ -686,13 +685,7 @@ class SpaceAnnotator(object):
             choices = []
             for value in values:
                 choice = self.objectspace.choice()
-                try:
-                    if type_ in CONVERSION:
-                        choice.name = CONVERSION[type_](value)
-                    else:
                 choice.name = str(value)
-                except:
-                    raise CreoleDictConsistencyError(_(f'unable to change type of a valid_enum entry "{value}" is not a valid "{type_}" for "{variable.name}"'))
                 choices.append(choice.name)
                 choice.type = type_
                 variable.choice.append(choice)
@@ -701,11 +694,7 @@ class SpaceAnnotator(object):
             if hasattr(variable, 'value'):
                 for value in variable.value:
                     value.type = type_
-                    if type_ in CONVERSION:
-                        cvalue = CONVERSION[type_](value.name)
-                    else:
-                        cvalue = value.name
-                    if cvalue not in choices:
+                    if value.name not in choices:
                         raise CreoleDictConsistencyError(_('value "{}" of variable "{}" is not in list of all expected values ({})').format(value.name, variable.name, choices))
             else:
                 new_value = self.objectspace.value()

src/rougail/i18n.py

@@ -44,8 +44,8 @@ mo_location = LOCALE_DIR
 
 gettext.find(APP_NAME, mo_location)
 gettext.textdomain(APP_NAME)
-#gettext.bind_textdomain_codeset(APP_NAME, "UTF-8")
-#gettext.translation(APP_NAME, fallback=True)
+gettext.bind_textdomain_codeset(APP_NAME, "UTF-8")
+gettext.translation(APP_NAME, fallback=True)
 
 t = gettext.translation(APP_NAME, fallback=True)
 

src/rougail/template.py

@@ -262,17 +262,13 @@ class CreoleTemplateEngine:
                  distrib_dir: str,
                  tmp_dir: str,
                  dest_dir: str,
-                 override_dest_dir: str,
                  tmpfile_name: str,
-                 factory_prefix: str,
                  ) -> None:
         self.config = config
         self.dest_dir = dest_dir
-        self.override_dest_dir = override_dest_dir
         self.tmp_dir = tmp_dir
         self.distrib_dir = distrib_dir
         self.tmpfile_name = tmpfile_name
-        self.factory_prefix = factory_prefix
         eos = {}
         if eosfunc_file is not None:
             eosfunc = imp.load_source('eosfunc', eosfunc_file)
@@ -386,7 +382,7 @@ class CreoleTemplateEngine:
         else:
             variable = None
         if override:
-            filenames = [f'/systemd/system/{service_name}.service.d/rougail.conf']
+            filenames = [f'/system/{service_name}.service.d/rougail.conf']
         else:
             filenames = filevar['name']
             if not isinstance(filenames, list):
@@ -394,10 +390,8 @@ class CreoleTemplateEngine:
                 if variable:
                     variable = [variable]
         for idx, filename in enumerate(filenames):
-            if override:
-                destfilename = join(self.override_dest_dir, filename[1:])
-            else:
-                destfilename = join(self.dest_dir, filename[1:])
+            destfilename = join(self.dest_dir,
+                                filename[1:])
             makedirs(dirname(destfilename), exist_ok=True)
             if variable:
                 var = variable[idx]
@@ -412,7 +406,7 @@ class CreoleTemplateEngine:
             else:
                 copy(source, destfilename)
             if not override and self.tmpfile_name:
-                systemd_rights.append(f'C {filename} {filevar["mode"]} {filevar["owner"]} {filevar["group"]} - {self.factory_prefix}{filename}')
+                systemd_rights.append(f'C {filename} {filevar["mode"]} {filevar["owner"]} {filevar["group"]} - -')
                 systemd_rights.append(f'z {filename} - - - - -')
 
     async def instance_files(self) -> None:
@@ -461,22 +455,11 @@ async def generate(config: Config,
                    distrib_dir: str,
                    tmp_dir: str,
                    dest_dir: str,
-                   override_dest_dir: str,
-                   tmpfile_name: str=None,
-                   factory_prefix: str=None,
-                   ) -> None:
-    if not tmpfile_name and factory_prefix:
-        raise Exception(_(f'only specify factory_prefix if tmpfile_name is set'))
-    if tmpfile_name and not factory_prefix:
-        raise Exception(_(f'if tmpfile_name is specify, set factory_prefix too'))
-
+                   tmpfile_name: str=None) -> None:
     engine = CreoleTemplateEngine(config,
                                   eosfunc_file,
                                   distrib_dir,
                                   tmp_dir,
                                   dest_dir,
-                                  override_dest_dir,
-                                  tmpfile_name,
-                                  factory_prefix,
-                                  )
+                                  tmpfile_name)
     await engine.instance_files()

tests/flattener_dicos/88valid_enum_not_number/00-base.xml

@@ -17,7 +17,7 @@
 
     <constraints>
         <check name="valid_enum" target="enumvar">
-            <param>['a', 'b', 'c']</param>
+            <param>[i for i in range(3, 13)]</param>
         </check>
     </constraints>
 

tests/flattener_dicos/88valid_enum_numberdefaultstring/00-base.xml

@@ -1,32 +0,0 @@
-<?xml version='1.0' encoding='UTF-8'?>
-<rougail>
-
-    <services/>
-
-    <variables>
-        <family name="general" mode="expert">
-            <variable name="mode_conteneur_actif" type="oui/non" description="No change">
-                <value>non</value>
-            </variable>
-        </family>
-        <family name="enumfam" mode="expert">
-            <variable name="enumvar" type="string" description="enumvar">
-                <value>a</value>
-            </variable>
-        </family>
-        <separators/>
-    </variables>
-
-    <constraints>
-        <check name="valid_enum" target="enumvar">
-            <param>[1, 2, 3]</param>
-        </check>
-    </constraints>
-
-    <help>
-        <variable name="enumvar">bla bla bla</variable>
-    </help>
-
-</rougail>
-<!-- vim: ts=4 sw=4 expandtab
--->

tests/test_flattener.py

@@ -83,7 +83,7 @@ def launch_flattener(test_dir, test_ok=False):
     eolobj.save(destfile)
     result_file = join(test_dir, 'result/00-base.xml')
     if isfile(result_file):
-        #eolobj.save(result_file)
+        eolobj.save(result_file)
         compare_xml(destfile, result_file)
     elif test_ok:
         raise Exception(f'no test found for {test_dir}')