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

Upstream FindBotan.cmake to CMake project #2121

Open
securitykernel opened this issue Aug 23, 2023 · 9 comments
Open

Upstream FindBotan.cmake to CMake project #2121

securitykernel opened this issue Aug 23, 2023 · 9 comments

Comments

@securitykernel
Copy link

It has been a long-term issue that CMake does not ship with a FindBotan.cmake find module to search for and link against a Botan installation. People start writing their own find modules again and again, that basically do the same, which you can see GitHub search, for example. For existing but also new users of Botan, it would be great if they had out-of-the-box support for Botan in CMake projects.

I found that the FindBotan script you ship with rnp looks the most mature and already supports Botan 2.x and 3.x. Would it be possible to upstream it to the CMake project? The only requirement from CMake's side would be to find a dedicated module maintainer. If none of you can do this, I would be willing to step in to make this happen.

/cc @randombit @reneme @lieser

@ni4
Copy link
Contributor

ni4 commented Aug 23, 2023

@ronaldtse what do you think?
We already have separate repository for CMake modules, used in rnp: https://github.com/rnpgp/cmake-modules/
Actually never know/think before about upstreaming CMake modules to the CMake repository, but the idea itself looks good to me.

P.S. Thanks for raising this!

@securitykernel
Copy link
Author

Any update on this?

@ni4
Copy link
Contributor

ni4 commented Sep 15, 2023

@securitykernel Sorry for a delay. I absolutely don't mind moving it to upstream/CMake. However let's wait for @ronaldtse reply.

@securitykernel
Copy link
Author

Sure, sounds fine!

@ronaldtse
Copy link
Contributor

@securitykernel thank you for the suggestion! We'd be happy to contribute it upstream. Would you be willing to jointly maintain the module upstream?

@securitykernel
Copy link
Author

Sure, let's do it!

@securitykernel
Copy link
Author

Asked the CMake maintainers if it would be accepted. Waiting for their response. Meanwhile, I started to integrate it into their repo. Are you fine with re-licensing it to BSD 3-Clause License, as this is the default for all find modules shipped with CMake?

@ronaldtse
Copy link
Contributor

@securitykernel yes we would be willing to re-license the module to BSD 3-clause -- BSD 2-clause is more general than BSD 3-clause so it is fine.

@securitykernel
Copy link
Author

So we got everything right now, I think. Can you please test that randombit/botan#3722 works for you? That would give us a bit of more confirmation.

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

3 participants