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

VC: remove the makefile.vc6 build infra #1215

wants to merge 11 commits into from


Copy link

@bagder bagder commented Jan 16, 2017

All hail winbuild/

Copy link

@bagder, thanks for your PR! By analyzing the history of the files in this pull request, we identified @yangtse, @captain-caveman2k and @gknauf to be potential reviewers.

@bagder bagder added the build label Jan 16, 2017
bagder and others added 10 commits January 20, 2017 00:07
By supporting this, subsequent connects would load a lot less data from

Closes #1110
- Fix logic error in Curl_ossl_random.

Broken a few days ago in 807698d.
Regression since 1d4202a, which moved the buffer into a more narrow
scope, but the data in that buffer was used outside of that more narrow

Reported-by: Dan Fandrich
Fix compile warnings that appeared only when curl has been configured
with '--disable-verbose'.
Implemented since curl-7_36_0-130-g8868a22

Reported-by: Fahim Chandurwala
Reported-by: Gisle Vanem
Replace use of fixed macro BUFSIZE to define the size of the receive
buffer.  Reappropriate CURLOPT_BUFFERSIZE to include enlarging receive
buffer size.  Upon setting, resize buffer if larger than the current
default size up to a MAX_BUFSIZE (512KB). This can benefit protocols
like SFTP.

Closes #1222
@bagder bagder closed this in 8611d98 Jan 23, 2017
@bagder bagder deleted the remove-makefile-vc6 branch January 23, 2017 13:32
peterpih pushed a commit to railsnewbie257/curl that referenced this pull request Jan 24, 2017
The winbuild/ build files is now the single MSVC makefile build choice.

Closes curl#1215
@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
None yet

Successfully merging this pull request may close these issues.

None yet

7 participants