]> git.ipfire.org Git - thirdparty/git.git/commit - sequencer.c
sequencer: fix gpg option passed to merge subcommand
authorSamuel Čavoj <samuel@cavoj.net>
Sat, 17 Oct 2020 23:15:55 +0000 (01:15 +0200)
committerJunio C Hamano <gitster@pobox.com>
Sun, 18 Oct 2020 20:08:31 +0000 (13:08 -0700)
commitae03c97ac0e36a33183562464f9506c8b3a73060
treebb3dd39cac6bf83feedbe2f29a13e06de8409229
parentd4a392452e292ff924e79ec8458611c0f679d6d4
sequencer: fix gpg option passed to merge subcommand

When performing a rebase with --rebase-merges using either a custom
strategy specified with -s or an octopus merge, and at the same time
having gpgsign enabled (either rebase -S or config commit.gpgsign), the
operation would fail on making the merge commit. Instead of "-S%s" with
the key id substituted, only the bare key id would get passed to the
underlying merge command, which tried to interpret it as a ref.

Fix the issue and add test cases as suggested by Johannes Schindelin and
Junio C Hamano.

Signed-off-by: Samuel Čavoj <samuel@cavoj.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
sequencer.c
t/t3435-rebase-gpg-sign.sh