Re: [PATCH] rebase -i: improve error message when picking merge

[phillip.wood123@gmail.com]

Hi Rubén

On 06/04/2024 15:24, Rubén Justo wrote:
> On Fri, Apr 05, 2024 at 10:30:37AM +0100, phillip.wood123@gmail.com wrote:
>> On 04/04/2024 20:44, Rubén Justo wrote:
>>> On Mon, Feb 26, 2024 at 10:58:07AM +0000, Phillip Wood via GitGitGadget wrote:
>>> ... I find these repeated messages confusing:
>>>
>> I think that printing the error messages when the todo list is parsed before
>> it is given to the user to edit is helpful as it reminds the user what the
>> problem is. Your test looks confusing because it doesn't really simulate the
>> user editing the todo list.
> 
> Certainly, the test was not clear to express my confusion.
> 
> The ones that are printed _before_ the editor is run are the ones that
> confuse me, because when the user exits the editor we leave those
> lines there:
> 
> 	$ GIT_EDITOR='sed -i s/pick/merge/' ./git rebase --edit
> 	error: 'pick' does not accept merge commits, please use 'merge -C'
> 	error: invalid line 1: pick 17381ab62a
> 
> But maybe it is my interpretation.  Your reasoning of giving it as a
> help to the user makes sense.

Ideally we'd clear the error messages when the user edited the todo list 
but there isn't an easy way to do that. In principle we could stop 
printing the error messages when parsing the list before it is edited 
but I'm not sure how easy that would be. The behavior you are seeing is 
already present is not changed by this patch - existing error messages 
behave in the same way.

Best Wishes

Phillip

> Thank you for your explanations.
> 
>> Best Wishes
>>
>> Phillip