Silence CMake policy CMP0072

[chapulina]

🦟 Bug fix

• Fixes CMake policy CMP0072 #360

Summary

Calling cmake_policy is complicated in this situation because the find_package call is made in a different scope, within ign_find_package. Setting the variable directly does the trick though. See #360 (comment).

This doesn't solve the problem for downstream projects though. I didn't find a way of passing variables to ignition-component-config.cmake.in.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸

[codecov]

Codecov Report

Merging #528 (aac778b) into ign-rendering3 (e4ad6da) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           ign-rendering3     #528   +/-   ##
===============================================
  Coverage           53.35%   53.35%           
===============================================
  Files                 131      131           
  Lines               12035    12035           
===============================================
  Hits                 6421     6421           
  Misses               5614     5614           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e4ad6da...aac778b. Read the comment docs.

[iche033]

looks good to me.

[osrf-triage]

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1

[osrf-triage]

This pull request has been mentioned on Gazebo Community. There might be relevant details there:

https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1