Skip to content

Commit 73fce29

Browse files
dschottaylorr
authored andcommitted
Turn git bisect into a full built-in
Now that the shell script hands off to the `bisect--helper` to do _anything_ (except to show the help), it is but a tiny step to let the helper implement the actual `git bisect` command instead. This retires `git-bisect.sh`, concluding a multi-year journey that many hands helped with, in particular Pranit Bauna, Tanushree Tumane and Miriam Rubio. Signed-off-by: Johannes Schindelin <[email protected]> Signed-off-by: Junio C Hamano <[email protected]> Signed-off-by: Đoàn Trần Công Danh <[email protected]> Signed-off-by: Taylor Blau <[email protected]>
1 parent 0da4b53 commit 73fce29

File tree

4 files changed

+4
-5
lines changed

4 files changed

+4
-5
lines changed

Makefile

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -627,7 +627,6 @@ THIRD_PARTY_SOURCES =
627627
# interactive shell sessions without exporting it.
628628
unexport CDPATH
629629

630-
SCRIPT_SH += git-bisect.sh
631630
SCRIPT_SH += git-difftool--helper.sh
632631
SCRIPT_SH += git-filter-branch.sh
633632
SCRIPT_SH += git-merge-octopus.sh
@@ -1137,7 +1136,7 @@ BUILTIN_OBJS += builtin/am.o
11371136
BUILTIN_OBJS += builtin/annotate.o
11381137
BUILTIN_OBJS += builtin/apply.o
11391138
BUILTIN_OBJS += builtin/archive.o
1140-
BUILTIN_OBJS += builtin/bisect--helper.o
1139+
BUILTIN_OBJS += builtin/bisect.o
11411140
BUILTIN_OBJS += builtin/blame.o
11421141
BUILTIN_OBJS += builtin/branch.o
11431142
BUILTIN_OBJS += builtin/bugreport.o

builtin.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ int cmd_am(int argc, const char **argv, const char *prefix);
116116
int cmd_annotate(int argc, const char **argv, const char *prefix);
117117
int cmd_apply(int argc, const char **argv, const char *prefix);
118118
int cmd_archive(int argc, const char **argv, const char *prefix);
119-
int cmd_bisect__helper(int argc, const char **argv, const char *prefix);
119+
int cmd_bisect(int argc, const char **argv, const char *prefix);
120120
int cmd_blame(int argc, const char **argv, const char *prefix);
121121
int cmd_branch(int argc, const char **argv, const char *prefix);
122122
int cmd_bugreport(int argc, const char **argv, const char *prefix);

builtin/bisect--helper.c renamed to builtin/bisect.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1401,7 +1401,7 @@ static int cmd_bisect__run(int argc, const char **argv, const char *prefix UNUSE
14011401
return res;
14021402
}
14031403

1404-
int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
1404+
int cmd_bisect(int argc, const char **argv, const char *prefix)
14051405
{
14061406
int res = 0;
14071407
parse_opt_subcommand_fn *fn = NULL;

git.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -492,7 +492,7 @@ static struct cmd_struct commands[] = {
492492
{ "annotate", cmd_annotate, RUN_SETUP },
493493
{ "apply", cmd_apply, RUN_SETUP_GENTLY },
494494
{ "archive", cmd_archive, RUN_SETUP_GENTLY },
495-
{ "bisect--helper", cmd_bisect__helper, RUN_SETUP },
495+
{ "bisect", cmd_bisect, RUN_SETUP },
496496
{ "blame", cmd_blame, RUN_SETUP },
497497
{ "branch", cmd_branch, RUN_SETUP | DELAY_PAGER_CONFIG },
498498
{ "bugreport", cmd_bugreport, RUN_SETUP_GENTLY },

0 commit comments

Comments
 (0)