From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 03 May 2016 17:50:50 GMT
Subject: Re: [PATCH v4 09/11] connect: use "-l user" instead of "user@" on ssh command line
Message-ID: <xmqqk2jbni85.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <8f470378-07a9-525b-ff8b-f0de011019cb@web.de>

Torsten Bögershausen <tboegi@web.de> writes:

> On 2016-05-03 10.50, Mike Hommey wrote:
>> While it is not strictly necessary, it makes the connect code simpler
>> when there is user.
>> 
>
> That commit message does't tell too much, I think.

"Doesn't tell too much" is not necessarily bad, but "tells too
little" is, and I think this tells me enough to say it is not a good
change ;-)

> Besides that, I'm sure it will break (at least) my ssh wrapper scripts,
> which rely on user@host to be passed into the script.

Thanks for bringing it up.  "By reducing the language we accept it
makes my coding simpler" is not a good excuse to break existing
users, and "While it is not strictly necessary, " is a good hint
that the author _knows_ that the change can either (1) be done
without, or (2) be done in a way that does not break existing users
and yet make the end result easier to read.