Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't consider lib/release for impi >= 2021.11 #3375

Merged
merged 1 commit into from
Jul 3, 2024
Merged

Don't consider lib/release for impi >= 2021.11 #3375

merged 1 commit into from
Jul 3, 2024

Conversation

bartoldeman
Copy link
Contributor

lib/release is a symbolic link to lib, so we no longer need to include it in LD_LIBRARY_PATH and LIBRARY_PATH

@bartoldeman
Don't consider lib/release for impi >= 2021.11
a60c47d 
lib/release is a symbolic link to lib, so we no longer need to
include it in LD_LIBRARY_PATH and LIBRARY_PATH
@bartoldeman bartoldeman mentioned this pull request Jun 27, 2024
Merged
@bartoldeman bartoldeman added this to the release after 4.9.2 milestone Jun 28, 2024
@boegel boegel added update and removed bug fix labels Jul 3, 2024
@boegel
Copy link
Member

boegel commented Jul 3, 2024

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS impi-2021.10.0-intel-compilers-2023.2.1.eb
  • SUCCESS impi-2021.11.0-intel-compilers-2024.0.0.eb
  • SUCCESS impi-2021.7.1-intel-compilers-2022.2.1.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
node4015.donphan.os - Linux RHEL 8.8, x86_64, Intel(R) Xeon(R) Gold 6240 CPU @ 2.60GHz, 1 x NVIDIA NVIDIA A2, 545.23.08, Python 3.6.8
See https://gist.github.com/boegel/2d2ac1bf19fc9a3248dafc70062ebadf for a full test report.

boegel
boegel approved these changes Jul 3, 2024
View reviewed changes
Copy link
Member

@boegel boegel left a comment •
edited
Loading

Choose a reason for hiding this comment

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

lgtm

@boegel boegel merged commit 8f4ca9d into easybuilders:develop Jul 3, 2024
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boegel boegel boegel approved these changes

Assignees
No one assigned
Labels
update
Projects
None yet
Milestone
4.9.3
Development

Successfully merging this pull request may close these issues.
2 participants
@bartoldeman @boegel