Re: [PATCH] Fix handle leak in builtin-pack-objects

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Johannes Sixt
Date: Wednesday, November 19, 2008 - 6:55 am

Alex Riesen schrieb:

Oh, the patch only works around the failure in the test case. In a real
repository there is usually no problem because the destination pack file
does not exist.

The unusual case is where you do this:

 $ git rev-list -10 HEAD | git pack-objects foobar

twice in a row: In this case the second invocation fails on Windows
because the destination pack file already exists *and* is open. But not
even git-repack does this even if it is called twice. OTOH, the test case
*does* exactly this.

-- Hannes

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 19, 4:13 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Johannes Sixt, (Wed Nov 19, 4:54 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 19, 5:13 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Johannes Sixt, (Wed Nov 19, 5:26 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Nicolas Pitre, (Wed Nov 19, 5:45 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Nicolas Pitre, (Wed Nov 19, 5:55 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Johannes Sixt, (Wed Nov 19, 6:06 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 19, 6:30 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 19, 6:34 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Johannes Sixt, (Wed Nov 19, 6:55 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 19, 7:17 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Nicolas Pitre, (Wed Nov 19, 7:31 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Nicolas Pitre, (Wed Nov 19, 7:42 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Johannes Sixt, (Wed Nov 19, 7:52 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 26, 6:18 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Nicolas Pitre, (Wed Nov 26, 7:33 am)
Re: [PATCH] Fix handle leak in builtin-pack-objects, Alex Riesen, (Wed Nov 26, 11:43 am)