From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 12 Aug 2016 16:47:36 +0000 (-0700)
Subject: Merge branch 'rs/child-process-init'
X-Git-Tag: v2.10.0-rc0~13
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/18f3ce8841b1cf7a3bee8e8da75fd7520060a4c0?ds=inline;hp=-c

Merge branch 'rs/child-process-init'

A small code clean-up.

* rs/child-process-init:
use CHILD_PROCESS_INIT to initialize automatic variables
---

18f3ce8841b1cf7a3bee8e8da75fd7520060a4c0
diff --combined builtin/submodule--helper.c
index 343d25b758,e3bd2461f4..e79790f0bd
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@@ -444,8 -444,7 +444,7 @@@ static int module_name(int argc, const 
  static int clone_submodule(const char *path, const char *gitdir, const char *url,
  			   const char *depth, const char *reference, int quiet)
  {
- 	struct child_process cp;
- 	child_process_init(&cp);
+ 	struct child_process cp = CHILD_PROCESS_INIT;
  
  	argv_array_push(&cp.args, "clone");
  	argv_array_push(&cp.args, "--no-checkout");
@@@ -748,12 -747,8 +747,12 @@@ static int update_clone_get_next_task(s
  	if (index < suc->failed_clones_nr) {
  		int *p;
  		ce = suc->failed_clones[index];
 -		if (!prepare_to_clone_next_submodule(ce, child, suc, err))
 -			die("BUG: ce was a submodule before?");
 +		if (!prepare_to_clone_next_submodule(ce, child, suc, err)) {
 +			suc->current ++;
 +			strbuf_addf(err, "BUG: submodule considered for cloning,"
 +				    "doesn't need cloning any more?\n");
 +			return 0;
 +		}
  		p = xmalloc(sizeof(*p));
  		*p = suc->current;
  		*idx_task_cb = p;
@@@ -896,64 -891,13 +895,64 @@@ static int resolve_relative_path(int ar
  {
  	struct strbuf sb = STRBUF_INIT;
  	if (argc != 3)
 -		die("submodule--helper relative_path takes exactly 2 arguments, got %d", argc);
 +		die("submodule--helper relative-path takes exactly 2 arguments, got %d", argc);
  
  	printf("%s", relative_path(argv[1], argv[2], &sb));
  	strbuf_release(&sb);
  	return 0;
  }
  
 +static const char *remote_submodule_branch(const char *path)
 +{
 +	const struct submodule *sub;
 +	gitmodules_config();
 +	git_config(submodule_config, NULL);
 +
 +	sub = submodule_from_path(null_sha1, path);
 +	if (!sub)
 +		return NULL;
 +
 +	if (!sub->branch)
 +		return "master";
 +
 +	if (!strcmp(sub->branch, ".")) {
 +		unsigned char sha1[20];
 +		const char *refname = resolve_ref_unsafe("HEAD", 0, sha1, NULL);
 +
 +		if (!refname)
 +			die(_("No such ref: %s"), "HEAD");
 +
 +		/* detached HEAD */
 +		if (!strcmp(refname, "HEAD"))
 +			die(_("Submodule (%s) branch configured to inherit "
 +			      "branch from superproject, but the superproject "
 +			      "is not on any branch"), sub->name);
 +
 +		if (!skip_prefix(refname, "refs/heads/", &refname))
 +			die(_("Expecting a full ref name, got %s"), refname);
 +		return refname;
 +	}
 +
 +	return sub->branch;
 +}
 +
 +static int resolve_remote_submodule_branch(int argc, const char **argv,
 +		const char *prefix)
 +{
 +	const char *ret;
 +	struct strbuf sb = STRBUF_INIT;
 +	if (argc != 2)
 +		die("submodule--helper remote-branch takes exactly one arguments, got %d", argc);
 +
 +	ret = remote_submodule_branch(argv[1]);
 +	if (!ret)
 +		die("submodule %s doesn't exist", argv[1]);
 +
 +	printf("%s", ret);
 +	strbuf_release(&sb);
 +	return 0;
 +}
 +
  struct cmd_struct {
  	const char *cmd;
  	int (*fn)(int, const char **, const char *);
@@@ -967,8 -911,7 +966,8 @@@ static struct cmd_struct commands[] = 
  	{"relative-path", resolve_relative_path},
  	{"resolve-relative-url", resolve_relative_url},
  	{"resolve-relative-url-test", resolve_relative_url_test},
 -	{"init", module_init}
 +	{"init", module_init},
 +	{"remote-branch", resolve_remote_submodule_branch}
  };
  
  int cmd_submodule__helper(int argc, const char **argv, const char *prefix)