git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Nguyen Thai Ngoc Duy" <pclouds@gmail.com>
To: "James Pickens" <jepicken@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [RFC PATCH 0/9] Narrow/Sparse checkout round 3: "easy mode"
Date: Sat, 30 Aug 2008 16:21:50 +0700	[thread overview]
Message-ID: <fcaeb9bf0808300221q12d1d984mdf78c81363a909d1@mail.gmail.com> (raw)
In-Reply-To: <loom.20080819T210550-149@post.gmane.org>

On 8/20/08, James Pickens <jepicken@gmail.com> wrote:
> Nguyễn Thái Ngọc Duy <pclouds <at> gmail.com> writes:
>
>  > From user POV, we can now checkout a single file or a
>  > subdirectory (checking out subdirectory non-recursively is
>  > possible too). You may start with a narrow clone like:
>
>
> Is there any reason for the change in terminology from "sparse"
>  to "narrow"?  I understand the difference between "partial"
>  and "sparse", but I can't tell if there's any difference
>  between "narrow" and "sparse".  If they are the same thing, then
>  I think "sparse" is the better term.

I have no particular preference. It's up to the community to choose the name.

>  > Last bit. "Narrow rules" for --path|--add-path|--remove-path is
>  > currently wildcards separated by colons. While it does the job,
>  > it does not allow to checkout easily a subdirectory
>  > non-recusively. I was thinking about '*' as "match everything
>  > except slashes" and '**' as "match everything even slashes".
>
>
> I like this idea - it would make this much more intuitive to use,
>  since '*' and '**' would work the same as they do in the
>  shell (for shells that support '**' at least).  I tried the patch
>  in it's current form, and it took me a while to figure out that
>  paths were non-recursive and '*' was matching everything,
>  including slashes.

Tried the last few days but it was not easy, needed to duplicate
fnmatch code. I may come up with a less powerful syntax for
recusive/non-recursive '*'.
-- 
Duy

      reply	other threads:[~2008-08-30  9:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-15 14:24 [RFC PATCH 0/9] Narrow/Sparse checkout round 3: "easy mode" Nguyễn Thái Ngọc Duy
2008-08-16 10:31 ` Junio C Hamano
2008-08-17  5:12   ` Nguyen Thai Ngoc Duy
2008-08-17  5:50     ` Junio C Hamano
2008-08-17  6:10       ` Junio C Hamano
2008-08-17  9:49         ` [RFC PATCH 0/9] Narrow/Sparse checkout round 3: Eric Raible
2008-08-19  9:06           ` Junio C Hamano
2008-08-17 13:36         ` [RFC PATCH 0/9] Narrow/Sparse checkout round 3: "easy mode" Nguyen Thai Ngoc Duy
2008-08-19 21:10 ` James Pickens
2008-08-30  9:21   ` Nguyen Thai Ngoc Duy [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=fcaeb9bf0808300221q12d1d984mdf78c81363a909d1@mail.gmail.com \
    --to=pclouds@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jepicken@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).