From: Johan Herland <johan@herland.net>
Date: Sat, 31 Oct 2009 11:32:27 GMT
Subject: Re: [PATCH] clone: detect extra arguments
Message-ID: <200910311232.27278.johan@herland.net>
In-Reply-To: <20091030145108.GA881@coredump.intra.peff.net>

On Friday 30 October 2009, Jeff King wrote:
> On Fri, Oct 30, 2009 at 10:45:25AM -0400, Jeff King wrote:
> > But looking at the usage message, there is some potential for cleanup.
> 
> Also, we should probably do this (I did it as a patch on master, though,
> as it is an independent fix):
> 
> -- >8 --
> Subject: [PATCH] clone: fix --recursive usage message
> 
> Looks like a mistaken cut-and-paste in e7fed18a.

Yes. Please fix my screwup.

> Signed-off-by: Jeff King <peff@peff.net>

Acked-by: Johan Herland <johan@herland.net>

> ---
>  builtin-clone.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/builtin-clone.c b/builtin-clone.c
> index 5762a6f..436e8da 100644
> --- a/builtin-clone.c
> +++ b/builtin-clone.c
> @@ -61,7 +61,7 @@ static struct option builtin_clone_options[] = {
>  	OPT_BOOLEAN('s', "shared", &option_shared,
>  		    "setup as shared repository"),
>  	OPT_BOOLEAN(0, "recursive", &option_recursive,
> -		    "setup as shared repository"),
> +		    "initialize submodules in the clone"),
>  	OPT_STRING(0, "template", &option_template, "path",
>  		   "path the template repository"),
>  	OPT_STRING(0, "reference", &option_reference, "repo",
> 

...Johan

-- 
Johan Herland, <johan@herland.net>
www.herland.net