-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
{2023.06}[foss/2023b] LAMMPS 29Aug2024 #725
base: 2023.06-software.eessi.io
Are you sure you want to change the base?
Conversation
Instance
|
Instance
|
Instance
|
bot: build repo:eessi.io-2023.06-software arch:aarch64/neoverse_v1 |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
bot: build repo:eessi.io-2023.06-software arch:aarch64/neoverse_v1 |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
@@ -301,7 +301,7 @@ 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' and ec.version in ('2Aug2023_update2',): | |||
if ec.name == 'LAMMPS' and ec.version in ('2Aug2023_update2', '29Aug2024'): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we will still needs this hook for 29Aug2024
it is probably not an issue but for later versions this might become a problem. When the version is not included, the build errors out with:
== 2024-09-19 07:39:28,463 build_log.py:171 ERROR EasyBuild crashed with an error (at easybuild/tools/build_log.py:111 in caller_info): Failed to process easyconfig /tmp/eb-3_9md2gk/eb-p410if82/files_pr21436/l/LAMMPS/LAMMPS-29Aug2024-foss-2023b-kokkos.eb: LAMMPS-specific hook triggered for non-LAMMPS easyconfig?! (at easybuild/framework/easyconfig/easyconfig.py:2094 in process_easyconfi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good to be strict here, because we do expect that for a future LAMMPS version this will not be required anymore?
Found it was the same with CP2K #630 (comment) |
It should however already apply that hook so I'm not understanding |
@laraPPr I see this:
|
easystacks/software.eessi.io/2023.06/eessi-2023.06-eb-4.9.3-2023b.yml
Outdated
Show resolved
Hide resolved
bot: build repo:eessi.io-2023.06-software arch:aarch64/neoverse_v1 |
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
Updates by the bot instance
|
bot: build repo:eessi.io-2023.06-software arch:aarch64/generic |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
New job on instance
|
No description provided.