Re: [PATCH] Introduce light weight commit annotations
- From
Alex Riesen <raa.lkml@gmail.com>
- Date
- Jun 11, 2007, 08:05 UTC
- Message-ID
- <81b0412b0706110105t369d908v70a140505caee778@mail.gmail.com>
- In-Reply-To
- <7vbqfnrl54.fsf@assigned-by-dhcp.pobox.com>
On 6/11/07, Junio C Hamano <gitster@pobox.com> wrote:
Show 13 quoted lines
> "Alex Riesen" <raa.lkml@gmail.com> writes:
>
> > It is still a reference, really. Besides, if it is not under refs/, we'd
> > have to change fetch/push to allow distribution of the notes/annotations
> > (there are special assumptions regarding reference names starting
> > with "refs/"). Right now it just works.
>
> Two issues.
>
> One is that it is unclear is how the reachability rules should
> be. Should an ??/?{38} entry in the refs/annotations/commits
> protects the commit the entry talks about (i.e. ???{38}) from
> getting pruned?I think it should be the other way around: annotations get loose and reported as such (and maybe even removed) by git-fsck.