Re: [PATCH 5/5] ssh: 'simple' variant does not support --port

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

 



On Mon, Oct 23, 2017 at 2:33 PM, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote:
> When trying to connect to an ssh:// URL with port explicitly specified
> and the ssh command configured with GIT_SSH does not support such a
> setting, it is less confusing to error out than to silently suppress
> the port setting and continue.
>
> This requires updating the GIT_SSH setting in t5603-clone-dirname.sh.
> That test is about the directory name produced when cloning various
> URLs.  It uses an ssh wrapper that ignores all its arguments but does
> not declare that it supports a port argument; update it to set
> GIT_SSH_VARIANT=ssh to do so.  (Real-life ssh wrappers that pass a
> port argument to OpenSSH would also support -G and would not require
> such an update.)
>
> Reported-by: William Yan <wyan@xxxxxxxxxx>
> Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx>
> ---
> That's the end of the series.  Thanks for reading.

Patches 4 & 5 look good to me,

Thanks,
Stefan



[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