Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:
> My preliminary guess is that this code in pack-write.c needs to use the
> lock file paradigm:
>
> if (!index_name) {
> [...]
> } else {
> unlink(index_name);
> fd = open(index_name, O_CREAT|O_EXCL|O_WRONLY, 0600);
> }
Whoa. That particular code has been (and is still) correct.
When repacking we should pack into a temporary pack and idx file and then
replace the real ones after both new pack and its idx are successfully
written, and I thought that is how we've been doing this all the time.
Maybe the caller has been broken at some point? Sigh...
--
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
