diff options
author | Jan Synacek <jsynacek@redhat.com> | 2019-01-30 12:22:41 +0100 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-02-10 19:06:58 +0100 |
commit | ed2dc503da57b0110819563e0d1c85d023435e07 (patch) | |
tree | 5e60dd824da44a4ad902b35684bc3ed2729e69cb /test/rule-syntax-check.py | |
parent | boot/efi: use a wildcard section copy for final EFI generation (diff) | |
download | systemd-ed2dc503da57b0110819563e0d1c85d023435e07.tar.xz systemd-ed2dc503da57b0110819563e0d1c85d023435e07.zip |
rule-syntax-check: allow PROGRAM as an assignment
Diffstat (limited to 'test/rule-syntax-check.py')
-rwxr-xr-x | test/rule-syntax-check.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/test/rule-syntax-check.py b/test/rule-syntax-check.py index dfb06d9ed9..8b26c988ab 100755 --- a/test/rule-syntax-check.py +++ b/test/rule-syntax-check.py @@ -17,6 +17,8 @@ if not rules_files: quoted_string_re = r'"(?:[^\\"]|\\.)*"' no_args_tests = re.compile(r'(ACTION|DEVPATH|KERNELS?|NAME|SYMLINK|SUBSYSTEMS?|DRIVERS?|TAG|PROGRAM|RESULT|TEST)\s*(?:=|!)=\s*' + quoted_string_re + '$') +# PROGRAM can also be specified as an assignment. +program_assign = re.compile(r'PROGRAM\s*=\s*' + quoted_string_re + '$') args_tests = re.compile(r'(ATTRS?|ENV|TEST){([a-zA-Z0-9/_.*%-]+)}\s*(?:=|!)=\s*' + quoted_string_re + '$') no_args_assign = re.compile(r'(NAME|SYMLINK|OWNER|GROUP|MODE|TAG|RUN|LABEL|GOTO|OPTIONS|IMPORT)\s*(?:\+=|:=|=)\s*' + quoted_string_re + '$') args_assign = re.compile(r'(ATTR|ENV|IMPORT|RUN){([a-zA-Z0-9/_.*%-]+)}\s*(=|\+=)\s*' + quoted_string_re + '$') @@ -51,7 +53,8 @@ for path in rules_files: for clause_match in comma_separated_group_re.finditer(line): clause = clause_match.group().strip() if not (no_args_tests.match(clause) or args_tests.match(clause) or - no_args_assign.match(clause) or args_assign.match(clause)): + no_args_assign.match(clause) or args_assign.match(clause) or + program_assign.match(clause)): print('Invalid line {}:{}: {}'.format(path, lineno, line)) print(' clause:', clause) |