From: Jonas Rebmann <kernel@schlaraffenlan.de>
To: git@vger.kernel.org
Cc: Chris Down <chris@chrisdown.name>, Jeff King <peff@peff.net>,
Jonas Rebmann <kernel@schlaraffenlan.de>,
Phillip Wood <phillip.wood@dunelm.org.uk>
Subject: [PATCH v3 0/2] Avoid hardcoded "good"/"bad" bisect terms
Date: Fri, 17 Apr 2026 18:48:29 +0200 [thread overview]
Message-ID: <20260417-bisect-terms-v3-0-d659fa547261@schlaraffenlan.de> (raw)
While checking whether all output messages of git bisect were covered by
[PATCH 1/2] bisect: use selected alternate terms in status output I
found hardcoded good/bad refs leading to incompatibility of git
rev-parse --bisect with alternate bisect run terms. This is addressed by
[PATCH 2/2] rev-parse: use selected alternate terms to look up refs
Signed-off-by: Jonas Rebmann <kernel@schlaraffenlan.de>
---
Changes in v3:
- when referencing newly introduced terms, reference them in single
quotes (Thanks, Phillip)
- Prefer test_grep over grep in updated Tests (Thanks, Phillip)
- Improve commit messages (Thanks, Phillip)
- Don't leak memory after read_bisect_terms() (Thanks, Phillip)
- Don't leak memory after xstrfmt() (Thanks, Junio)
- Add test case to patch 2/2
- Link to v2: https://patch.msgid.link/20260323-bisect-terms-v2-0-8d6bdb2c9c7e@schlaraffenlan.de
Changes in v2:
- Improve commit message
- Add tests
- Include second patch for hardcoded good/bad in rev-parse
- Link to v1: https://lore.kernel.org/r/20260320-bisect-terms-v1-1-c30c9540542a@schlaraffenlan.de
---
Jonas Rebmann (2):
bisect: use selected alternate terms in status output
rev-parse: use selected alternate terms to look up refs
builtin/bisect.c | 23 +++++++++++++----------
builtin/rev-parse.c | 15 +++++++++++++--
t/t1500-rev-parse.sh | 25 +++++++++++++++++++++++++
t/t6030-bisect-porcelain.sh | 38 +++++++++++++++++++++++++-------------
4 files changed, 76 insertions(+), 25 deletions(-)
---
base-commit: 1b296b0f55885fa8fc649c4b31c37f3d86f3f9cf
change-id: 20260320-bisect-terms-76036676769c
Best regards,
--
Jonas Rebmann <kernel@schlaraffenlan.de>
next reply other threads:[~2026-04-17 18:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-17 16:48 Jonas Rebmann [this message]
2026-04-17 16:48 ` [PATCH v3 1/2] bisect: use selected alternate terms in status output Jonas Rebmann
2026-04-17 19:50 ` Junio C Hamano
2026-04-17 16:48 ` [PATCH v3 2/2] rev-parse: use selected alternate terms to look up refs Jonas Rebmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260417-bisect-terms-v3-0-d659fa547261@schlaraffenlan.de \
--to=kernel@schlaraffenlan.de \
--cc=chris@chrisdown.name \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
--cc=phillip.wood@dunelm.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox