Hi Junio, On Mon, 29 Aug 2016, Junio C Hamano wrote: > Johannes Schindelin <johannes.schindelin@xxxxxx> writes: > > > strbuf_addf(&buf, "%s\n", head); > > if (write_in_full(fd, buf.buf, buf.len) < 0) > > - die_errno(_("Could not write to %s"), git_path_head_file()); > > + return error_errno(_("Could not write to %s"), > > + git_path_head_file()); > > Same comment around a left-over lockfile applies to this. An extra > rollback being minimally intrusive also applies here, I think. Sure. I added rollbacks in case of failure. Ciao, Dscho