Skip to content

Conversation

robert-mijakovic
Copy link

(created using eb --new-pr)

@boegel boegel changed the title Adds nvompi toolchain. The idea is to replicate what gompi/foss/GCC(core) do add nvompi toolchain definition (NVHPC + OpenMPI) Mar 1, 2022
Copy link
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 added this to the next release (4.5.4?) milestone Mar 1, 2022
EasyBuild support for nvompi compiler toolchain (NVHPC + Open MPI).

:author: Maxime Boissonneault (Universite Laval, Calcul Quebec, Compute Canada)
:author: Bart Oldeman (McGill University, Calcul Quebec, Compute Canada)
Copy link
Member

Choose a reason for hiding this comment

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

@robert-mijakovic Please change author to yourself?

@ocaisa
Copy link
Member

ocaisa commented Apr 12, 2022

@robert-mijakovic This looks good to merge, just need to address the comment

@robert-mijakovic
Copy link
Author

@robert-mijakovic This looks good to merge, just need to address the comment

Done. Thanks for reminding me.

Copy link
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

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.

3 participants