ecb3f3733c
Reimplement the `write_terms` shell function in C and add a `write-terms` subcommand to `git bisect--helper` to call it from git-bisect.sh . Also remove the subcommand `--check-term-format` as it can now be called from inside the function write_terms() C implementation. Also `|| exit` is added when calling write-terms subcommand from git-bisect.sh so as to exit whenever there is an error. Using `--write-terms` subcommand is a temporary measure to port shell function to C so as to use the existing test suite. As more functions are ported, this subcommand will be retired and its implementation will be called by some other method. Mentored-by: Lars Schneider <larsxschneider@gmail.com> Mentored-by: Christian Couder <chriscool@tuxfamily.org> Signed-off-by: Pranit Bauva <pranit.bauva@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
117 lines
2.9 KiB
C
117 lines
2.9 KiB
C
#include "builtin.h"
|
|
#include "cache.h"
|
|
#include "parse-options.h"
|
|
#include "bisect.h"
|
|
#include "refs.h"
|
|
|
|
static GIT_PATH_FUNC(git_path_bisect_terms, "BISECT_TERMS")
|
|
|
|
static const char * const git_bisect_helper_usage[] = {
|
|
N_("git bisect--helper --next-all [--no-checkout]"),
|
|
N_("git bisect--helper --write-terms <bad_term> <good_term>"),
|
|
NULL
|
|
};
|
|
|
|
/*
|
|
* Check whether the string `term` belongs to the set of strings
|
|
* included in the variable arguments.
|
|
*/
|
|
LAST_ARG_MUST_BE_NULL
|
|
static int one_of(const char *term, ...)
|
|
{
|
|
int res = 0;
|
|
va_list matches;
|
|
const char *match;
|
|
|
|
va_start(matches, term);
|
|
while (!res && (match = va_arg(matches, const char *)))
|
|
res = !strcmp(term, match);
|
|
va_end(matches);
|
|
|
|
return res;
|
|
}
|
|
|
|
static int check_term_format(const char *term, const char *orig_term)
|
|
{
|
|
int res;
|
|
char *new_term = xstrfmt("refs/bisect/%s", term);
|
|
|
|
res = check_refname_format(new_term, 0);
|
|
free(new_term);
|
|
|
|
if (res)
|
|
return error(_("'%s' is not a valid term"), term);
|
|
|
|
if (one_of(term, "help", "start", "skip", "next", "reset",
|
|
"visualize", "replay", "log", "run", "terms", NULL))
|
|
return error(_("can't use the builtin command '%s' as a term"), term);
|
|
|
|
/*
|
|
* In theory, nothing prevents swapping completely good and bad,
|
|
* but this situation could be confusing and hasn't been tested
|
|
* enough. Forbid it for now.
|
|
*/
|
|
|
|
if ((strcmp(orig_term, "bad") && one_of(term, "bad", "new", NULL)) ||
|
|
(strcmp(orig_term, "good") && one_of(term, "good", "old", NULL)))
|
|
return error(_("can't change the meaning of the term '%s'"), term);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int write_terms(const char *bad, const char *good)
|
|
{
|
|
FILE *fp = NULL;
|
|
int res;
|
|
|
|
if (!strcmp(bad, good))
|
|
return error(_("please use two different terms"));
|
|
|
|
if (check_term_format(bad, "bad") || check_term_format(good, "good"))
|
|
return -1;
|
|
|
|
fp = fopen(git_path_bisect_terms(), "w");
|
|
if (!fp)
|
|
return error_errno(_("could not open the file BISECT_TERMS"));
|
|
|
|
res = fprintf(fp, "%s\n%s\n", bad, good);
|
|
res |= fclose(fp);
|
|
return (res < 0) ? -1 : 0;
|
|
}
|
|
|
|
int cmd_bisect__helper(int argc, const char **argv, const char *prefix)
|
|
{
|
|
enum {
|
|
NEXT_ALL = 1,
|
|
WRITE_TERMS
|
|
} cmdmode = 0;
|
|
int no_checkout = 0;
|
|
struct option options[] = {
|
|
OPT_CMDMODE(0, "next-all", &cmdmode,
|
|
N_("perform 'git bisect next'"), NEXT_ALL),
|
|
OPT_CMDMODE(0, "write-terms", &cmdmode,
|
|
N_("write the terms to .git/BISECT_TERMS"), WRITE_TERMS),
|
|
OPT_BOOL(0, "no-checkout", &no_checkout,
|
|
N_("update BISECT_HEAD instead of checking out the current commit")),
|
|
OPT_END()
|
|
};
|
|
|
|
argc = parse_options(argc, argv, prefix, options,
|
|
git_bisect_helper_usage, 0);
|
|
|
|
if (!cmdmode)
|
|
usage_with_options(git_bisect_helper_usage, options);
|
|
|
|
switch (cmdmode) {
|
|
case NEXT_ALL:
|
|
return bisect_next_all(prefix, no_checkout);
|
|
case WRITE_TERMS:
|
|
if (argc != 2)
|
|
return error(_("--write-terms requires two arguments"));
|
|
return write_terms(argv[0], argv[1]);
|
|
default:
|
|
return error("BUG: unknown subcommand '%d'", cmdmode);
|
|
}
|
|
return 0;
|
|
}
|