Re: [PATCH] Make repack less likely to corrupt repository
From: Robin Rosenberg <hidden>
Date: 2016-06-15 22:46:14
Subsystem:
documentation, the rest · Maintainers:
Jonathan Corbet, Linus Torvalds
Possibly related (same subject, not in this thread)
- 2016-06-15 · Re: [PATCH] Make repack less likely to corrupt repository · Junio C Hamano <hidden>
- 2016-06-15 · Re: [PATCH] Make repack less likely to corrupt repository · Junio C Hamano <hidden>
- 2016-06-15 · Re: [PATCH] Make repack less likely to corrupt repository · Robin Rosenberg <hidden>
- 2016-06-15 · Re: [PATCH] Make repack less likely to corrupt repository · Johannes Schindelin <hidden>
- 2016-06-15 · Re: [PATCH] Make repack less likely to corrupt repository · Robin Rosenberg <hidden>
Here is a new version on my ammendments to Junios patch. -- robin
From 67347a63ce5ba324a750eb2c1ed7b9b0260d966a Mon Sep 17 00:00:00 2001
From: Robin Rosenberg <redacted> Date: Thu, 19 Feb 2009 23:18:59 +0100 Subject: [PATCH] Recover from some known repack failure situations For the know problems that we can recover from we exit with code 2 instead of 1. Also removed the old packs when repack succeeds Signed-off-by: Robin Rosenberg <redacted> --- Documentation/git-repack.txt | 6 ++++++ git-repack.sh | 18 +++++++++++++++++- 2 files changed, 23 insertions(+), 1 deletions(-)
diff --git a/Documentation/git-repack.txt b/Documentation/git-repack.txt
index aaa8852..e5ecd66 100644
--- a/Documentation/git-repack.txt
+++ b/Documentation/git-repack.txt@@ -24,6 +24,12 @@ associated index file. Packs are used to reduce the load on mirror systems, backup engines, disk storage, etc. +On Windows repacking may fail because packs that should be replaced +are locked by other programs. In this case the program will recover +from the situation with an exit code of 2. The user does not have +to take any action to recover. For repacking to succeed the user +mustc stop the offending program or wait for it to close the packs. + OPTIONS -------
diff --git a/git-repack.sh b/git-repack.sh
index 0f13043..519c83a 100755
--- a/git-repack.sh
+++ b/git-repack.sh@@ -136,8 +136,15 @@ then do echo >&2 "WARNING: old-$file -> $file" done + exit 1 fi - exit 1 + echo >&2 "INFO: We recovered from the repack error, but your repo" + echo >&2 "INFO: is probably suboptimally packed. You may try to repack" + echo >&2 "INFO: later. A common reason for repacking failure is that" + echo >&2 "INFO: a Windows program was locking one of the old pack files." + echo >&2 "INFO: To repack successfully you may have to close that program" + echo >&2 "INFO: before repacking." + exit 2 fi # Now the ones with the same name are out of the way...
@@ -152,6 +159,15 @@ do exit done +# Remove the "old-" files +for name in $names +do + rm -f "$PACKDIR/old-pack-$name.idx" + rm -f "$PACKDIR/old-pack-$name.pack" +done + +# End of pack replacement. + if test "$remove_redundant" = t then # We know $existing are all redundant.
--
1.6.1.285.g35d8b