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

Add missing Threads dependency to Config.cmake.in #278

Merged
merged 1 commit into from
May 14, 2020

Conversation

sum01
Copy link
Contributor

@sum01 sum01 commented May 14, 2020

Without this, systems that linked against Threads::Threads (namely
Linux) don't "just work" with find_package(SQLiteCpp), but instead
require you to find & link against Threads manually.

So this fixes that by finiding it for you, if it was used.

Without this, systems that linked against Threads::Threads (namely
Linux) don't "just work" with find_package(SQLiteCpp), but instead
require you to find & link against Threads manually.

So this fixes that by finiding it for you, if it was used.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 42f476d on sum01:fix_cmake_config into a5e1c70 on SRombauts:master.

@SRombauts SRombauts self-assigned this May 14, 2020
@SRombauts SRombauts merged commit f0cd9e3 into SRombauts:master May 14, 2020
@SRombauts
Copy link
Owner

Thanks !

@sum01 sum01 deleted the fix_cmake_config branch May 14, 2020 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants