Skip to content

Fix CMake configuration - #114

Merged
cwsmith merged 1 commit into
SCOREC:mainfrom
Fuad-HH:patch-1
Oct 16, 2026
Merged

cwsmith merged 1 commit into
SCOREC:mainfrom
Fuad-HH:patch-1

Conversation

@Fuad-HH

@Fuad-HH Fuad-HH commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

I was trying to install MeshFields using spack and got this error:

     78    -- Configuring done (19.1s)
  >> 79    CMake Error at /global/homes/f/fuad/.spack/opt/spack/linux-zen3/meshfields-main-f5qmikve52v4ntypoh7ooohxnuit4z6t/lib64/cmake/meshfields/meshfields-targets.cmake:61 (set_target_properties):
     80      The link interface of target "meshfields::meshfields" contains:
     81
     82        Cabana::Core
     83
     84      but the target was not found.  Possible reasons include:
     85

Fixes:

  • The conditional for finding the Cabana dependency was corrected to use the CMake variable substitution @MeshFields_USE_Cabana@ instead of the non-expanded MeshFields_USE_Cabana, ensuring the check works as intended during configuration.
  • The include("${CMAKE_CURRENT_LIST_DIR}/meshfields-targets.cmake") and check_required_components(meshfields) statements were moved to after the dependency checks for better ordering and to ensure all dependencies are found before targets are included.

@cwsmith cwsmith left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

@cwsmith
cwsmith merged commit f1a8ca7 into SCOREC:main Oct 16, 2026
6 checks passed
@Fuad-HH
Fuad-HH deleted the patch-1 branch October 16, 2026 03:39

This branch was successfully deployed

1 active deployment
asics — 51e43c06 Deployed Oct 16, 2026 by Fuad-HH via build_and_test #21
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