Skip to content

[2.7.x] Set tinyxml2_LIBRARIES after find_package()#3932

Merged
danmar merged 1 commit into
cppcheck-opensource:2.7.xfrom
c72578:2022-03-23_tinyxml2_update_for_2.7.x_branch
Mar 25, 2022
Merged

[2.7.x] Set tinyxml2_LIBRARIES after find_package()#3932
danmar merged 1 commit into
cppcheck-opensource:2.7.xfrom
c72578:2022-03-23_tinyxml2_update_for_2.7.x_branch

Conversation

@c72578

@c72578 c72578 commented Mar 23, 2022

Copy link
Copy Markdown
Contributor

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 commented Mar 23, 2022

Copy link
Copy Markdown
Contributor Author

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 cppcheck-opensource:2.7.x Mar 25, 2022
@danmar

danmar commented Mar 25, 2022

Copy link
Copy Markdown
Collaborator

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