Hi, I am struggling a little with the development process, is a sign-off strictly required for git as it is for kernel development? If so here would be my sign-off: Signed-off-by: Stefan Beller <stefanbeller@xxxxxxxxxxxxxx> This adds a warning and the previous patch adds the documentation. However I agree to Junio, that the meaning should actually depth=0 -> just the tip and no (0) other commits before. depth=n -> the tip and n other commits before. On 01/08/2013 09:07 AM, Stefan Beller wrote: > --- > builtin/clone.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/builtin/clone.c b/builtin/clone.c > index ec2f75b..5e91c1e 100644 > --- a/builtin/clone.c > +++ b/builtin/clone.c > @@ -818,6 +818,9 @@ int cmd_clone(int argc, const char **argv, const char *prefix) > remote = remote_get(option_origin); > transport = transport_get(remote, remote->url[0]); > > + if (option_depth && transport->smart_options->depth < 1) > + die(_("--depth less or equal 0 makes no sense; read manpage.")); > + > if (!is_local) { > if (!transport->get_refs_list || !transport->fetch) > die(_("Don't know how to clone %s"), transport->url); > -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html