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

CMake version for Qt 6.6.x #20

Open
wants to merge 3 commits into
base: 6.4
Choose a base branch
from
Open

CMake version for Qt 6.6.x #20

wants to merge 3 commits into from

Conversation

bAmpT
Copy link

@bAmpT bAmpT commented Dec 9, 2023

Changes the build config to the qt_base version for the sqlite plugin that uses CMake. Then I also replaced the 3rdparty/libtomcrypt folder with a submodule to the develop branch to get the CMake config files.

Maybe this could be used as a start for the cmake version of the plugin (README needs some work)...

Copy link
Owner

@sjemens sjemens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems the most work is done.

#####################################################################

qt_internal_add_plugin(QSQLCipherPlugin
OUTPUT_NAME qsqlitecipher
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer the output name to remain qsqlcipher

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for cleanup

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for cleanup

qt_feature_module_end(NO_MODULE)

set(WITH_LTM OFF CACHE BOOL "Build with support for libtommath" FORCE)
add_subdirectory(3rdparty/libtomcrypt)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

set(BUILD_SHARED_LIBS OFF CACHE BOOL "Build the libtomcrypt as static library" FORCE)
Keep the plugin files to a minimum. Seems to be working but I am not sure if the BUILD_SHARED_LIBS is not used later

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

Successfully merging this pull request may close these issues.

2 participants