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

[PATCH v2 2/3] rebase -p: error out if --signoff is given

From
PWPhillip Wood <phillip.wood@talktalk.net>
Date
Mar 20, 2018, 11:10 UTC
Message-ID
<20180320111057.23862-3-phillip.wood@talktalk.net>
In-Reply-To
<20180320111057.23862-1-phillip.wood@talktalk.net>
From: Phillip Wood <phillip.wood@dunelm.org.uk>

rebase --preserve-merges does not support --signoff so error out rather than just silently ignoring it so that the user knows the commits will not be signed off.

Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
---
 git-rebase.sh | 2 ++
 1 file changed, 2 insertions(+)
diff --git a/git-rebase.sh b/git-rebase.sh
index 5d854657a7..7d3bb9d443 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -479,6 +479,8 @@ fi
 
 if test -n "$signoff"
 then
+	test -n "$preserve_merges" &&
+		die "$(gettext "error: cannot combine '--signoff' with '--preserve-merges'")"
 	git_am_opt="$git_am_opt $signoff"
 	force_rebase=t
 fi
-- 
2.16.2
Previous: Phillip WoodNext: Phillip Wood
Message 11 of 15 in “rebase: support --signoff with implicit rebase”
  1. 1/2 rebase: support --signoff with implicit rebasePhillip Wood, Mar 14, 2018
  2. 2/2 rebase --root -k: fix when root commit is emptyPhillip Wood, Mar 14, 2018
  3. Johannes SchindelinMar 15, 2018
  4. Phillip WoodMar 15, 2018
  5. Junio C HamanoMar 14, 2018
  6. Johannes SchindelinMar 15, 2018
  7. Johannes SchindelinMar 15, 2018
  8. Phillip WoodMar 15, 2018
  9. Johannes SchindelinMar 16, 2018
  10. 0/3 rebase: extend --signoff supportPhillip Wood, Mar 20, 2018
  11. 2/3 rebase -p: error out if --signoff is givenPhillip Wood, Mar 20, 2018
  12. 1/3 rebase: extend --signoff supportPhillip Wood, Mar 20, 2018
  13. 3/3 rebase --keep-empty: always use interactive rebasePhillip Wood, Mar 20, 2018
  14. Johannes SchindelinMar 20, 2018
  15. Phillip WoodMar 20, 2018

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.