Re: [PATCH] clone: ignore --depth when cloning locally (implicitly --local)

Hi,

On Thu, 26 Feb 2009, Junio C Hamano wrote:

Johannes Schindelin [off-list ref] writes:

quoted

When cloning locally, we default to --local, as it makes the whole
operation fast and efficient.

As the most common intent of cloning with a --depth parameter is to
save space, and --local saves space more than --depth ever can,
warn the user and ignore the --depth parameter when cloning locally.

Should --depth be desired, the user can always force proper cloning
by using a file:// URL.

Signed-off-by: Johannes Schindelin <redacted>

Makes sense.  I'm inclined to apply this as a "fix" before 1.6.2, but I
cannot exactly say what we are fixing.  User's expectations?

It seems somebody mentioned on IRC that --depth with --no-local will not 
work as expected, but was not willing to write to the list, or for that 
matter, do anything else.

So given that level of enthusiasm for _supporting_ this feature, I am 
inclined to postpone the patch, or drop it altogether.

I mean, I tried to be a good netizen and not let a patch I started go 
stale, but it is not my itch, and if those whose itch it is do not care, 
maybe I shouldn't, either.

Ciao,
Dscho