Skip to content

Exclude unused hdf5 and netcdf c components from build#43

Open
piotrows wants to merge 1 commit into
ecmwf-ifs:mainfrom
piotrows:napz_lean_netchdf5
Open

Exclude unused hdf5 and netcdf c components from build#43
piotrows wants to merge 1 commit into
ecmwf-ifs:mainfrom
piotrows:napz_lean_netchdf5

Conversation

@piotrows

Copy link
Copy Markdown
Contributor

This PR aims at optimization of the CI process. This minor change exclude compilation options of Necdf-C and HDF5 that are ON by default but not needed by the ecland software stack

@piotrows piotrows requested review from awnawab, gabardu and mlange05 May 12, 2026 10:32
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 20, 2026
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 21, 2026
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 21, 2026
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 22, 2026
Copilot AI pushed a commit to piotrows/ecland that referenced this pull request May 22, 2026
…rization

Co-authored-by: piotrows <22048166+piotrows@users.noreply.github.com>
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 29, 2026
piotrows pushed a commit to piotrows/ecland that referenced this pull request May 31, 2026
mlange05 pushed a commit to mlange05/ecland that referenced this pull request Jun 2, 2026
mlange05 pushed a commit to mlange05/ecland that referenced this pull request Jun 2, 2026
mlange05 pushed a commit to mlange05/ecland that referenced this pull request Jun 2, 2026
mlange05 pushed a commit to mlange05/ecland that referenced this pull request Jun 9, 2026

@awnawab awnawab left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @piotrows 🙏 This should hopefully shave a useful amount of time from the CI builds.

@mlange05 mlange05 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.

Looks good to me. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants