From: Michael J Gruber Date: Mon, 29 Mar 2010 13:05:58 GMT Subject: [PATCH 2/2] refs.c: Write reflogs for notes just like for branch heads Message-ID: <0f06a30f395c14307066d0e4365fc61a294997db.1269867675.git.git@drmicha.warpmail.net> In-Reply-To: The notes code intends to write reflog entries, but currently they are not written because log_ref_write() checks for the refname path explicitly. Add refs/notes to the list of allowed paths so that notes references are treated just like branch heads, i.e. according to core.logAllRefUpdates and core.bare. Signed-off-by: Michael J Gruber --- This is actually inspired by Jeff's novel notes use. I think there are use cases where a notes log makes sense (notes on commits) and those where it does not (metadata/textconv). In both cases having a reflog is useful. So, the next step is really to allow notes trees without history, which also takes care of the pruning issue. I know how to do this, I just have to decide about the configuration options. refs.c | 1 + t/t3301-notes.sh | 2 +- 2 files changed, 2 insertions(+), 1 deletions(-) diff --git a/refs.c b/refs.c index 0f24c8d..d3db15a 100644 --- a/refs.c +++ b/refs.c @@ -1276,6 +1276,7 @@ static int log_ref_write(const char *ref_name, const unsigned char *old_sha1, if (log_all_ref_updates && (!prefixcmp(ref_name, "refs/heads/") || !prefixcmp(ref_name, "refs/remotes/") || + !prefixcmp(ref_name, "refs/notes/") || !strcmp(ref_name, "HEAD"))) { if (safe_create_leading_directories(log_file) < 0) return error("unable to create directory for %s", diff --git a/t/t3301-notes.sh b/t/t3301-notes.sh index 5410a6d..b2e7b07 100755 --- a/t/t3301-notes.sh +++ b/t/t3301-notes.sh @@ -69,7 +69,7 @@ cat >expect <output && test_cmp expect output ' -- 1.7.0.3.448.g82eeb