From: Mark Levedahl Date: Mon, 13 Aug 2007 13:22:24 GMT Subject: Re: [PATCH] gitk - dont warn when deleting synonym for current head Message-ID: <30e4a070708130622l7558230cpb870378aeba537fb@mail.gmail.com> In-Reply-To: <30e4a070708130604g5b848b2fm4b42e145384c0ba4@mail.gmail.com> On 8/13/07, Mark Levedahl wrote: > On 8/12/07, Paul Mackerras wrote: > > It does load the full repository DAG - that's how it gets the > > information for the Precedes:, Follows: and Branch(es): fields in the > > commit display. It's true that that can be turned off, though, and in > > that case an "are you sure" dialog would be appropriate. > > > > Paul. > Double groan - I hit the wrong key (send rather than abort) with a partially written incorrect message. Drat. I was fooled as the "Precedes:" entry is blank if I do, for instance: git checkout -b foo master~1 gitk even though HEAD=foo which precedes master. However, master shows up in the list of branches containing the commit. So yes, I see that gitk has the full DAG available and could therefore generate a useful warning when deleting a branch would leave dangling commits. Mark