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

[FEATURE] Add support for Botan 3 #2786

Open
triallax opened this issue May 25, 2023 · 12 comments
Open

[FEATURE] Add support for Botan 3 #2786

triallax opened this issue May 25, 2023 · 12 comments

Comments

@triallax
Copy link

Is your feature request related to a problem? Please describe.

Just trying to bump Botan to 3 in Void Linux, but blocked by QOwnNotes supporting only Botan 2 (Void doesn't use the vendored Botan provided by QOwnNotes).

Describe the solution you'd like

Add support for Botan 3.

@pbek
Copy link
Owner

pbek commented May 25, 2023

Probably will need integration in https://github.com/pbek/QOwnNotes/blob/main/src/libraries/botan/botanwrapper.cpp and the qmake and CMake config.

PRs welcome.

Any thoughts, @Waqar144?

@pbek
Copy link
Owner

pbek commented May 25, 2023

NixOS is still at Botan 2. 😉

@Waqar144
Copy link
Contributor

afaik botan 3 uses c++ 20, so we can't upgrade to it. However, it might be possible to link to the system botan3.

@pbek
Copy link
Owner

pbek commented May 26, 2023

Hard to test that for me.
I didn't even manage to build QOwnNotes with external Botan 2 under NixOS yet. 😅

@TimB87
Copy link

TimB87 commented Jul 21, 2023

This appears solved with qt6 6.5.2 from an uneducated view. @Waqar144 can you confirm?
image

@Waqar144
Copy link
Contributor

Not sure I understand you but no, it's still botan 2 under the hood.

@TimB87
Copy link

TimB87 commented Jul 22, 2023

Sorry. I have botan 3 installed and just compiled the latest qt6 release against that - it works fine again over here.
@mhmdanas Can you possibly confirm?

@triallax
Copy link
Author

@TimB87 sure, I'll take a look and get back to you.

@triallax
Copy link
Author

@TimB87 From what I can see, QOwnNotes still only supports Botan 2? https://github.com/pbek/QOwnNotes/blob/ab6f39b5b81c3eaa3ba5689abacdaa40e4c5e41c/cmake/FindBotan2.cmake

Maybe you built QOwnNotes with the vendored Botan?

@TimB87
Copy link

TimB87 commented Jul 24, 2023

@mhmdanas you might be right, I don't see system botan linked to it

@Waqar144
Copy link
Contributor

We will not add bundled botan3 support in the near future (read 3-5 years at least), simply because we can't have code that uses anything above C++11. This is so that QON keeps compiling on old systems (win xp, centos7).

However, any PRs that add support for system botan3 conditionally are welcome. It should be possible I believe without any big changes.

@pbek
Copy link
Owner

pbek commented Jul 25, 2023

Thank you, @Waqar144.
Plus, it's easy to use the system Botan (if that works with Botan 3), like for example I did in https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/applications/office/qownnotes/default.nix#L49.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants