Skip to content

Remove cmake-build, the hpc lane name, the cc/cxx/fc aliases and the build-action fields - #103

Merged
mcocdawc merged 4 commits into
mainfrom
remove-cmake-build
Oct 5, 2026
Merged

mcocdawc merged 4 commits into
mainfrom
remove-cmake-build

Conversation

@mcocdawc

@mcocdawc mcocdawc commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Removes what #97 deprecated. Closes #98.

  • actions/cmake-build: builds go through a job-script recipe and run-job-script. The manifest tutorial shows eckit's recipe and ecflow's cmake_args instead of their build actions.
  • cmake-build.sh.j2, the alias of cmake-atos.sh.j2.
  • execution = "hpc", now a schema error; hpc-atos is the only HPC lane.
  • The -hpc fallbacks: the rebuild-dispatch retry, the old names among the build workflows, and the deletion of old -hpc files on regenerate.
  • The cc/cxx/fc aliases: a recipe reads c_compiler_binary, cxx_compiler_binary and fortran_compiler_binary, which default to the leg's c-compiler, cxx-compiler and fortran-compiler. Without compiler-inputs, a job title names cxx-compiler-binary, so a leg moving from cxx to cxx-compiler-binary keeps its title.
  • The samples point to docs/reference/hpc.rst rather than the removed docs/howto/hpc.rst.
  • The build-action fields action, forwarded-inputs, forwarded-deps-outputs and ctest, with the step that called a repo's action, the generated ctest step and _resolved.ctest/ctest-args. A triggered kind now needs a job-script in either lane. The generated "Set up Python" step reads matrix.python-version; the decode step only exported it for a forwarded input.

HPC renders stay byte-identical, so HPC_TEMPLATE_VERSION stays at 2. Docs speak of shared Jinja templates rather than a CMake template.

Draft until these are merged, since the generator validates every sibling's manifest and recipes:

Checked: with those branches as siblings, all five validate against this branch, render every leg byte-identically to main, and regenerate cross-repo-trigger*.yml without drift. metkit#293 has to switch to the *-compiler-binary names before it merges.

🤖 Generated with Claude Code

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

Documentation preview
https://sites.ecmwf.int/docs/dev-section/ci-infrastructure/pull-requests/PR-103

@mcocdawc
mcocdawc force-pushed the remove-cmake-build branch 2 times, most recently from 9c38bbf to a3a6656 Compare October 2, 2026 10:04
@mcocdawc mcocdawc changed the title Remove actions/cmake-build Remove cmake-build (action and template alias) and the hpc lane name Oct 2, 2026
mcocdawc and others added 2 commits October 5, 2026 11:34
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mcocdawc
mcocdawc force-pushed the remove-cmake-build branch from a3a6656 to 74d66d9 Compare October 5, 2026 09:39
@mcocdawc mcocdawc changed the title Remove cmake-build (action and template alias) and the hpc lane name Remove cmake-build, the hpc lane name and the cc/cxx/fc aliases Oct 5, 2026
…m the manifest

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mcocdawc
mcocdawc marked this pull request as ready for review October 5, 2026 14:14
@mcocdawc
mcocdawc merged commit c6a3667 into main Oct 5, 2026
56 checks passed
@mcocdawc
mcocdawc deleted the remove-cmake-build branch October 5, 2026 14:26
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.

Remove cmake-build: template alias and action

1 participant