Skip to content

{geo}[gfbf/2024a] split cf-units and cftime into their own easyconfig#24949

Closed
lexming wants to merge 1 commit intoeasybuilders:developfrom
lexming:20251221163658_new_pr_cf-units330
Closed

{geo}[gfbf/2024a] split cf-units and cftime into their own easyconfig#24949
lexming wants to merge 1 commit intoeasybuilders:developfrom
lexming:20251221163658_new_pr_cf-units330

Conversation

@lexming
Copy link
Contributor

@lexming lexming commented Dec 21, 2025

(created using eb --new-pr)

cf-units and cftime are not trivial python packages to install. Avoid duplication by separating them into their own easyconfig.

@lexming lexming added the new label Dec 21, 2025
@github-actions github-actions bot added 2024a issues & PRs related to 2024a common toolchains change labels Dec 21, 2025
@lexming
Copy link
Contributor Author

lexming commented Dec 21, 2025

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Collaborator

@lexming: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=24949 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_24949 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 9270

Test results coming soon (I hope)...

Details

- notification for comment with ID 3679017145 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (total: 7 mins 43 secs) (3 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.7, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.23
See https://gist.github.com/boegelbot/cf3bd9f42727fad9a3bf004a4a994a92 for a full test report.

@lexming lexming changed the title split cf-units and cftime into their own easyconfig {geo}[gfbf/2024a] split cf-units and cftime into their own easyconfig Dec 21, 2025
@lexming
Copy link
Contributor Author

lexming commented Dec 21, 2025

Actually, I need to add another cfunits module in 2024a instead of this one call cf-units. See #24953.
So there is no longer any need for this split as there is no duplication of code. Closing.

@lexming lexming closed this Dec 21, 2025
@lexming lexming deleted the 20251221163658_new_pr_cf-units330 branch December 21, 2025 17:44
@lexming lexming added this to the next release (5.2.0) milestone Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2024a issues & PRs related to 2024a common toolchains change new

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants