Skip to content

stop setting separate_build_dir to True in custom easyblock for LAMMPS#3845

Merged
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250715145059_new_pr_lammps
Jul 16, 2025
Merged

stop setting separate_build_dir to True in custom easyblock for LAMMPS#3845
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:20250715145059_new_pr_lammps

Conversation

@branfosj
Copy link
Copy Markdown
Member

(created using eb --new-pr)

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

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS LAMMPS-23Jun2022-foss-2021a-kokkos.eb
  • SUCCESS LAMMPS-23Jun2022-foss-2021b-kokkos.eb
  • SUCCESS LAMMPS-29Aug2024-foss-2023b-kokkos.eb
  • SUCCESS LAMMPS-29Aug2024_update2-foss-2023a-kokkos.eb
  • SUCCESS LAMMPS-29Aug2024_update2-foss-2024a-kokkos.eb
  • SUCCESS LAMMPS-2Aug2023_update2-foss-2023a-kokkos.eb
  • SUCCESS archspec-0.1.4-GCCcore-11.3.0.eb
  • SUCCESS Voro++-0.4.6-GCCcore-11.3.0.eb
  • SUCCESS ScaFaCoS-1.0.4-foss-2022a.eb
  • SUCCESS xxd-8.2.4220-GCCcore-11.3.0.eb
  • SUCCESS PLUMED-2.8.1-foss-2022a.eb
  • SUCCESS LAMMPS-23Jun2022-foss-2022a-kokkos.eb

Build succeeded for 12 out of 12 (7 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/f51b87bac1c23c9d1234ab154c7055c6 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 3293743 into easybuilders:develop Jul 16, 2025
17 checks passed
@branfosj branfosj deleted the 20250715145059_new_pr_lammps branch July 16, 2025 15:12
@boegel boegel changed the title remove default separate_build_dir = True in lammps.py stop setting separate_build_dir to True in custom easyblock for LAMMPS Jul 31, 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