On Tue, 2014-09-02 at 15:39 -0700, Junio C Hamano wrote:
> We punt from repairing the cache-tree during a branch switching if
> it involves having to create a new tree object that does not yet
> exist in the object store. "mkdir dir && >dir/file && git add dir"
> followed by "git checkout" is one example, when a tree that records
> the state of such "dir/" is not in the object store.
>
> However, after discovering that we do not have a tree object that
> records the state of "dir/", the caller failed to remember the fact
> that it noticed the cache-tree entry it received for "dir/" is
> invalidated, it already knows it should not be populating the level
> callchain to stop the code to attempt populating the level that has
> "dir/" as its immediate subdirectory, and it is not an error at all
> for the sublevel cache-tree entry gave it a bogus object name it
> shouldn't even look at.
>
> This led the caller detect and report a non-existent error. The end
> result was the same and we avoided stuffing a non-existent tree to
> the cache-tree, but we shouldn't have issued an alarming error
> message to the user.
>
> Signed-off-by: Junio C Hamano <gits...@pobox.com>
> ---
>
> * Second try. The level that has intent-to-add entries needs to be
> kept invalidated but the level above it needs to treat as if the
> i-t-a entries do not exist and build the whole tree; a directory
> that does not yet have corresponding tree object while repairing
> the cache-tree needs to invalidate itself *and* propagate the
> (in)validity upwards. They have to be treated differently but
> the first attempt failed to do so.
>
> cache-tree.c | 7 ++++++-
> t/t0090-cache-tree.sh | 8 ++++++++
> 2 files changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/cache-tree.c b/cache-tree.c
> index f951d7d..57597ac 100644
> --- a/cache-tree.c
> +++ b/cache-tree.c
> @@ -316,6 +316,7 @@ static int update_one(struct cache_tree *it,
> int pathlen, entlen;
> const unsigned char *sha1;
> unsigned mode;
> + int expected_missing = 0;
>
> path = ce->name;
> pathlen = ce_namelen(ce);
> @@ -332,8 +333,10 @@ static int update_one(struct cache_tree *it,
> i += sub->count;
> sha1 = sub->cache_tree->sha1;
> mode = S_IFDIR;
> - if (sub->cache_tree->entry_count < 0)
> + if (sub->cache_tree->entry_count < 0) {
> to_invalidate = 1;
> + expected_missing = 1;
> + }
> }
> else {
> sha1 = ce->sha1;
> @@ -343,6 +346,8 @@ static int update_one(struct cache_tree *it,
> }
> if (mode != S_IFGITLINK && !missing_ok && !has_sha1_file(sha1))
> {
> strbuf_release(&buffer);
> + if (expected_missing)
> + return -1;
> return error("invalid object %06o %s for '%.*s'",
> mode, sha1_to_hex(sha1), entlen+baselen, path);
> }
> diff --git a/t/t0090-cache-tree.sh b/t/t0090-cache-tree.sh
> index 48c4240..f9648a8 100755
> --- a/t/t0090-cache-tree.sh
> +++ b/t/t0090-cache-tree.sh
> @@ -210,4 +210,12 @@ test_expect_success 'partial commit gives cache-tree' '
> test_cache_tree
> '
>
> +test_expect_success 'no phantom error when switching trees' '
> + mkdir newdir &&
> + >newdir/one &&
> + git add newdir/one &&
> + git checkout 2>errors &&
> + ! test -s errors
> +'
> +
> test_done
LGTM.
--
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
