Skip to content

Conversation

@c72578
Copy link
Contributor

@c72578 c72578 commented Mar 23, 2022

If tinyxml2 is found by find_package(), then tinyxml2_LIBRARIES
is empty. Set tinyxml2_LIBRARIES to "tinyxml2::tinyxml2" in this case.

  • Fixes "undefined reference to tinyxml2::`"
  • printInfo.cmake: Fix indentation of tinyxml2_LIBRARIES

@c72578
Copy link
Contributor Author

c72578 commented Mar 23, 2022

This is based on #3918 for the 2.7.x branch.

If tinyxml2 is found by find_package(), then tinyxml2_LIBRARIES
is empty. Set tinyxml2_LIBRARIES to "tinyxml2::tinyxml2" in this case.

- Fixes "undefined reference to `tinyxml2::"
- printInfo.cmake: Fix indentation of tinyxml2_LIBRARIES
@c72578 c72578 force-pushed the 2022-03-23_tinyxml2_update_for_2.7.x_branch branch from 6049995 to 4ae07fe Compare March 23, 2022 20:42
@c72578 c72578 changed the title Set tinyxml2_LIBRARIES after find_package() [2.7.x] Set tinyxml2_LIBRARIES after find_package() Mar 25, 2022
@danmar danmar merged commit 1cc234a into danmar:2.7.x Mar 25, 2022
@danmar
Copy link
Owner

danmar commented Mar 25, 2022

I am fixing the CI error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants