git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/5] support fetching into a shallow repository

From
Junio C Hamano <junkio@cox.net>
Date
Nov 14, 2006, 07:24 UTC
Message-ID
<7vac2u1oee.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<Pine.LNX.4.63.0610302008520.26682@wbgn013.biozentrum.uni-wuerzburg.de>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 58 quoted lines
> A shallow commit is a commit which has parents, which in turn are
> "grafted away", i.e. the commit appears as if it were a root.
>
> Since these shallow commits should not be edited by the user, but
> only by core git, they are recorded in the file $GIT_DIR/shallow.
>
> A repository containing shallow commits is called shallow.
>
> The advantage of a shallow repository is that even if the upstream
> contains lots of history, your local (shallow) repository needs not
> occupy much disk space.
>
> The disadvantage is that you might miss a merge base when pulling
> some remote branch.
>
> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
> diff --git a/shallow.c b/shallow.c
> new file mode 100644
> index 0000000..3cf2127
> --- /dev/null
> +++ b/shallow.c
>...
> +struct commit_list *get_shallow_commits(struct object_array *heads, int depth)
> +{
> +	int i = 0, cur_depth = 0;
> +	struct commit_list *result = NULL;
> +	struct object_array stack = {0, 0, NULL};
> +	struct commit *commit = NULL;
> +
> +	while (commit || i < heads->nr || stack.nr) {
> +		struct commit_list *p;
> +		if (!commit) {
>...
> +		}
> +		parse_commit(commit);
> +		cur_depth++;
> +		for (p = commit->parents, commit = NULL; p; p = p->next) {
> +			if (!p->item->util) {
>...
> +			} else {
>...
> +			}
> +			if (cur_depth < depth) {
> +				if (p->next)
> +					add_object_array(&p->item->object,
> +							NULL, &stack);
> +				else {
> +					commit = p->item;
> +					cur_depth = *(int *)commit->util;
> +				}
> +			} else
> +				commit_list_insert(p->item, &result);
> +		}
> +	}
> +
> +	return result;
> +}

I think the "commit = p->item" part is trying to do a tail recursion optimization, but this is a bit too clever to my liking (at first I mistook that the code forgot to re-point p at its parents list and incrementing cur_depth).

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 2 of 4 in “support fetching into a shallow repository”
  1. 2/5 support fetching into a shallow repositoryJohannes Schindelin, Oct 30, 2006
  2. Junio C HamanoNov 14, 2006
  3. Johannes SchindelinNov 14, 2006
  4. Junio C HamanoNov 14, 2006

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.