Re: [RFC] custom strategies in builtin-merge
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jul 26, 2008, 00:51 UTC
- Message-ID
- <alpine.DEB.1.00.0807260250480.11976@eeepc-johanness>
- In-Reply-To
- <7viqutpjqq.fsf@gitster.siamese.dyndns.org>
Hi,
On Fri, 25 Jul 2008, Junio C Hamano wrote:
Show 21 quoted lines
> "Sverre Rabbelier" <alturin@gmail.com> writes:
>
> > On Fri, Jul 25, 2008 at 13:33, Miklos Vajna <vmiklos@frugalware.org> wrote:
> >> 1) Maintain a list of commands that has a git-merge- prefix, but not a
> >> strategy. This list would currently contain "base, file, index,
> >> one-file and tree".
> >
> > Sounds a bit error prone, and could lead to unexpected results if/when
> > someone creates a new command ('git merge status' anyone?) which is
> > then suddenly treated as a merge strategy.
> >
> >> 2) Require custom strategies to have a different naming scheme, like
> >> if "foo" is a custom strategy, then it would have to be named
> >> git-merge-custom-foo, _not_ git-merge-foo.
> >
> > I think this is cleaner, what would be even nicer is to change the
> > current names too, so name them all "git-merge-stragegy-foo".
>
> I think you could retroactively impose "git-merge-strategy-" prefix rule
> and grandfather the existing ones by maintaining a list of them that by
> definition will not grow anymore.... especially since I hope that we have them builtin soon, and not only that, but have builtin-merge call them as C functions, not with fork() and exec().
Ciao, Dscho