Skip to content

update OpenMPI easyblock to fix sanity check for Clang-based compilers#2774

Merged
boegel merged 1 commit intoeasybuilders:developfrom
zemu-unile:develop
Aug 8, 2023
Merged

update OpenMPI easyblock to fix sanity check for Clang-based compilers#2774
boegel merged 1 commit intoeasybuilders:developfrom
zemu-unile:develop

Conversation

@zemu-unile
Copy link
Copy Markdown
Contributor

@zemu-unile zemu-unile commented Aug 12, 2022

Clang compilers always have the same output when using --version. There is no different information when using C++ or Fortran compilers.

e.g.

$ clang++ --version
clang version 11.0.1
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /software/all/Clang/11.0.1-GCCcore-10.2.0/bin

@zemu-unile zemu-unile mentioned this pull request Aug 12, 2022
Closed
@boegel boegel added this to the release after 4.7.3 milestone Jul 5, 2023
@boegel boegel added the bug fix label Jul 5, 2023
@boegel boegel changed the title fix sanity check for clang based compilers update OpenMPI easyblock to fix sanity check for Clang-based compilers Aug 8, 2023
@boegel
Copy link
Copy Markdown
Member

boegel commented Aug 8, 2023

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS OpenMPI-4.1.1-intel-compilers-2021.4.0.eb
  • SUCCESS OpenMPI-3.1.4-GCC-8.3.0.eb
  • SUCCESS PMIx-4.2.4-GCCcore-12.3.0.eb
  • SUCCESS UCC-1.2.0-GCCcore-12.3.0.eb
  • SUCCESS OpenMPI-4.1.5-GCC-12.3.0.eb

Build succeeded for 5 out of 5 (3 easyconfigs in total)
node3115.skitty.os - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/boegel/48ae9fbce77acee50eab5d623b827466 for a full test report.

boegel
boegel approved these changes Aug 8, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

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 3f95af4 into easybuilders:develop Aug 8, 2023
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

bug fix

Projects

None yet

Milestone

4.8.1

Development

Successfully merging this pull request may close these issues.

2 participants

@zemu-unile @boegel