8000
Skip to content

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

Merged
Groovounet merged 1 commit intog-truc:masterfrom
ffontaine:master
Mar 1, 2019
Merged

CMakeLists: add BUILD_{SHARED,STATIC}_LIBS options#871
Groovounet merged 1 commit intog-truc:masterfrom
ffontaine:master

Conversation

@ffontaine
Copy link
Copy Markdown

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
Copy Markdown
Mem 8000 ber

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

Thanks for contributing.

@ffontaine
Copy link
Copy Markdown
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
Copy Markdown

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

0