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

[PATCH 2/6] fixup! Add reftable library

From
Johannes Schindelin via GitGitGadget <gitgitgadget@gmail.com>
Date
May 4, 2020, 13:31 UTC
Message-ID
<ff60fde10919b6b8c54ecb8f38b710fac37624e3.1588599086.git.gitgitgadget@gmail.com>
In-Reply-To
<pull.623.git.1588599086.gitgitgadget@gmail.com>
From: Johannes Schindelin <johannes.schindelin@gmx.de>

Do close the file descriptors to obsolete files before trying to delete them. This is actually required on Windows.

Note: this patch is probably a bit heavy-handed, releasing more than
necessary (which will then have to be re-read). But it seems to be
enough to let t0031 pass on Windows, so it is at least a clue as to what
the fix will look like eventually.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 reftable/stack.c | 4 ++++
 1 file changed, 4 insertions(+)
diff --git a/reftable/stack.c b/reftable/stack.c
index 2e32f7671c2..3c5f4a43130 100644
--- a/reftable/stack.c
+++ b/reftable/stack.c
@@ -941,6 +941,10 @@ static int stack_compact_range(struct reftable_stack *st, int first, int last,
 	}
 	have_lock = false;
 
+	reftable_merged_table_close(st->merged);
+	merged_table_clear(st->merged);
+	reftable_merged_table_free(st->merged);
+	st->merged = NULL;
 	{
 		char **p = delete_on_success;
 		while (*p) {
-- 
gitgitgadget
Previous: Johannes Schindelin via GitGitGadgetNext: Johannes Schindelin via GitGitGadget
Message 3 of 12 in “Minimal patch series to fix the CI runs of hn/reftable”
  1. 0/6 Minimal patch series to fix the CI runs of hn/reftableJohannes Schindelin via GitGitGadget, May 4, 2020
  2. 1/6 fixup! Add reftable libraryJohannes Schindelin via GitGitGadget, May 4, 2020
  3. 2/6 fixup! Add reftable libraryJohannes Schindelin via GitGitGadget, May 4, 2020
  4. 4/6 fixup! Add reftable libraryJohannes Schindelin via GitGitGadget, May 4, 2020
  5. 3/6 fixup! Add reftable libraryJohannes Schindelin via GitGitGadget, May 4, 2020
  6. 6/6 vcxproj: adjust for the reftable changesJohannes Schindelin via GitGitGadget, May 4, 2020
  7. Junio C HamanoMay 4, 2020
  8. 5/6 fixup! Add reftable libraryJohannes Schindelin via GitGitGadget, May 4, 2020
  9. Junio C HamanoMay 4, 2020
  10. Johannes SchindelinMay 5, 2020
  11. Han-Wen NienhuysMay 4, 2020
  12. Johannes SchindelinMay 5, 2020

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.