From mboxrd@z Thu Jan 1 00:00:00 1970 From: richard.maw at gmail.com (Richard Maw) Date: Mon, 1 Aug 2016 22:35:49 +0100 Subject: [PATCH v3 17/21] Display notes from namespace In-Reply-To: <20160801213553.16807-1-richard.maw@gmail.com> References: <20160801213553.16807-1-richard.maw@gmail.com> Message-ID: <20160801213553.16807-18-richard.maw@gmail.com> We must not leave it at its default behaviour, as that results in notes from the root namespace being used and ignoring notes from the current namespace, since it is not namespace aware. We can handle this by instructing it to not load the default refs, and providing a set of extra refs to use. The provided extra refs are globs rather than ref names, but ref names may not contain glob characters so this is safe. We get an annoying warning if the provided ref does not exist, so we check whether the ref exists before attempting to provide it. Signed-off-by: Richard Maw --- cgit.c | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/cgit.c b/cgit.c index d664ab4..321a00f 100644 --- a/cgit.c +++ b/cgit.c @@ -586,7 +586,26 @@ static int prepare_repo_cmd(void) * load local configuration from the git repository, so we do them both while * the HOME variables are unset. */ setup_git_directory_gently(&nongit); - init_display_notes(NULL); + if (get_git_namespace()) { + struct display_notes_opt opt = { + .use_default_notes = 0, + .extra_notes_refs = STRING_LIST_INIT_NODUP, + }; + struct strbuf namespaced_note_ref = STRBUF_INIT; + + strbuf_add(&namespaced_note_ref, get_git_namespace(), + strlen(get_git_namespace())); + strbuf_add(&namespaced_note_ref, "refs/notes/commits", + strlen("refs/notes/commits")); + + if (ref_exists(namespaced_note_ref.buf)) + string_list_append(&opt.extra_notes_refs, namespaced_note_ref.buf); + + init_display_notes(&opt); + string_list_clear(&opt.extra_notes_refs, 1); + strbuf_release(&namespaced_note_ref); + } else + init_display_notes(NULL); if (nongit) { const char *name = ctx.repo->name; -- 2.9.0