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

Addresses issue #487 #488

Merged
merged 1 commit into from
Jun 18, 2023
Merged

Conversation

louis-langholtz
Copy link
Owner

Description - What's this PR do?

Addresses issue #487.

See also response by Craig Scott at
https://gitlab.kitware.com/cmake/cmake/-/issues/20636 asserting that having COMPONENT after INCLUDES DESTINATION is "malformed".

Related Issues

See also response by Craig Scott at
https://gitlab.kitware.com/cmake/cmake/-/issues/20636
asserting that having COMPONENT after INCLUDES DESTINATION is
"malformed".
@louis-langholtz louis-langholtz added Bug For issues or changes that describe or fix bugs. CMake Use to associate with the CMake tool. labels Jun 18, 2023
@louis-langholtz louis-langholtz added this to the 2.0 Release milestone Jun 18, 2023
@louis-langholtz louis-langholtz self-assigned this Jun 18, 2023
@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@coveralls
Copy link

Coverage Status

coverage: 98.767%. remained the same when pulling 9383689 on change_cmake_install_libraries into 2db57cd on master.

@louis-langholtz louis-langholtz merged commit 6c207ae into master Jun 18, 2023
19 checks passed
@louis-langholtz louis-langholtz deleted the change_cmake_install_libraries branch June 18, 2023 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For issues or changes that describe or fix bugs. CMake Use to associate with the CMake tool.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants