Re: [PATCH 3/5] ssh: 'auto' variant to select between 'ssh' and 'simple'

From

Jonathan Nieder <jrnieder@gmail.com>

Date

Oct 25, 2017, 16:32 UTC

Message-ID

<20171025163243.xmm7szrkwgblpgcc@aiede.mtv.corp.google.com>

In-Reply-To

<CAGZ79kYTYg09A7UbhvyiT0QmRQG5ZJV6v1iaQBNCZY8RTj300A@mail.gmail.com>

Hi,

Stefan Beller wrote:

> On Wed, Oct 25, 2017 at 5:51 AM, Johannes Schindelin
> <Johannes.Schindelin@gmx.de> wrote:

>> This breaks on Windows. And it is not immediately obvious how so, so let
>> me explain:

[nice explanation snipped]

>> As a consequence, the test fails. Could you please squash in this, to fix
>> the test on Windows?
>
> This explanation is in detail and would even make a good commit message
> for a follow up commit. (Squashing just that line would loose the explanation
> as I suspect the original commit will not dedicate so much text to
> this single line)

Thanks for catching and diagnosing this, Dscho!

Jonathan