]> git.ipfire.org Git - thirdparty/git.git/commit
replay: die descriptively when invalid commit-ish
authorKristoffer Haugsbakk <code@khaugsbakk.name>
Mon, 22 Dec 2025 22:04:42 +0000 (23:04 +0100)
committerJunio C Hamano <gitster@pobox.com>
Tue, 23 Dec 2025 02:51:39 +0000 (11:51 +0900)
commit5be4d46eeeeee1576b4c8e00d99240aae6949d99
treedc341239b3f4cafee75d6c8d3a20b34b814dcb27
parent66ce5f8e8872f0183bb137911c52b07f1f242d13
replay: die descriptively when invalid commit-ish

Giving an invalid commit-ish to `--onto` or `--advance` makes
git-replay(1) fail with:

    fatal: Replaying down to root commit is not supported yet!

Going backwards from this point:

1. `onto` is `NULL` from `determine_replay_mode`;
2. that function in turn calls `peel_committish`; and
3. here we return `NULL` if `repo_get_oid` fails.

Let’s die immediately with a descriptive error message instead.

Doing this also provides us with a descriptive error if we “forget” to
provide an argument to `--onto` (but we really do unintentionally):[1]

    $ git replay --onto ^main topic1
    fatal: '^main' is not a valid commit-ish

† 1: The argument to `--onto` is mandatory and the option parser accepts
     both `--onto=<name>` (stuck form) and `--onto name`. The latter
     form makes it easy to unintentionally pass something to the option
     when you really meant to pass a positional argument.

Signed-off-by: Kristoffer Haugsbakk <code@khaugsbakk.name>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/replay.c
t/t3650-replay-basics.sh