[Openvpn-devel] gerrit-send-mail.py: Add patch version to subject

Message ID 20231025122039.1031284-1-frank@lichtenheld.com
State Accepted
Headers show
Series [Openvpn-devel] gerrit-send-mail.py: Add patch version to subject | expand

Commit Message

Frank Lichtenheld Oct. 25, 2023, 12:20 p.m. UTC
Change-Id: I75403dfbebeeb4d667c7dd6b8276c6a4f2ae4842
Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
---
 dev-tools/gerrit-send-mail.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Gert Doering Oct. 29, 2023, 6:52 p.m. UTC | #1
Acked-by: Gert Doering <gert@greenie.muc.de>

Thanks, this is useful :-) - I haven't actually tested it yet, just
looked at the change (and the results of you using it on mbedtls v7...).

Your patch has been applied to the master branch.

commit ba5ed490d3844705cc950dc2eae8c6a92938e757 (master)
Author: Frank Lichtenheld
Date:   Wed Oct 25 14:20:39 2023 +0200

     gerrit-send-mail.py: Add patch version to subject

     Signed-off-by: Frank Lichtenheld <frank@lichtenheld.com>
     Acked-by: Gert Doering <gert@greenie.muc.de>
     Message-Id: <20231025122039.1031284-1-frank@lichtenheld.com>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg27296.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering

Patch

diff --git a/dev-tools/gerrit-send-mail.py b/dev-tools/gerrit-send-mail.py
index 851a20a0..5429aef6 100755
--- a/dev-tools/gerrit-send-mail.py
+++ b/dev-tools/gerrit-send-mail.py
@@ -108,8 +108,9 @@  Acked-by according to Gerrit (reflected above):
         + patch_text[comment_start:]
     )
     filename = f"gerrit-{args.changeid}-{details['revision']}.patch"
+    patch_text_final = patch_text_mod.replace("Subject: [PATCH]", f"Subject: [PATCH v{details['revision']}]")
     with open(filename, "w") as patch_file:
-        patch_file.write(patch_text_mod)
+        patch_file.write(patch_text_final)
     print("send with:")
     print(f"git send-email --in-reply-to {details['msg_id']} {filename}")