On Wed, 19 Nov 2008, Johannes Sixt wrote:
> Alex Riesen schrieb:
> > The opened packs seem to stay open forever.
>
> In my MinGW port I have the patch below that avoids that t5303 fails
> because of a pack file that remains open. (Open files cannot be replaced
> on Windows.) I had hoped that your patch would help, but it does not.
> Something else still keeps the pack file open. Can anything be done about
> that?
>
> -- Hannes
>
> From: Johannes Sixt <j6t@xxxxxxxx>
> Date: Mon, 17 Nov 2008 09:25:19 +0100
> Subject: [PATCH] t5303: Do not overwrite an existing pack
>
> This test corrupts a pack file, then repacks the objects. The consequence
> is that the repacked pack file has the same name as the original file
> (that has been corrupted).
>
> During its operation, git-pack-objects opens the corrupted file and keeps
> it open at all times. On Windows, this is a problem because a file that is
> open in any process cannot be delete or replaced, but that is what we do
> in some of the test cases, and so they fail.
>
> The work-around is to write the repacked objects to a file of a different
> name, and replace the original after git-pack-objects has terminated.
>
> Signed-off-by: Johannes Sixt <j6t@xxxxxxxx>
Acked-by: Nicolas Pitre <nico@xxxxxxx>
> ---
> t/t5303-pack-corruption-resilience.sh | 7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/t/t5303-pack-corruption-resilience.sh
> b/t/t5303-pack-corruption-resilience.sh
> index 5132d41..41c83e3 100755
> --- a/t/t5303-pack-corruption-resilience.sh
> +++ b/t/t5303-pack-corruption-resilience.sh
> @@ -43,8 +43,11 @@ create_new_pack() {
>
> do_repack() {
> pack=`printf "$blob_1\n$blob_2\n$blob_3\n" |
> - git pack-objects $@ .git/objects/pack/pack` &&
> - pack=".git/objects/pack/pack-${pack}"
> + git pack-objects $@ .git/objects/pack/packtmp` &&
> + packtmp=".git/objects/pack/packtmp-${pack}" &&
> + pack=".git/objects/pack/pack-${pack}" &&
> + mv "${packtmp}.pack" "${pack}.pack" &&
> + mv "${packtmp}.idx" "${pack}.idx"
> }
>
> do_corrupt_object() {
> --
> 1.6.0.4.1683.g35125
>
>
Nicolas
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
