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

Review multi fixes in weyou's fork #19

Open
bovine opened this issue Mar 18, 2021 · 3 comments
Open

Review multi fixes in weyou's fork #19

bovine opened this issue Mar 18, 2021 · 3 comments

Comments

@bovine
Copy link
Member

bovine commented Mar 18, 2021

@weyou made some fixes to multi curl handling that might be useful to incorporate back into this repo.

It looks like there were also some Windows-specific fixes also, which may or may not still be necessary after the recent work by apnadkarni.

master...weyou:master

@wrenashe
Copy link

Yes, please.

@bovine
Copy link
Member Author

bovine commented Jun 1, 2022

Yes, please.

Is there anything specific that you are aware of in that fork that would be important? Would you be able to help identify those things?

@wrenashe
Copy link

wrenashe commented Jun 2, 2022

Without that fix, TclCurl does not work with the later Curl lib version, like 7.72.0. And it looks not Windows only, I tried Linux, TclCurl does not work either without that patch.

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

No branches or pull requests

2 participants