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

Re: [PATCH 2nd try] Make git-fmt-merge-msg a builtin

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:42:32

Possibly related (same subject, not in this thread)

Hi,

On Mon, 3 Jul 2006, Junio C Hamano wrote:
Johannes Schindelin [off-list ref] writes:
quoted
Signed-off-by: Johannes Schindelin <redacted>

---
	This retires git-fmt-merge-msg.perl, since it passes all the
	tests, but removes the Perl version not now.
There is no point not removing the script if you update git.c
and Makefile to point at the new one.
I should have put PATH/RFC. I did not want to remove the script right 
away, so people could compare outputs (like Martin did with 
git-format-patch).
We do not have a test specific for fmt-merge-msg, so it
obviously passes all the tests ;-).  A new one is attached.
Well, t4013 showed me two output bugs, but it evidently did not check for 
merge.summary.
I think we should extend boolean to accept 'yes' and 'no',
In that case, could we have "YesPlease", "oui", "si" and "da", too? ;-)

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