Skip to content

Commit 90af0b1

Browse files
committed
cc_maintainers: convert all addresses to lower case
Heiner reports false positives when CC list and MAINTAINERS differ on case. Signed-off-by: Jakub Kicinski <[email protected]>
1 parent 471071a commit 90af0b1

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

form-letters/net-next-closed

+1
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@ RFC patches sent for review only are obviously welcome at any time.
99
See: https://www.kernel.org/doc/html/next/process/maintainer-netdev.html#development-cycle
1010
--
1111
pw-bot: defer
12+
pv-bot: closed

tests/patch/cc_maintainers/test.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ def cc_maintainers(tree, thing, result_dir) -> Tuple[int, str, str]:
148148
addrs += msg.get_all('cc', [])
149149
addrs += msg.get_all('from', [])
150150
addrs += msg.get_all('sender', [])
151-
included = set([e for n, e in email.utils.getaddresses(addrs)])
151+
included = set([e.lower() for n, e in email.utils.getaddresses(addrs)])
152152
out += ["=== Email ===",
153153
f"From: {msg.get_all('from')}",
154154
f"Included: {included}", ""]
@@ -174,7 +174,7 @@ def cc_maintainers(tree, thing, result_dir) -> Tuple[int, str, str]:
174174
raw_gm.append(line.strip())
175175
match = emailpat.search(line)
176176
if match:
177-
addr = match.group(1)
177+
addr = match.group(1).lower()
178178
expected.add(addr)
179179
if 'blamed_fixes' in line:
180180
blamed.add(addr)
@@ -226,7 +226,7 @@ def cc_maintainers(tree, thing, result_dir) -> Tuple[int, str, str]:
226226
continue
227227
for have in included:
228228
if have in mmap_emails:
229-
mapped.add(m)
229+
mapped.add(m.lower())
230230

231231
found.update(mapped)
232232
missing.difference_update(mapped)

0 commit comments

Comments
 (0)