git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[GSoC][RFC PATCH 2/6] builtin/log: die if -k and --suject-extra-prefix are used together

From
Lucas Seiki Oshiro <lucasseikioshiro@gmail.com>
Date
Mar 3, 2025, 22:00 UTC
Message-ID
<20250303220029.10716-3-lucasseikioshiro@gmail.com>
In-Reply-To
<20250303220029.10716-1-lucasseikioshiro@gmail.com>

When calling format-patch, -k keeps the commit title as the patch subject and can't be used with other options that depend on the existence of a prefix. This way, -k shouldn't also be used with --subject-extra-prefix, as it introduces an extra prefix.

If -k and --subject-extra-prefix are used together, die.
Signed-off-by: Lucas Seiki Oshiro <lucasseikioshiro@gmail.com>
---
 builtin/log.c | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/builtin/log.c b/builtin/log.c
index 95667c93ca..52dc0301cf 100644
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -2256,6 +2256,8 @@ int cmd_format_patch(int argc,
 		die(_("options '%s' and '%s' cannot be used together"), "-n", "-k");
 	if (cfg.keep_subject && cfg.subject_prefix)
 		die(_("options '%s' and '%s' cannot be used together"), "--subject-prefix/--rfc", "-k");
+	if (cfg.keep_subject && extra_prefix)
+		die(_("options '%s' and '%s' cannot be used together"), "--subject-extra-prefix", "-k");
 	rev.preserve_subject = cfg.keep_subject;
 
 	argc = setup_revisions(argc, argv, &rev, &s_r_opt);
-- 
2.39.5 (Apple Git-154)
Previous: Lucas Seiki OshiroNext: Eric Sunshine
Message 3 of 15 in “Add --subject-extra-prefix flag to format-patch”
  1. Lucas Seiki OshiroMar 3, 2025
  2. [GSoC][RFC PATCH 1/6] builtin/log: add subject-extra-prefix flag to format-patchLucas Seiki Oshiro, Mar 3, 2025
  3. [GSoC][RFC PATCH 2/6] builtin/log: die if -k and --suject-extra-prefix are used togetherLucas Seiki Oshiro, Mar 3, 2025
  4. Eric SunshineMar 3, 2025
  5. [GSoC][RFC PATCH 3/6] revision: add subject_extra_field to struct rev_infoLucas Seiki Oshiro, Mar 3, 2025
  6. [GSoC][RFC PATCH 4/6] builtin/log: fill subject extra prefix in format-patchLucas Seiki Oshiro, Mar 3, 2025
  7. [GSoC][RFC PATCH 5/6] log-tree: add subject prefix in output email subjectLucas Seiki Oshiro, Mar 3, 2025
  8. [GSoC][RFC PATCH 6/6] t4014: add tests for the new flag --subject-extra-prefixLucas Seiki Oshiro, Mar 3, 2025
  9. Junio C HamanoMar 3, 2025
  10. Patrick SteinhardtMar 4, 2025
  11. Junio C HamanoMar 4, 2025
  12. Junio C HamanoMar 4, 2025
  13. Lucas Seiki OshiroMar 4, 2025
  14. Junio C HamanoMar 5, 2025
  15. Lucas Seiki OshiroMar 5, 2025

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.