Commit fe839c6d authored by Aleksy Barcz's avatar Aleksy Barcz

Revert "i2smo: add brackets as invalid chars for name"

This reverts commit 013445da.

+ parentheses in param names break RPN formulas, so if we don't use a param in a RPN formula, we should be fine
+ if we use a param name containing parentheses in a RPN formula in <defined> or <drawdefinable> section, i2smo will emit and error anyway, as ipk2szarp will fail on parsing formula
+ it's not practical to enforce this rule on existing configurations as lots of params exist on existing SZARP sites and i2smo prints lots of confusing errors then
parent 1fe08b87
......@@ -291,8 +291,8 @@ def check_script(script_path):
def stat_shebang(exec_name):
return any(
os.access(os.path.join(path, exec_name), os.X_OK)
for path in os.environ["PATH"].split(os.pathsep)
os.access(os.path.join(path, exec_name), os.X_OK)
for path in os.environ["PATH"].split(os.pathsep)
)
def check_python_script(script_path):
......@@ -322,7 +322,7 @@ def check_if_parameter_exist():
isl_element = list(isl)
find_parameter_from_isl(isl_element, params_list, isl_file)
invalid_chars = set(['\\', '\"', '<', '>', '`', '~', '*', '(', ')'])
invalid_chars = set(['\\', '\"', '<', '>', '`', '~', '*'])
def validate_parameter_name(param):
name = param.get("name")
components = name.split(":")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment