From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 15 Apr 2013 17:39:08 GMT
Subject: Re: [PATCH 16/33] t3210: test for spurious error messages for dangling packed refs
Message-ID: <7vobdfvglf.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1365944088-10588-17-git-send-email-mhagger@alum.mit.edu>

Michael Haggerty <mhagger@alum.mit.edu> writes:

> A packed reference can be overridden by a loose reference, in which
> case the packed reference is obsolete and is never used.  The object
> pointed to by such a reference can be garbage collected.  Since
> d66da478f2, this could lead to the emission of a spurious error
> message:
>
>     error: refs/heads/master does not point to a valid object!
>
> The error is generated by repack_without_ref() if there is an obsolete
> dangling packed reference in packed-refs when the packed-refs file has
> to be rewritten due to the deletion of another packed reference.  Add
> a failing test demonstrating this problem and some passing tests of
> related scenarios.

That is one nasty recent bug.

>
> Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
> ---
>
> How can I get rid of the sleeps in these tests?

Would test-chmtime help?