Merge branch 'rs/submodule-config-code-cleanup'

Code cleanup.

* rs/submodule-config-code-cleanup:
  submodule-config: fix test binary crashing when no arguments given
  submodule-config: combine early return code into one goto
  submodule-config: passing name reference for .gitmodule blobs
  submodule-config: use explicit empty string instead of strbuf in config_from()
This commit is contained in:
Junio C Hamano 2016-08-03 15:10:28 -07:00
commit f4fa8a9b18
3 changed files with 28 additions and 17 deletions

View File

@ -371,9 +371,9 @@ static int parse_config(const char *var, const char *value, void *data)
}
static int gitmodule_sha1_from_commit(const unsigned char *commit_sha1,
unsigned char *gitmodules_sha1)
unsigned char *gitmodules_sha1,
struct strbuf *rev)
{
struct strbuf rev = STRBUF_INIT;
int ret = 0;
if (is_null_sha1(commit_sha1)) {
@ -381,11 +381,10 @@ static int gitmodule_sha1_from_commit(const unsigned char *commit_sha1,
return 1;
}
strbuf_addf(&rev, "%s:.gitmodules", sha1_to_hex(commit_sha1));
if (get_sha1(rev.buf, gitmodules_sha1) >= 0)
strbuf_addf(rev, "%s:.gitmodules", sha1_to_hex(commit_sha1));
if (get_sha1(rev->buf, gitmodules_sha1) >= 0)
ret = 1;
strbuf_release(&rev);
return ret;
}
@ -399,7 +398,7 @@ static const struct submodule *config_from(struct submodule_cache *cache,
{
struct strbuf rev = STRBUF_INIT;
unsigned long config_size;
char *config;
char *config = NULL;
unsigned char sha1[20];
enum object_type type;
const struct submodule *submodule = NULL;
@ -420,8 +419,8 @@ static const struct submodule *config_from(struct submodule_cache *cache,
return entry->config;
}
if (!gitmodule_sha1_from_commit(commit_sha1, sha1))
return NULL;
if (!gitmodule_sha1_from_commit(commit_sha1, sha1, &rev))
goto out;
switch (lookup_type) {
case lookup_name:
@ -432,16 +431,11 @@ static const struct submodule *config_from(struct submodule_cache *cache,
break;
}
if (submodule)
return submodule;
goto out;
config = read_sha1_file(sha1, &type, &config_size);
if (!config)
return NULL;
if (type != OBJ_BLOB) {
free(config);
return NULL;
}
if (!config || type != OBJ_BLOB)
goto out;
/* fill the submodule config into the cache */
parameter.cache = cache;
@ -450,6 +444,7 @@ static const struct submodule *config_from(struct submodule_cache *cache,
parameter.overwrite = 0;
git_config_from_mem(parse_config, "submodule-blob", rev.buf,
config, config_size, &parameter);
strbuf_release(&rev);
free(config);
switch (lookup_type) {
@ -460,6 +455,11 @@ static const struct submodule *config_from(struct submodule_cache *cache,
default:
return NULL;
}
out:
strbuf_release(&rev);
free(config);
return submodule;
}
static const struct submodule *config_from_path(struct submodule_cache *cache,

View File

@ -23,7 +23,7 @@ int cmd_main(int argc, const char **argv)
arg++;
my_argc--;
while (starts_with(arg[0], "--")) {
while (arg[0] && starts_with(arg[0], "--")) {
if (!strcmp(arg[0], "--url"))
output_url = 1;
if (!strcmp(arg[0], "--name"))

View File

@ -82,6 +82,17 @@ test_expect_success 'error in one submodule config lets continue' '
)
'
test_expect_success 'error message contains blob reference' '
(cd super &&
sha1=$(git rev-parse HEAD) &&
test-submodule-config \
HEAD b \
HEAD submodule \
2>actual_err &&
grep "submodule-blob $sha1:.gitmodules" actual_err >/dev/null
)
'
cat >super/expect_url <<EOF
Submodule url: 'git@somewhere.else.net:a.git' for path 'b'
Submodule url: 'git@somewhere.else.net:submodule.git' for path 'submodule'