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

CMakeLists: add BUILD_{SHARED,STATIC}_LIBS options #871

Merged
merged 1 commit into from
Mar 1, 2019
Merged

CMakeLists: add BUILD_{SHARED,STATIC}_LIBS options #871

merged 1 commit into from
Mar 1, 2019

Conversation

ffontaine
Copy link

Add BUILD_SHARED_LIBS and BUILD_STATIC_LIBS options to allow the user
to enable/disable building of shared and static library

This is especially useful when cross-compiling as some toolchains are
not always able to build a shared or a static library

Signed-off-by: Fabrice Fontaine fontaine.fabrice@gmail.com

Add BUILD_SHARED_LIBS and BUILD_STATIC_LIBS options to allow the user
to enable/disable building of shared and static library

This is especially useful when cross-compiling as some toolchains are
not always able to build a shared or a static library

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@Groovounet Groovounet self-assigned this Mar 1, 2019
@Groovounet Groovounet added this to the GLM 0.9.9 milestone Mar 1, 2019
@Groovounet Groovounet merged commit a40bd4d into g-truc:master Mar 1, 2019
@Groovounet
Copy link
Member

I might ask you to support this feature here some users encounter issues with it.

Thanks for contributing.

@ffontaine
Copy link
Author

I'll be pleased to support this feature however I have a small question: why does the glm CMakeLists.txt has provision to build static and shared libraries if it's a header-only library?

@ivanstepanovftw
Copy link

I'll be pleased to support this feature however I have a small question: why does the glm CMakeLists.txt has provision to build static and shared libraries if it's a header-only library?

I still doesn't know why, but I suggest:

  1. They wanted to link library to every test to build tests faster
  2. Code inspection does not worked because of bad CMake infrastructure

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

Successfully merging this pull request may close these issues.

3 participants