-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Adding Botan 3.0.0 recipe and make it compatible with Conan 2.x #18079
Merged
conan-center-bot
merged 57 commits into
conan-io:master
from
lieser:17247-botan-upgrade-conanfile-2.0
Feb 16, 2024
Merged
Changes from all commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
506c84e
adding Botan 3.0.0 recipe
Maaown e15db3e
updated to conan 2
Maaown 1ae7f48
ci lint fix
Maaown fa36a8d
ci lint fix
Maaown c59b651
ci lint fix
Maaown 434a062
replacing version check
Maaown 538273e
some more fixes to be ready for 2.0
memsharded 4edba84
fixes
Maaown 4d4ac88
Keep old test package for Conan 1.x
lieser 057e16f
Order imports
lieser 5ef9c7b
Remove unused import
lieser 1bd7805
Fix packaging
lieser 8c35266
Fix msvc build and test
lieser c36f70a
Use Version helper in package_info
lieser 4711f28
Small fixes
lieser d3d1cdf
Don't add -fPIC for msvc
lieser 1c03491
Fix msvc package info for Botan 3.0.0
lieser 0e38f72
Add missing patch_description and patch_type
lieser bd486ed
Revert back to "sources" name for src folder
lieser b00ed3c
Try fixing KB-H010 for Conan 1
lieser 58e1082
Remove base_path for patches and switch again to `src` for src folder
lieser 1cfa026
Don't import from conan.tools.microsoft.subsystems
lieser 1479347
Fix KB-H010 by using double quotes
lieser 28ba807
Disable getentropy
lieser 0625d76
Move layout method
lieser 23ccba0
Add check for minimal compiler version
lieser de1edb8
Fix getting env variables
lieser fcb0fc9
Remove some older Botan versions
lieser 0db2689
Fix removal of old versions
lieser 0a16d99
Replace apple_deployment_target_flag with apple_min_version_flag
lieser 3823a16
Disable getrandom
lieser 00fdd96
Set 11.2 as minimal GCC
lieser fe6a674
Set C++ standard in test package according to Botan version
lieser 669edab
Add min version for apple-clang
lieser d70872d
Remove redundant cpp_info.names["pkg_config"]
lieser 9a7504c
Use leading _ for member variable extra_cxxflags
lieser 0df7c2b
Readd comment
lieser fb2ca6d
Further improvements
AbrilRBS 2d04e27
Support for the OpenSSL provider was removed in Botan 2.19.2
lieser ea11907
Add support for tools.build:sysroot
lieser 27744a3
add a patch to support getrandome() in glibc < 2.25
reneme 42f87c6
add Botan 3.1.0 and 3.1.1
reneme 3a1db0e
add Botan 3.2.0
reneme 821dca3
Botan 2.x is not compatible with OpenSSL 3.x
reneme d47adc9
pin to specific OpenSSL version (1.1.1s)
reneme 12ca6c7
FIX: warnings KB-H043 and KB-H077
reneme 3a06093
don't explicitly disable getrandom/getentropy on Linux
reneme d29e8c9
Apply suggestions from code review
reneme e029672
add patch sources to patches
reneme 2ab02ec
Apply suggestions from code review
reneme 3060781
FIX: linter warnings
reneme 6a69bc3
FIX: default of CXXFLAGS
reneme 3379fd0
update recipe's meta data
reneme 16aca89
use self.dependencies['boost'].options
reneme 4915388
add -o disable_modules
reneme 6c52402
Workaround to support glibc < 2.25
reneme 8caf736
Apply suggestions from code review
reneme File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't remember if was discussed before, but these patches are in the upstream already or is there a PR? It would be nice including it in the upstream first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This backport is partly from one of the PRs you yourself created upstream 😁. It combines the merged PRs:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please also note that the recipe applies these patches to Botan 3.0.0 and 3.1.x, only. Besides those two versions, this pull request also adds Botan 3.2.0, which already includes them in the upstream release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@uilianries poke, poke 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lieser Could you please add
patch_source
then? Try to keep it as a standard, so we can track any new patch easier 😉 https://github.com/conan-io/conan-center-index/blob/master/docs/adding_packages/conandata_yml_format.md#patch_sourceThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a nice one! Thanks for pointing that out. Fixed now in: 616e0df (along with changing the
patch_type
on the 2.18.2 release to "bugfix" as this is more suitable for the backported build fix).