Re: [PATCH v3 2/2] Ensure consistent usage of mergetool.keepBackup in git
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 11, 2009, 20:19 UTC
- Message-ID
- <7v4owvdju6.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <49E0F33B.8070806@pelagic.nl>
"Ferry Huberts (Pelagic)" <ferry.huberts@pelagic.nl> writes:
Show 23 quoted lines
> Ferry Huberts (Pelagic) wrote: >> Junio C Hamano wrote: >>> Ferry Huberts <ferry.huberts@pelagic.nl> writes: >>> >>>> In several places merge.keepBackup is used i.s.o. >>>> mergetool.keepBackup. This patch makes it all >>>> consistent for git >>> A silly question. The above makes it sound as if both merge.keepBackup >>> and mergetool.keepBackup are valid, but the latter is preferred, and your >>> patch makes things consistent even though without it there is nothing >>> broken per-se. >>> >>> Is that really is the case? >>> >>> Otherwise, perhaps the title and the message should read like this: >>> >>> Fix misspelled mergetool.keepBackup >>> >>> In several places mergetool.keepBackup was misspelled as merge.keepBackup. >>> >>> Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl> >> >> feel free to change it :-)
Notice that I didn't follow the discussion closely, and I am asking if this is a "bugfix" or "changing for consistency even though there is no breakage".