Open simonimpey opened 1 year ago
Hi @simonimpey
Thanks for your report.
This is unexpected, we thought that CMake would be the one avoiding the duplicate call once the first time it has been found, in theory the manual if(NOT ${package_name}_FOUND)
shouldn't be needed. We need to check this.
I am making the following find package call
find_package(pugixml CONFIG REQUIRED)
and since this PR I am seeing the following warning in the CMake output:This cause by two things. Firstly the new behaviour in the conan_provider.cmake file is causing find_package to be called twice:
Secondly the code in the generated pugixmlTargets.cmake file:
I am uncertain if this is an issue with the new behaviour in conan_provider.cmake or something to do with the recipe generating a warning when it shouldn't. Should there potentially be a check in the conan_provider.cmake file e.g.