Merge branch 'bc/sha-256-part-3'
The final leg of SHA-256 transition. * bc/sha-256-part-3: (39 commits) t: remove test_oid_init in tests docs: add documentation for extensions.objectFormat ci: run tests with SHA-256 t: make SHA1 prerequisite depend on default hash t: allow testing different hash algorithms via environment t: add test_oid option to select hash algorithm repository: enable SHA-256 support by default setup: add support for reading extensions.objectformat bundle: add new version for use with SHA-256 builtin/verify-pack: implement an --object-format option http-fetch: set up git directory before parsing pack hashes t0410: mark test with SHA1 prerequisite t5308: make test work with SHA-256 t9700: make hash size independent t9500: ensure that algorithm info is preserved in config t9350: make hash size independent t9301: make hash size independent t9300: use $ZERO_OID instead of hard-coded object ID t9300: abstract away SHA-1-specific constants t8011: make hash size independent ...
This commit is contained in:
@ -7,21 +7,26 @@
|
||||
#define VERIFY_PACK_VERBOSE 01
|
||||
#define VERIFY_PACK_STAT_ONLY 02
|
||||
|
||||
static int verify_one_pack(const char *path, unsigned int flags)
|
||||
static int verify_one_pack(const char *path, unsigned int flags, const char *hash_algo)
|
||||
{
|
||||
struct child_process index_pack = CHILD_PROCESS_INIT;
|
||||
const char *argv[] = {"index-pack", NULL, NULL, NULL };
|
||||
struct strvec *argv = &index_pack.args;
|
||||
struct strbuf arg = STRBUF_INIT;
|
||||
int verbose = flags & VERIFY_PACK_VERBOSE;
|
||||
int stat_only = flags & VERIFY_PACK_STAT_ONLY;
|
||||
int err;
|
||||
|
||||
strvec_push(argv, "index-pack");
|
||||
|
||||
if (stat_only)
|
||||
argv[1] = "--verify-stat-only";
|
||||
strvec_push(argv, "--verify-stat-only");
|
||||
else if (verbose)
|
||||
argv[1] = "--verify-stat";
|
||||
strvec_push(argv, "--verify-stat");
|
||||
else
|
||||
argv[1] = "--verify";
|
||||
strvec_push(argv, "--verify");
|
||||
|
||||
if (hash_algo)
|
||||
strvec_pushf(argv, "--object-format=%s", hash_algo);
|
||||
|
||||
/*
|
||||
* In addition to "foo.pack" we accept "foo.idx" and "foo";
|
||||
@ -31,9 +36,8 @@ static int verify_one_pack(const char *path, unsigned int flags)
|
||||
if (strbuf_strip_suffix(&arg, ".idx") ||
|
||||
!ends_with(arg.buf, ".pack"))
|
||||
strbuf_addstr(&arg, ".pack");
|
||||
argv[2] = arg.buf;
|
||||
strvec_push(argv, arg.buf);
|
||||
|
||||
index_pack.argv = argv;
|
||||
index_pack.git_cmd = 1;
|
||||
|
||||
err = run_command(&index_pack);
|
||||
@ -60,12 +64,15 @@ int cmd_verify_pack(int argc, const char **argv, const char *prefix)
|
||||
{
|
||||
int err = 0;
|
||||
unsigned int flags = 0;
|
||||
const char *object_format = NULL;
|
||||
int i;
|
||||
const struct option verify_pack_options[] = {
|
||||
OPT_BIT('v', "verbose", &flags, N_("verbose"),
|
||||
VERIFY_PACK_VERBOSE),
|
||||
OPT_BIT('s', "stat-only", &flags, N_("show statistics only"),
|
||||
VERIFY_PACK_STAT_ONLY),
|
||||
OPT_STRING(0, "object-format", &object_format, N_("hash"),
|
||||
N_("specify the hash algorithm to use")),
|
||||
OPT_END()
|
||||
};
|
||||
|
||||
@ -75,7 +82,7 @@ int cmd_verify_pack(int argc, const char **argv, const char *prefix)
|
||||
if (argc < 1)
|
||||
usage_with_options(verify_pack_usage, verify_pack_options);
|
||||
for (i = 0; i < argc; i++) {
|
||||
if (verify_one_pack(argv[i], flags))
|
||||
if (verify_one_pack(argv[i], flags, object_format))
|
||||
err = 1;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user