Re: [PATCH v3 0/2] Correctly handle transient files in shared repositories

From: Jeff King <peff@peff.net>
Date: Jan 11, 2016, 20:22 UTC
Message-ID: <20160111202243.GA14289@sigill.intra.peff.net>
In-Reply-To: <cover.1452537321.git.johannes.schindelin@gmx.de>

On Mon, Jan 11, 2016 at 07:35:42PM +0100, Johannes Schindelin wrote:

Show 10 quoted lines

> Transient files, e.g. commit messages, are writable only by the owner,
> even in shared repositories, to avoid interference between competing
> users working on the same files.
> 
> These files are typically not deleted after use. As a consequence, we
> have to delete such files before writing when they are owned by someone
> else than the current user.
> 
> The only change relative to v2 is that the second commit message
> clarifies why apply, fsck and fast-import are left unchanged.

This looks fine to me. Even if if there are sites that could also use conversion, this is a strict improvement, and we can revisit the other ones in the future.

Thanks.

-Peff