From: Jonathan Nieder <jrnieder@gmail.com>
Date: Sat, 10 Jul 2010 21:40:50 GMT
Subject: Re: Question about 'branch -d' safety
Message-ID: <20100710214050.GA3680@burratino>
In-Reply-To: <20100710065549.GA24296@localhost>

Clemens Buchacher wrote:

>          Maybe the real solution is a reflog for deleted branch
> heads, rather than being too careful about whether or not a branch
> can be deleted.

Yes, I think so.  Would there be any bad side effects to this?

diff --git a/refs.c b/refs.c
index b540067..7ed0154 100644
--- a/refs.c
+++ b/refs.c
@@ -1084,7 +1084,6 @@ int delete_ref(const char *refname, const unsigned char *sha1, int delopt)
 	 */
 	ret |= repack_without_ref(refname);
 
-	unlink_or_warn(git_path("logs/%s", lock->ref_name));
 	invalidate_cached_refs();
 	unlock_ref(lock);
 	return ret;
--