Subject: Re: [RFC PATCH] clone: Deprecate the --recursive option in favor of --recurse-submodules

From: Jakub Narebski <[email-masked]>
Date: Wed, 03 Nov 2010 15:13:47 -0700 (PDT)

Jens Lehmann <[email-masked]> writes:

> @@ -65,6 +65,8 @@ static struct option builtin_clone_options[] = {
>  	OPT_BOOLEAN('s', "shared", &option_shared,
>  		    "setup as shared repository"),
>  	OPT_BOOLEAN(0, "recursive", &option_recursive,
> +		    "initialize submodules in the clone (deprecated)"),
> +	OPT_BOOLEAN(0, "recurse_submodules", &option_recurse_submodules,
>  		    "initialize submodules in the clone"),
>  	OPT_STRING(0, "template", &option_template, "path",
>  		   "path the template repository"),

Shouldn't we use PARSE_OPT_HIDDEN?  Or should it be left for later?

-- 
Jakub Narebski
Poland
ShadeHawk on #git