Subject: Re: [PATCH 16/33] t3210: test for spurious error messages for dangling packed refs

From: Junio C Hamano <[email-masked]>
Date: Mon, 15 Apr 2013 10:39:08 -0700

Michael Haggerty <[email-masked]> 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 <[email-masked]>
> ---
>
> How can I get rid of the sleeps in these tests?

Would test-chmtime help?