git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[Outreachy PATCH] notes-utils: use the passed repository for notes refs

From
OAOkhuomon Ajayi <okhuomonajayi54@gmail.com>
Date
Oct 31, 2026, 10:45 UTC
Message-ID
<20261031104526.46711-1-okhuomonajayi54@gmail.com>

create_notes_commit() and commit_notes() receive a repository, but read and update the notes ref in the_repository.

Use the repository they are given.

Keep USE_THE_REPOSITORY_VARIABLE because init_copy_notes_for_rewrite() still uses the_repository.

Signed-off-by: Okhuomon Ajayi <okhuomonajayi54@gmail.com>
---
 notes-utils.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/notes-utils.c b/notes-utils.c
index 5c1c75d5b8..d4606bc576 100644
--- a/notes-utils.c
+++ b/notes-utils.c
@@ -26,7 +26,7 @@ void create_notes_commit(struct repository *r,
 	if (!parents) {
 		/* Deduce parent commit from t->ref */
 		struct object_id parent_oid;
-		if (!refs_read_ref(get_main_ref_store(the_repository), t->ref, &parent_oid)) {
+		if (!refs_read_ref(get_main_ref_store(r), t->ref, &parent_oid)) {
 			struct commit *parent = lookup_commit(r, &parent_oid);
 			if (repo_parse_commit(r, parent))
 				die("Failed to find/parse commit %s", t->ref);
@@ -61,7 +61,7 @@ void commit_notes(struct repository *r, struct notes_tree *t, const char *msg)
 
 	create_notes_commit(r, t, NULL, buf.buf, buf.len, &commit_oid);
 	strbuf_insertstr(&buf, 0, "notes: ");
-	refs_update_ref(get_main_ref_store(the_repository), buf.buf,
+	refs_update_ref(get_main_ref_store(r), buf.buf,
 			t->update_ref, &commit_oid, NULL, 0,
 			UPDATE_REFS_DIE_ON_ERR);
 

base-commit: 50fe5cc63f9cd6e89b87a2c9ccf96a444f4cfdb7
-- 
2.56.0
Next: Patrick Steinhardt
Message 1 of 9 in “notes-utils: use the passed repository for notes refs”
  1. notes-utils: use the passed repository for notes refsOkhuomon Ajayi, Oct 31, 2026
  2. Patrick SteinhardtNov 2, 2026
  3. Okhuomon AjayiNov 2, 2026
  4. 0/2 notes-utils: stop using the_repositoryOkhuomon Ajayi, Nov 3, 2026
  5. 1/2 notes-utils: use the passed repository for notes refsOkhuomon Ajayi, Nov 3, 2026
  6. Karthik NayakNov 3, 2026
  7. 2/2 notes-utils: pass the repository to init_copy_notes_for_rewrite()Okhuomon Ajayi, Nov 3, 2026
  8. Karthik NayakNov 3, 2026
  9. Patrick SteinhardtNov 3, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.