Standardize package found logic for tpls

[bartgol]

Motivation

As shown in the failed CI runs for eamxx-v1 workflow in E3SM-Project/E3SM#8258, there is an issue with how we decide whether to fetch and install yaml-cpp. Namely, if the pkg is found, and only provides the (nicely) scoped yaml-cpp::yaml-cpp target, the if condition that establish whether to call FetchContent is wrong. This PR changes it to be more robust, for all the three packages: ensure that _FOUND is always set, and use that to decide whether to use FetchContent.

Side fix: if the yaml-cpp pkg is found but does not define the scoped tgt, we create the scoped one ourselves. But we were not propagating this to the installed EkatConfig.cmake, so downstream tgt would not know what yaml-cpp::yaml-cpp was (again, only for the case where a pre-installed yaml-cpp did NOT define the scoped tgt). To fix this, I added the same logic to our EkatConfig.cmake.in file, so that the scoped target is guaranteed to exist for downstream customers.

E3SM Stakeholder Feedback

Testing

[Copilot]

Pull request overview

This PR fixes how EKAT decides whether to fetch/build certain third-party
libraries (TPLs) by standardizing the decision logic on <pkg>_FOUND, and it
ensures yaml-cpp::yaml-cpp is available to downstream consumers via EKAT’s
installed CMake config.

Changes:

• Switch Catch2/yaml-cpp/spdlog FetchContent decisions to use <pkg>_FOUND.
• Ensure <pkg>_FOUND is set when EKAT_SKIP_FIND_<PKG> is enabled.
• Add downstream yaml-cpp::yaml-cpp alias creation in EkatConfig.cmake.in.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File	Description
src/testing-support/CMakeLists.txt	Uses Catch2_FOUND to decide whether to FetchContent Catch2.
src/parser/CMakeLists.txt	Uses yaml-cpp_FOUND to decide whether to FetchContent yaml-cpp.
src/logging/CMakeLists.txt	Uses spdlog_FOUND to decide whether to FetchContent spdlog.
cmake/EkatConfig.cmake.in	Ensures yaml-cpp::yaml-cpp exists for downstream consumers.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[bartgol]

The last version should be extra-extra safe:

1. check if a target (scoped/unscoped) exists. If so, use that
2. if no tgt exists, use find_package
3. if either of the above succeeds, ensure scoped tgt is defined. If not, create an alias scoped tgt (pointing to the unscoped one)
4. if tgt or pkg was not found, proceed to use FetchContent.
5. in EkatConfig.cmake.in, replicate similar logic, to ensure we honor any existing tgt in the host project.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[bartgol]

I built e3sm with this version of ekat on my workstation, and it builds fine.

[jeff-cohere]

I have one question about an empty if block. But the rest looks good to me.