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

Sync CURL_DISABLE options in CMake #7624

Closed
wants to merge 1 commit into from

Conversation

@donny-dont
Copy link
Contributor

@donny-dont donny-dont commented Aug 24, 2021

Adds the full listing of CURL_DISABLE options to the CMakeLists.txt. Moves all option code, except for CURL_DISABLE_OPENSSL_AUTO_LOAD_CONFIG which resides near OpenSSL configuration, to the same block of code. Also sorts the options.

Additionally sorted the CURL-DISABLE listing and fixed the CURL_DISABLE_POP3 option.

@donny-dont donny-dont force-pushed the sync-cmake-options branch 2 times, most recently from c3241b7 to 0de640e Aug 24, 2021
@deepcode-ci-bot
Copy link

@deepcode-ci-bot deepcode-ci-bot bot commented Aug 24, 2021

Congratulations 🎉. DeepCode analyzed your code in 0.46 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

👉 The DeepCode service and API will be deprecated in August, 2021. Here is the information how to migrate. Thank you for using DeepCode 🙏 ❤️ !

If you are using our plugins, you might be interested in their successors: Snyk's JetBrains plugin and Snyk's VS Code plugin.

@bagder bagder added the cmake label Aug 24, 2021
docs/CURL-DISABLE.md Outdated Show resolved Hide resolved
Adds the full listing of CURL_DISABLE options to the CMake build. Moves all option code, except for CURL_DISABLE_OPENSSL_AUTO_LOA_CONFIG which resides near OpenSSL configuration, to the same block of code. Also sorts the options here and in the cmake config header.

Additionally sorted the CURL-DISABLE listing and fixed the CURL_DISABLE_POP3 option.
@donny-dont donny-dont force-pushed the sync-cmake-options branch from 0de640e to 8747c83 Aug 24, 2021
@donny-dont
Copy link
Contributor Author

@donny-dont donny-dont commented Aug 25, 2021

Ok I think I got everything ready @bagder

@bagder bagder self-assigned this Aug 25, 2021
bagder
bagder approved these changes Aug 25, 2021
@bagder
Copy link
Member

@bagder bagder commented Aug 25, 2021

Thanks!

@bagder bagder closed this in 62dbfa3 Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants