modify_pc: Modify to remove regex syntax warning
authorSangYoun Kwak <sy.kwak@samsung.com>
Fri, 21 Jun 2024 08:17:05 +0000 (17:17 +0900)
committerSangYoun Kwak <sy.kwak@samsung.com>
Fri, 21 Jun 2024 09:26:47 +0000 (18:26 +0900)
The '\s' in the regex for python means whitespace. Also, '\' is used for
escape sequence, so '\s' is ambiguous.
In python, to remove this ambiguity, '\\s' is used for whitespace.

In modify_pc.py script, '\s' is used so python raises warning. To remove
this warning, '\s' is replaced with '\\s'.

Change-Id: Ic879fa46d5919082a4b6ab8a3294dff19e61a0dc
Signed-off-by: SangYoun Kwak <sy.kwak@samsung.com>
modify_pc.py
packaging/hal-rootstrap.spec

index b86e2ec..771d674 100755 (executable)
@@ -6,9 +6,9 @@ from re import compile as regex_compile
 
 include_dir_root_path = "hal_rootstrap_dir"
 
-regex_assignment = regex_compile("\s*([-_a-zA-Z0-9]+)\s*=\s*([^\s]*)")
-regex_declaration = regex_compile("\s*([-_a-zA-Z0-9]+)\s*:\s*(.+)\s*")
-regex_substitution = regex_compile("\${([-_a-zA-Z0-9]+)}")
+regex_assignment = regex_compile("\\s*([-_a-zA-Z0-9]+)\\s*=\\s*([^\\s]*)")
+regex_declaration = regex_compile("\\s*([-_a-zA-Z0-9]+)\\s*:\\s*(.+)\\s*")
+regex_substitution = regex_compile("\\${([-_a-zA-Z0-9]+)}")
 
 libs = { "-L${" + include_dir_root_path + "}" }
 cflags = { "-I${" + include_dir_root_path + "}" }
index 5e97714..8086583 100644 (file)
@@ -14,6 +14,7 @@ ExcludeArch:  riscv64
 
 BuildRequires: cmake
 BuildRequires: xmlstarlet
+BuildRequires: python3
 
 BuildRequires: capi-base-common
 BuildRequires: capi-base-common-devel