X Tutup
The Wayback Machine - https://web.archive.org/web/20210103135353/https://github.com/git-ftp/git-ftp/pull/472
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update INSTALL.md #472

Open
wants to merge 1 commit into
base: master
from
Open

Update INSTALL.md #472

wants to merge 1 commit into from

Conversation

@lennard0711
Copy link

@lennard0711 lennard0711 commented Sep 19, 2018

Windows 10 comes now preinstalled with the correct cURL version so you don't need to install it by yourself anymore

lennard7001
Windows 10 comes now preinstalled with the correct cURL version so you don't need to install it by yourself anymore
Copy link
Collaborator

@LukasFritzeDev LukasFritzeDev left a comment

Thank you for this information. We don’t have a Windows so machine so we can't help here, but I left a few comments.

Could you please rebase your pull request on top of the current master (there have been some other changes since then) to avoid conflicts, and update this? It would be great to finally merge this.

In order to use SFTP, [download curl](http://curl.haxx.se/download.html) for
Windows with SFTP support.
Win32 2000/XP MSI or Win64 2000/XP x86_64 MSI is recommended.
If you installed curl, then remove `bin/curl.exe` from your Git for Windows
installation directory. It will fall back to the newly installed version.
installation directory. It will fall back to the newly installed version.~~

This comment has been minimized.

@LukasFritzeDev

LukasFritzeDev Jun 18, 2019
Collaborator

I would suggest not crossing that off. There may still be users of older versions of Windows who need this information. So why not say something like:

+ Older versions of Windows come with curl installed, but …
In order to use SFTP, [download curl](http://curl.haxx.se/download.html) for
Windows with SFTP support.
Win32 2000/XP MSI or Win64 2000/XP x86_64 MSI is recommended.
If you installed curl, then remove `bin/curl.exe` from your Git for Windows
installation directory. It will fall back to the newly installed version.
installation directory. It will fall back to the newly installed version.~~
Windows 10 comes now preinstalled with the correct cURL version.

This comment has been minimized.

@LukasFritzeDev

LukasFritzeDev Jun 18, 2019
Collaborator

Can you specify which version of Windows 10 introduces the updated cURL version?
Again maybe a user has a older version. Then this information could be useful.
Maybe like:

+ Since Windows 10 version XXXX a cURL installation supporting SFTP is preinstalled.

And then the suggestion from above.

This comment has been minimized.

@fxmauricard

fxmauricard Jul 31, 2020

If we do curl -V in Windows 10 2004 we have this:

curl 7.55.1 (Windows) libcurl/7.55.1 WinSSL
Release-Date: 2017-11-14, security patched: 2019-11-05
Protocols: dict file ftp ftps http https imap imaps pop3 pop3s smtp smtps telnet tftp
Features: AsynchDNS IPv6 Largefile SSPI Kerberos SPNEGO NTLM SSL

If we do curl -V in the latest Git Bash we have this:

curl 7.71.1 (x86_64-w64-mingw32) libcurl/7.71.1 OpenSSL/1.1.1g (Schannel) zlib/1.2.11 brotli/1.0.7 libidn2/2.3.0 libssh2/1.9.0 nghttp2/1.41.0
Release-Date: 2020-07-01
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtsp scp sftp smtp smtps telnet tftp
Features: AsynchDNS brotli HTTP2 HTTPS-proxy IDN IPv6 Kerberos Largefile libz Metalink MultiSSL NTLM SPNEGO SSL SSPI TLS-SRP

So, AFAIK, there's no support for SFTP unless we use the cURL provided on the cURL website or the latest Git for Windows.
In fact it was only broken between version 2.16.2 and 2.25 versions. From version 2.25.1, SFTP support in the included cURL was restored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.
X Tutup