Thread (11 messages) flat view 11 messages, 3 authors, 2016-06-15

Re: [PATCH v5 3/5] merge: Make 'merge.log' an integer or boolean option

From: Ramkumar Ramachandra <hidden>
Date: 2016-06-15 22:49:23

Hi Junio,

Junio C Hamano writes:
This makes sense, but I agree with Jonathan's reviews on 1/5 and 2/5, I'll
ask you to reroll this on top of what I'll push out shortly.
Thanks! I'll rebase on top of rr/fmt-merge-msg in `pu` and re-roll
later today.
As to 4/5, I think we should proceed a lot more carefully than what your
patch does, just like we were extra careful when we went to v1.7.0 than
when we went to v1.6.0 (see release notes to 1.6.6 for a summary,
especially paying attention to the third and the fourth paragraph in
"Preparing yourselves" section).
Ok, I'll look at this and fix the patch accordingly.
I've queued 5/5 directly on maint.
Right, I saw that.

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