Skip to content

stop setting separate_build_dir to True in custom easyblock for Armadillo#3836

Merged
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250714172116_new_pr_armadillo
Jul 14, 2025
Merged

stop setting separate_build_dir to True in custom easyblock for Armadillo#3836
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250714172116_new_pr_armadillo

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Jul 14, 2025

(created using eb --new-pr)

Armadillo-14.0.3-foss-2024a.eb - uses CMakeMake easyblock, so does not need testing

@branfosj branfosj added this to the release after 5.1.1 milestone Jul 14, 2025
@branfosj
Copy link
Copy Markdown
Member Author

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS Armadillo-11.4.3-foss-2022b.eb
  • SUCCESS Armadillo-12.6.2-foss-2023a.eb
  • SUCCESS Armadillo-12.8.0-foss-2023b.eb
  • SUCCESS arpack-ng-3.8.0-foss-2021a.eb
  • SUCCESS arpack-ng-3.8.0-foss-2022a.eb
  • SUCCESS Armadillo-10.7.5-foss-2021a.eb
  • SUCCESS Armadillo-11.4.3-foss-2022a.eb

Build succeeded for 7 out of 7 (5 easyconfigs in total)
bear-pg0105u03a - Linux RHEL 8.10, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/branfosj/dc3bb6dec67ee176b23c117b5daafbd7 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 8bbf192 into easybuilders:develop Jul 14, 2025
17 checks passed
@branfosj branfosj deleted the 20250714172116_new_pr_armadillo branch July 15, 2025 06:20
@boegel boegel changed the title remove default separate_build_dir = True in aramdillo.py stop setting separate_build_dir to True in custom easyblock for Armadillo Jul 30, 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