Skip to content

fix use of self.cfg.dependencies() in OpenFOAM easyblock#3814

Merged
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250630155221_new_pr_openfoam
Jul 1, 2025
Merged

fix use of self.cfg.dependencies() in OpenFOAM easyblock#3814
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250630155221_new_pr_openfoam

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Jun 30, 2025

(created using eb --new-pr)

This code path is only used for OpenFOAM-Extend (or if someone builds OpenFOAM < 2)

@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Jul 1, 2025

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS Mesquite-2.3.0-GCCcore-12.3.0.eb
  • SUCCESS ParMGridGen-1.0-gompi-2023a.eb
  • SUCCESS Python-2.7.18-GCCcore-12.3.0.eb
  • SUCCESS OpenFOAM-Extend-4.1-20210705-foss-2023a-Python-2.7.18.eb

Build succeeded for 4 out of 4 (1 easyconfigs in total)
bear-pg0210u03a - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8480CL (sapphirerapids), Python 3.6.8
See https://gist.github.com/branfosj/8e628e901aec243da36c4dbb09f64fde for a full test report.

Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

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

lgtm

@Micket Micket merged commit f2d589d into easybuilders:develop Jul 1, 2025
17 checks passed
@Micket Micket added this to the next release (5.1.1?) milestone Jul 1, 2025
@branfosj branfosj deleted the 20250630155221_new_pr_openfoam branch July 1, 2025 12:16
@boegel boegel changed the title fix use of self.cfg.dependencies() in OpenFOAM fix use of self.cfg.dependencies() in OpenFOAM easyblock Jul 2, 2025
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.

2 participants