From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 16 Jun 2015 21:33:52 +0000 (-0700)
Subject: Merge branch 'jk/clone-dissociate' into maint
X-Git-Tag: v2.4.4~1
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/37d6f933dff7d5c295870c85399f5fd22072643a?ds=inline;hp=-c

Merge branch 'jk/clone-dissociate' into maint

Code clean-up.

* jk/clone-dissociate:
clone: reorder --dissociate and --reference options
clone: use OPT_STRING_LIST for --reference
---

37d6f933dff7d5c295870c85399f5fd22072643a
diff --combined builtin/clone.c
index 13030ee1d1,0006763074..4646922d6e
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@@ -34,7 -34,7 +34,7 @@@
   *
   */
  static const char * const builtin_clone_usage[] = {
 -	N_("git clone [options] [--] <repo> [<dir>]"),
 +	N_("git clone [<options>] [--] <repo> [<dir>]"),
  	NULL
  };
  
@@@ -51,15 -51,6 +51,6 @@@ static struct string_list option_config
  static struct string_list option_reference;
  static int option_dissociate;
  
- static int opt_parse_reference(const struct option *opt, const char *arg, int unset)
- {
- 	struct string_list *option_reference = opt->value;
- 	if (!arg)
- 		return -1;
- 	string_list_append(option_reference, arg);
- 	return 0;
- }
- 
  static struct option builtin_clone_options[] = {
  	OPT__VERBOSITY(&option_verbosity),
  	OPT_BOOL(0, "progress", &option_progress,
@@@ -83,8 -74,10 +74,10 @@@
  		    N_("initialize submodules in the clone")),
  	OPT_STRING(0, "template", &option_template, N_("template-directory"),
  		   N_("directory from which templates will be used")),
- 	OPT_CALLBACK(0 , "reference", &option_reference, N_("repo"),
- 		     N_("reference repository"), &opt_parse_reference),
+ 	OPT_STRING_LIST(0, "reference", &option_reference, N_("repo"),
+ 			N_("reference repository")),
+ 	OPT_BOOL(0, "dissociate", &option_dissociate,
+ 		 N_("use --reference only while cloning")),
  	OPT_STRING('o', "origin", &option_origin, N_("name"),
  		   N_("use <name> instead of 'origin' to track upstream")),
  	OPT_STRING('b', "branch", &option_branch, N_("branch"),
@@@ -95,8 -88,6 +88,6 @@@
  		    N_("create a shallow clone of that depth")),
  	OPT_BOOL(0, "single-branch", &option_single_branch,
  		    N_("clone only one branch, HEAD or --branch")),
- 	OPT_BOOL(0, "dissociate", &option_dissociate,
- 		 N_("use --reference only while cloning")),
  	OPT_STRING(0, "separate-git-dir", &real_git_dir, N_("gitdir"),
  		   N_("separate git dir from working tree")),
  	OPT_STRING_LIST('c', "config", &option_config, N_("key=value"),
@@@ -906,8 -897,6 +897,8 @@@ int cmd_clone(int argc, const char **ar
  
  	remote = remote_get(option_origin);
  	transport = transport_get(remote, remote->url[0]);
 +	transport_set_verbosity(transport, option_verbosity, option_progress);
 +
  	path = get_repo_path(remote->url[0], &is_bundle);
  	is_local = option_local != 0 && path && !is_bundle;
  	if (is_local) {
@@@ -934,6 -923,8 +925,6 @@@
  	if (option_single_branch)
  		transport_set_option(transport, TRANS_OPT_FOLLOWTAGS, "1");
  
 -	transport_set_verbosity(transport, option_verbosity, option_progress);
 -
  	if (option_upload_pack)
  		transport_set_option(transport, TRANS_OPT_UPLOADPACK,
  				     option_upload_pack);