From: Jonathan Nieder <jrnieder@gmail.com>
Date: Wed, 27 Jul 2011 05:05:15 GMT
Subject: Re: [PATCH 12/18] revert: Save command-line options for continuing operation
Message-ID: <20110727050515.GH18470@elie>
In-Reply-To: <1311736755-24205-13-git-send-email-artagnon@gmail.com>

Ramkumar Ramachandra wrote:

> In the same spirit as ".git/sequencer/head" and ".git/sequencer/todo",
> introduce ".git/sequencer/opts" to persist the replay_opts structure
> for continuing after a conflict resolution.  Use the gitconfig format
> for this file so that it looks like:
>
> [core]
> 	signoff = true
> 	record-origin = true
> 	mainline = 1
> 	strategy = recursive
> 	strategy-option = patience
> 	strategy-option = ours

Maybe "[options]" instead of "[core]" would be more self-explanatory.

If this is meant to be closely analagous to the gitconfig file, then
variable names should not contain dashes.  It would be nice if some
day the config parser could learn to treat dashes as insignificant,
just like it already treats case distinctions as insignificant.