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
Use BUILD_SHARED_LIBS instead of QTKEYCHAIN_STATIC? #180
Comments
That seems to be the cleaner way indeed. One might keep QTKEYCHAIN_STATIC around as "alias" to not break people's builds although that somehow defeats the purpose of the cleanup... |
I think both can be supported for some transition perioud, with |
Fixed in #204 |
Be-ing
added a commit
to Be-ing/qtkeychain
that referenced
this issue
Nov 11, 2021
BUILD_SHARED_LIBS is the standard CMake variable for controlling whether libraries are built dynamically or statically. Using a nonstandard variable requires extra work for packagers. https://cmake.org/cmake/help/latest/variable/BUILD_SHARED_LIBS.html Fixes frankosterfeld#180
Be-ing
added a commit
to Be-ing/qtkeychain
that referenced
this issue
Nov 11, 2021
BUILD_SHARED_LIBS is the standard CMake variable for controlling whether libraries are built dynamically or statically. Using a nonstandard variable requires extra work for packagers. https://cmake.org/cmake/help/latest/variable/BUILD_SHARED_LIBS.html Fixes frankosterfeld#180
frankosterfeld
pushed a commit
that referenced
this issue
Nov 18, 2021
BUILD_SHARED_LIBS is the standard CMake variable for controlling whether libraries are built dynamically or statically. Using a nonstandard variable requires extra work for packagers. https://cmake.org/cmake/help/latest/variable/BUILD_SHARED_LIBS.html Fixes #180
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there any particular reason why
QTKEYCHAIN_STATIC
is used in order to switch between static/shared library instead ofBUILD_SHARED_LIBS
? Is it just to build a shared library by default?The text was updated successfully, but these errors were encountered: