Skip to content

Makefile.m32: rename -winssl option to -schannel and tidy up#8053

Closed
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:m32sch
Closed

Makefile.m32: rename -winssl option to -schannel and tidy up#8053
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:m32sch

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Nov 24, 2021

  • Accept -schannel as an alternative to CFG option -winssl. (Latter is still accepted, but deprecated)
  • Rename internal variable WINSSL to SCHANNEL
  • Make the CFG option evaluation shorter, without repeating the option name

Let me know if dropping the previous, -winssl option name would be desirable/acceptable.

Which is the same as the existing -winssl CFG option.
This marks -winssl deprecated.
By altering the condition a little bit and to avoid repeating the
option names.
@vszakats vszakats changed the title Makefile.m32: Replace -winssl option with -schannel and tidy up Makefile.m32: replace -winssl option with -schannel and tidy up Nov 24, 2021
@vszakats vszakats changed the title Makefile.m32: replace -winssl option with -schannel and tidy up Makefile.m32: rename -winssl option to -schannel and tidy up Nov 24, 2021
@vszakats vszakats added build tidy-up Windows Windows-specific labels Nov 24, 2021
@vszakats vszakats closed this in 8c0336c Nov 25, 2021
@vszakats
Copy link
Copy Markdown
Member Author

Thanks for the reviews, merged. Test build: https://ci.appveyor.com/project/curlorg/curl-for-win/builds/41669517

@vszakats vszakats deleted the m32sch branch November 25, 2021 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants