Thread (11 messages) flat view 11 messages, 5 authors, 2016-12-08

Re: [PATCH 3/3] lockfile: LOCK_REPORT_ON_ERROR

From: Robbie Iannucci <hidden>
Date: 2016-12-08 18:10:20

Possibly related (same subject, not in this thread)

Thanks all for taking a look at this, I didn't expect such a quick response :).

No hard feelings re: breakage; I know how gnarly these sorts of
refactors can be (and more libification is always better :)).

Robbie

On Thu, Dec 8, 2016 at 3:53 AM, Johannes Schindelin
[off-list ref] wrote:
Hi Junio,

On Wed, 7 Dec 2016, Junio C Hamano wrote:
quoted
The "libify sequencer" topic stopped passing the die_on_error option
to hold_locked_index(), and this lost an error message from "git
merge --ff-only $commit" when there are competing updates in
progress.
Sorry for the breakage.

When libifying the code, I tried to be careful to retain the error
messages when not dying, and mistakenly assumed that hold_locked_index()
would do the same.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help