From: Mario Limonciello <mario.limonciello@amd.com>
Date: Thu, 19 Oct 2023 17:16:32 +0000 (-0500)
Subject: Fix the robot email script
X-Git-Tag: 20231111~15^2
X-Git-Url: https://git.proxmox.com/?p=mirror_linux-firmware.git;a=commitdiff_plain;h=6ed754659e1540f0df58f0b4272f78f39f731a14

Fix the robot email script

```
Traceback (most recent call last):
  File "~/linux-firmware-robot/./contrib/process_linux_firmware.py", line 339, in <module>
    process_database(conn, remote)
  File "~/linux-firmware-robot/./contrib/process_linux_firmware.py", line 276, in process_database
    reply_email(mbox, branch)
  File "~/linux-firmware-robot/./contrib/process_linux_firmware.py", line 86, in reply_email
    reply["To"] += email.utils.formataddr(target)
TypeError: unsupported operand type(s) for +=: 'NoneType' and 'str'
```

Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
---

diff --git a/contrib/process_linux_firmware.py b/contrib/process_linux_firmware.py
index 4ef0aa76..2b7c7f8d 100755
--- a/contrib/process_linux_firmware.py
+++ b/contrib/process_linux_firmware.py
@@ -79,11 +79,12 @@ def reply_email(content, branch):
     reply = email.message.EmailMessage()
 
     orig = email.message_from_string(content)
-    targets = email.utils.getaddresses(
-        orig.get_all("to", []) + orig.get_all("cc", []) + orig.get_all("from", [])
+    reply["To"] = ", ".join(
+        email.utils.formataddr(t)
+        for t in email.utils.getaddresses(
+            orig.get_all("from", []) + orig.get_all("to", []) + orig.get_all("cc", [])
+        )
     )
-    for target in targets:
-        reply["To"] += email.utils.formataddr(target)
 
     reply["From"] = "linux-firmware@kernel.org"
     reply["Subject"] = "Re: {}".format(orig["Subject"])