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

[bug]: curl: option --tcp-fastopen: is unknown #298

Closed
raivivek opened this issue Jun 2, 2022 · 4 comments · Fixed by #299
Closed

[bug]: curl: option --tcp-fastopen: is unknown #298

raivivek opened this issue Jun 2, 2022 · 4 comments · Fixed by #299
Labels
bug Something isn't working released

Comments

@raivivek
Copy link

raivivek commented Jun 2, 2022

Describe the bug

Using zinit to setup a few plugins on a remote machine where the installed curl version does not support this flag. Is it possible to specify a different downloader through an environment variable or otherwise?

Setting up snippet: OMZ::plugins/bedtools
Downloading `OMZ::plugins/bedtools` (with curl, wget, lftp)…
curl: option --tcp-fastopen: is unknowncurl: try 'curl --help' or 'curl --manual' for more information
ERROR: Download failed.
❯ /bin/curl --version
curl 7.29.0 (x86_64-redhat-linux-gnu) libcurl/7.29.0 NSS/3.53.1 zlib/1.2.7 libidn/1.28 libssh2/1.8.0
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtsp scp sftp smtp smtps telnet tftp

Features: AsynchDNS GSS-Negotiate IDN IPv6 Largefile NTLM NTLM_WB SSL libz unix-sockets

Steps to reproduce

  1. Enable any plugin
  2. Use a version of curl that does not support --tcp-fastopen
  3. Plugin download fails

Expected behavior

Plugin is installed successfully.

Screenshots and recordings

No response

Operating System & Version

linux-gnu | pc | x86_64 | x86_64 | x86_64 x86_64

Zsh version

zsh 5.8.1 (x86_64-pc-linux-gnu)

Terminal emulator

xterm-kitty

If using WSL on Windows, which version of WSL

No response

Additional context

No response

@raivivek raivivek added the bug Something isn't working label Jun 2, 2022
@vladdoster
Copy link
Member

Okay, I'll revert this change. The option sped up downloads. Sorry about this.

@vladdoster
Copy link
Member

@raivivek Merged fix! Thanks again for reporting the issue :)!

@raivivek
Copy link
Author

raivivek commented Jun 4, 2022

Thanks! This works for me. Although if it was a truly useful feature to have, perhaps one could detect if the flag is supported and only then use it. Maybe another ice that allows one to change download tool would work too. Just some thoughts! Thanks again!

@github-actions
Copy link

github-actions bot commented Nov 7, 2022

🎉 This issue has been resolved in version 3.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants