Re: [PATCH v2 00/14] Sparse checkout

[Jakub Narebski <jnareb@gmail.com>]

On Sun, 21 Sep 2008, Nguyen Thai Ngoc Duy wrote:
> On 9/21/08, Junio C Hamano <gitster@pobox.com> wrote:

> >  How would that --narrow-match that is not stored anywhere on the
> >  filesystem but used only for filtering the output be any more useful than
> >  a grep that filters ls-files output in practice?
> 
> Well, it works exactly like 'grep' internally.
> 
> >  I would imagine it would be much more useful if .git/info/attributes can
> >  specify "checkout" attribute that is defined like this:
> >
> >         `checkout`
> >         ^^^^^^^^^^
[...]

> >  Then whenever a new path enters the index, you _could_ check with the
> >  attribute mechanism to set the CE_NOCHECKOUT flag.  Just like an already
> >  tracked path is not ignored even if it matches .gitignore pattern, a path
> >  without CE_NOCHECKOUT that is in the index is checked out even if it has
> >  checkout attribute Unset.
> >
> >  Hmm?
> 
> Well I think people would want to save no-checkout rules eventually.
> But I don't know how they want to use it. Will the saved rules be hard
> restriction, that no files can be checked out outside defined areas?
> Will it be to save a couple of keystrokes,   that is, instead of
> typing "--reset-sparse=blah" all the time, now just "--reset-sparse"
> and default rules will be applied? Your suggestion would be the third,
> applying on new files only.
> 
> Anyway I will try to extend attr.c a bit to take input from command
> line, then move "sparse patterns" over to use attr.c.

First, I think that this was Junio asking for discussion more than
for changing the design.

Second, while unifying the "check the match" part of gitignore,
gitattribute and sparse checkout would be IMVHO a good idea, I'm
not sure if trying to use/reuse attr.c literally would be a good
idea, at least not without larger surgery.  AFAIK, IIUC gitattributes
have some limitations, one of which that they are read from working
area (and there is no API for reading from tree); although this could
be enough for `checkout' attribute, which is not that different in
work from `smudge' attribute, or `crlf` attribute.

-- 
Jakub Narebski
Poland