From 7bc5ea80c7a2e60f8c47ac33b20909e693de5bbc Mon Sep 17 00:00:00 2001 From: Daniel Lang Date: Mon, 11 Sep 2023 05:59:44 +0200 Subject: [PATCH] utils/getdeveloperlib.py: handle file removal If a patch only removes files, it is ignored. Meaning, that the registered developer isn't automatically picked up when calling get-developer. Fix this by also checking if the line starts with ---, as a patch removing a file has a line starting with --- with the name of the removed file and one started with +++ /dev/null. A set is used to store the changed files, which doesn't allow duplicates. Therefore normal patches aren't affected by this change. Signed-off-by: Daniel Lang Signed-off-by: Thomas Petazzoni --- utils/getdeveloperlib.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/getdeveloperlib.py b/utils/getdeveloperlib.py index e7d0d23e49..469360b801 100644 --- a/utils/getdeveloperlib.py +++ b/utils/getdeveloperlib.py @@ -25,7 +25,7 @@ def analyze_patch(patch): m = FIND_INFRA_IN_PATCH.match(line) if m: infras.add(m.group(2)) - if not line.startswith("+++ "): + if not line.startswith("+++ ") and not line.startswith("--- "): continue line.strip() fname = line[line.find("/") + 1:].strip()