Force CAM to build yaml-cpp (vs searching for it first when building TUV-x)#1538
Open
briandobbins wants to merge 2 commits intoESCOMP:cam_developmentfrom
Open
Force CAM to build yaml-cpp (vs searching for it first when building TUV-x)#1538briandobbins wants to merge 2 commits intoESCOMP:cam_developmentfrom
briandobbins wants to merge 2 commits intoESCOMP:cam_developmentfrom
Conversation
… FindPackage. This addresses a build issue when Conda environments are active and find it in their search path.
fvitt
reviewed
Apr 16, 2026
| cmake_args += "-DCMAKE_DISABLE_FIND_PACKAGE_yaml-cpp=1 " | ||
| if (case.get_value("MACH") == "izumi") : | ||
| cmake_args += f"-DCMAKE_PREFIX_PATH={arg_dict['NETCDF_PATH']} " | ||
| cmake_args += f"-DCMAKE_IGNORE_PATH={os.environ.get('PYTHONHOME')} " |
Collaborator
There was a problem hiding this comment.
With the CMAKE_DISABLE arg at line 221, can the CMAKE_IGNORE_PATH for izumi builds at line 224 be removed? The conda environment issue sounds like an issue I had on izumi, so the IGNORE_PATH arg was added.
Collaborator
Author
There was a problem hiding this comment.
I'm an idiot - I didn't even think of Izumi, and that's a good question. Let me try things out and report back. Got a few other things I need to focus on tonight, so it may be tomorrow.
(I'm embarrassed to say I didn't even check out the other -DCMAKE args!)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
There's an issue with CAM / CESM cases and TUV-x when users have Conda environments active, since it finds a yaml-cpp library there, but it often isn't compatible. This PR introduces a one-line change to force CAM to use its own copy so we have consistent behavior regardless of someone's conda environment.
Fixes #1537