CMake: Use root project identification and make proper options for install + shared/static libs #1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The issue found was that when using GLM via
FetchContent_Declare
, if different paths were specified in the call (i.e. src path, build path, etc). this would cause GLM to not properly install and would become invalid to projects attempting to use GLM in a non-header only fashion.This PR accomplishes three things -
GLM_INSTALL
is defined asON
. Similar to today's master branch, however it gives end-users the ability to still install the project.This allows for determining specifically if GLM will build as static or shared (or neither).
This allows for the static/shared versions of GLM to be installed.