From: Jonathan Nieder <jrnieder@gmail.com>
Date: Tue, 16 Nov 2010 03:07:28 GMT
Subject: Re: [PATCH 10/10] clean: support cleaning sparse checkout with -S
Message-ID: <20101116030728.GB29358@burratino>
In-Reply-To: <AANLkTimrm2ArRn9Ym271iFEv2bstPPD0WPuP0=Majf0K@mail.gmail.com>

Nguyen Thai Ngoc Duy wrote:
> 2010/11/16 Jonathan Nieder <jrnieder@gmail.com>:

>> Hmm, that's new.  Seems fine, though; a person using -S would be
>> forwarned.
>
> It's safe, which is why I let it do that without '-f'. One can always
> checkout those entries again (provided that they know what entries to
> checkout).
>
> How do you want them to be warned?

Manual and -h output, ideally.

>		OPT_BOOLEAN('S', NULL, &sparse, "remove tracked files outside sparse checkout"),

Maybe something to the effect of

		OPT_SET_INT('S', NULL, &sparse,
			"prune worktree to sparse pattern (even without -f)", 1),

Probably someone will chime in with the perfect wording that fits
in the 80-column "git clean -h" output format. :)