Skip to content

Commit

Permalink
fix hook for LAMMPS to filter out ScaFaCoS and tbb dependencies on aa…
Browse files Browse the repository at this point in the history
…rch64/*
  • Loading branch information
boegel authored and truib committed Apr 5, 2024
1 parent d4fd6bc commit 404ded5
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions eb_hooks.py
Original file line number Diff line number Diff line change
Expand Up @@ -295,11 +295,15 @@ def parse_hook_lammps_remove_deps_for_CI_aarch64(ec, *args, **kwargs):
"""
Remove x86_64 specific dependencies for the CI to pass on aarch64
"""
if ec.name == 'LAMMPS':
if ec.version == '2Aug2023_update2':
if os.getenv('EESSI_CPU_FAMILY') == 'aarch64':
ec['dependencies'].remove(('ScaFaCoS', '1.0.4'))
ec['dependencies'].remove(('tbb', '2021.11.0'))
if ec.name == 'LAMMPS' and ec.version in ('2Aug2023_update2',):
if os.getenv('EESSI_CPU_FAMILY') == 'aarch64':
# ScaFaCoS and tbb are not compatible with aarch64/* CPU targets,
# so remove them as dependencies for LAMMPS (they're optional);
# see also https://github.com/easybuilders/easybuild-easyconfigs/pull/19164 +
# https://github.com/easybuilders/easybuild-easyconfigs/pull/19000;
# we need this hook because we check for missing installations for all CPU targets
# on an x86_64 VM in GitHub Actions (so condition based on ARCH in LAMMPS easyconfig is always true)
ec['dependencies'] = [dep for dep in ec['dependencies'] if dep[0] not in ('ScaFaCoS', 'tbb')]
else:
raise EasyBuildError("LAMMPS-specific hook triggered for non-LAMMPS easyconfig?!")

Expand Down

0 comments on commit 404ded5

Please sign in to comment.