Re: Shallow clones with explicit history cutoff?

2014-08-22 Thread Matthias Urlichs
Hi,

Duy Nguyen:
> On Thu, Aug 21, 2014 at 10:39 PM, Matthias Urlichs  
> wrote:
> > What I would like to have, instead, is a version of shallow cloning which
> > cuts off not at a pre-determined depth, but at a given branch (or set of
> > branches). In other words, given
> >
> > +-J--K  (packaged)
> >//
> >   +-F--G--HI(clean)
> >  /   /
> > A---B---C---D---E   (upstream)
> >
> > a command "git clone --shallow-until upstream $REPO" (or however that would
> > be named) would create a shallow git archive which contains branches
> > packaged+clean, with commits FGHIJK. In contrast, with --single-branch and
> > --depth 4 I would get CGHIJK, which isn't what I'd want.
> 
> I would imagine a more generic mechanism "git clone
> --shallow-rev= $REPO" where you could pass anything that "git
> rev-list" can accept (maybe more restricted, and some verification
> required). --shallow-rev could be repeated. So in your case it could
> be "git clone --shallow-rev="^A" $REPO".

Umm, no. ^E (or ^upstream) would do what I want. Hopefully. ;-)

But you're right, that would fit far better into the existing git
paradigms.

> > As I have not spent too much time with the git sources lately (as in "None
> > at all"), some pointers where to start implementing this would be
> > appreciated, assuming (a) this has a reasonable chance of landing in git and
> > (b) nobody beats me to it. ;-)
> 
> I'd like to see this implemented. You are not the first one
> complaining about the (lack of) flexibility of --depth. If you have
> time, I may be able to support (I should not take on another topic
> given my many ongoing/unfinished topics).

Welcome to the club. :-/

Thanks for the pointers. I'll see what I can do (and when).

-- 
-- Matthias Urlichs
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: Shallow clones with explicit history cutoff?

2014-08-22 Thread Duy Nguyen
On Thu, Aug 21, 2014 at 10:39 PM, Matthias Urlichs  wrote:
> What I would like to have, instead, is a version of shallow cloning which
> cuts off not at a pre-determined depth, but at a given branch (or set of
> branches). In other words, given
>
> +-J--K  (packaged)
>//
>   +-F--G--HI(clean)
>  /   /
> A---B---C---D---E   (upstream)
>
> a command "git clone --shallow-until upstream $REPO" (or however that would
> be named) would create a shallow git archive which contains branches
> packaged+clean, with commits FGHIJK. In contrast, with --single-branch and
> --depth 4 I would get CGHIJK, which isn't what I'd want.

I would imagine a more generic mechanism "git clone
--shallow-rev= $REPO" where you could pass anything that "git
rev-list" can accept (maybe more restricted, and some verification
required). --shallow-rev could be repeated. So in your case it could
be "git clone --shallow-rev="^A" $REPO". We could even maybe turn
--depth into a generic thing that is accepted by rev-list so that it
could be easily combined with other rev-list options (--shallow-rev
and --depth are mutually exclusive).

> As I have not spent too much time with the git sources lately (as in "None
> at all"), some pointers where to start implementing this would be
> appreciated, assuming (a) this has a reasonable chance of landing in git and
> (b) nobody beats me to it. ;-)

I'd like to see this implemented. You are not the first one
complaining about the (lack of) flexibility of --depth. If you have
time, I may be able to support (I should not take on another topic
given my many ongoing/unfinished topics). The starting point is
upload-pack.c. And GIT_TRACE env variable will be your friend. Search
for get_shallow_commits(). There the function is supposed to traverse
down from want_obj and set/unset SHALLOW/NOT_SHALLOW flags  properly.

SHALLOW flag should be set right before the cut-out commit (e.g. B and
F if you want to cut A out). NOT_SHALLOW flags could be used to remove
shallow lines in the receiver repo. If you traverse past an existing
shallow point in the client (this is the fetch/pull case, not clone),
then you should set NOT_SHALLOW so the client knows to remove that
point from their $GIT_DIR/shallow. Once you set these properly, the
rest should work.
-- 
Duy
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html