Skip to content

[release/2.2]: Gaea C7: fix cairo problem and add loading INTEL befor… - #2206

Closed
RatkoVasic-NOAA wants to merge 1 commit into
JCSDA:developfrom
RatkoVasic-NOAA:gaea-c7-cairo
Closed

RatkoVasic-NOAA wants to merge 1 commit into
JCSDA:developfrom
RatkoVasic-NOAA:gaea-c7-cairo

Conversation

@RatkoVasic-NOAA

Copy link
Copy Markdown
Collaborator

…e MPI. (#2205)

(cherry picked from commit 9b1c885)

Use of generative AI

Please refer to our temporary spack-stack AI policy for further information.

  • I confirm that this contribution was submitted by a real person. If AI tools or agents were used in its preparation, I have personally reviewed and take responsibility for all submitted content, code, and proposed changes. This contribution is not autonomously generated by AI.

Description

Cherry pick from release/2.2

Dependencies

Issues addressed

Closes #2204

Applications affected

Systems affected

Gaea C7

Testing

  • CI: Note whether the automatic tests (GitHub actions tests that run automatically for every commit) pass or not
    • GitHub actions CI tests pass
    • GitHub actions CI tests do not pass (provide explanation)
    • GitHub actions CI tests skipped (provide explanation if necessary)
  • New tests added: List and describe any new tests added to GitHub actions
    • ...
  • Additional testing: Add information on any additional tests conducted
    • ...

Checklist

  • This PR addresses one issue/problem/enhancement or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.
  • All necessary updates to the documentation (spack-stack wiki) will be made when this PR is merged

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The Cairo override drops shared version and variant requirements, risking modulefile collisions, while the oneAPI changes leave the requested compiler-before-MPI module ordering unimplemented.

2 open issues
What changed in this PR

This cherry-pick updates Gaea C7’s site configuration to address Cairo linking and Intel module prerequisites.

Changes:

  • Requires Cairo’s X11 support to address the reported missing symbol.
  • Adds matching oneAPI compiler modules before MKL in both Intel configurations.
File Description
configs/​sites/​tier1/​gaea-c7/​packages.yaml Enables Cairo’s X11 variant.
configs/​sites/​tier1/​gaea-c7/​packages_oneapi-2026.0.0.yaml Loads oneAPI 2026.0.0 before MKL.
configs/​sites/​tier1/​gaea-c7/​packages_oneapi-2025.3.1.yaml Loads oneAPI 2025.3.1 before MKL.

Review effort: Balanced

- '@1.2.13'
cairo:
# /usr/lib64/libgdk-x11-2.0.so: undefined reference to `cairo_xlib_surface_set_size'
require: +X
- spec: intel-oneapi-mkl@2026.0.0
prefix: /sw/gaea-c7/spack/envs/env_26.06/opt/intel-oneapi-compilers-2026.0.0/intel-oneapi-mkl-2026.0.0-ferm7nss4cxphhlu5rbcxzwblz7xyg53
modules:
- oneapi/2026.0.0
@climbfuji

Copy link
Copy Markdown
Collaborator

We don't need this PR, release/2.2 will be merged back into develop shortly.

@climbfuji climbfuji closed this Nov 3, 2026
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.

Fix cairo problem, and add loading INTEL before MPI for Gaea-C7

3 participants