]> git.ipfire.org Git - thirdparty/git.git/commit - wt-status.h
status: fix branch shown when not only bisecting
authorRubén Justo <rjusto@gmail.com>
Sat, 9 Sep 2023 20:12:47 +0000 (22:12 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 16 Oct 2023 22:05:27 +0000 (15:05 -0700)
commit990adccbdf0d8d1056c49a5ed546e744f542ba8f
tree40928cf1aad4e70c8338bed47421e2c48c511d42
parentfb7d80edcae482f4fa5d4be0227dc3054734e5f3
status: fix branch shown when not only bisecting

In 83c750acde (wt-status.*: better advice for git status added,
2012-06-05), git-status received new informative messages to describe
the ongoing work in a worktree.

These messages were enhanced in 0722c805d6 (status: show the branch name
if possible in in-progress info, 2013-02-03), to show, if possible, the
branch where the operation was initiated.

Since then, we show incorrect information when several operations are in
progress and one of them is bisect:

   $ git checkout -b foo
   $ GIT_SEQUENCE_EDITOR='echo break >' git rebase -i HEAD~
   $ git checkout -b bar
   $ git bisect start
   $ git status
   ...

   You are currently editing a commit while rebasing branch 'bar' on '...'.

   You are currently bisecting, started from branch 'bar'.

   ...

Note that we erroneously say "while rebasing branch 'bar'" when we
should be referring to "foo".

This must have gone unnoticed for so long because it must be unusual to
start a bisection while another operation is in progress.  And even less
usual to involve different branches.

It caught my attention reviewing a leak introduced in 8b87cfd000
(wt-status: move strbuf into read_and_strip_branch(), 2013-03-16).

A simple change to deal with this situation can be to record in struct
wt_status_state, the branch where the bisect starts separately from the
branch related to other operations.

Let's do it and so we'll be able to display correct information and
we'll avoid the leak as well.

Signed-off-by: Rubén Justo <rjusto@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
branch.c
ref-filter.c
t/t7512-status-help.sh
worktree.c
wt-status.c
wt-status.h