Re: [PATCH v5 1/1] mergetool: add automerge configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:

> Junio C Hamano wrote:
>> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:
>> 
>> >> Yup, I already mentioned UI mistake so you do not have to repeat
>> >
>> > You said it was a UI mistake, not me. I am a different mind than yours.
>> 
>> Yes, but the point is that I do not need to nor particularly want to
>> hear your opinion on the behaviour of "git merge-file".
>
>> I know (and others reading the thread on the list also know) that the
>> exit code of the command is misdesigned already.
>
> Unless you can read minds, you don't know that.

Actually I do, because they heard from me already ;-).  If this were
the case where our messages crossed, perhaps, but in this case yours
was a response to my message.

>> Again, please realize that on list discussion is a team effort to
>> come up together a better design of a shared solution.
>
> Which is why agreement in a team with different minds and different
> viewpoints is important.

It is not like opinions on all points are important.  Whether the
exit code from merge-file is or is not a UI mistake does NOT have
any influence on what we were discussing.  My opinion is that exit
code from merge-file is a UI mistake, but even if you disagree with
that, that would not change the conclusion we already reached that
the code should ignore its exit status, like you originally wrote.

I am already trying to ignore your opinions on things that do not
matter in the context of this project, as you told me earlier ;-)
But just like patches, messages are written only once but read by
many people, so I'd always aim for reducing noise at the source.

Anyway, happy holidays and pleasant new year to you and to
everybody.



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux