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.

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

Add easyconfigs for system GCC/OpenMPI/iccifort/impi with binutils 2.29 #4136

Merged

Conversation

ocaisa
Copy link
Member

@ocaisa ocaisa commented Feb 15, 2017

(created using eb --new-pr)
Requires easybuilders/easybuild-easyblocks#1106


name = 'gompi'
version = 'system'
versionsuffix = '-2.27' # binutils version
Copy link
Member

Choose a reason for hiding this comment

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

either use system-2.27 to match GCC version, or use system as version and make sure the right GCC is loaded, otherwise you end up with a double -2.27 in OpenMPI...

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll drop back to system for OpenMPI which is actually more accurate anyway. The toolchain used there will dictate the GCC version. For gompi, I think the versionsuffix is a valid inclusion.

Copy link
Member Author

@ocaisa ocaisa Sep 25, 2017

Choose a reason for hiding this comment

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

system-2.27 would require a lot more logic in the easyblock since the system version is a trigger there

@ocaisa ocaisa changed the title {compiler,mpi}[GCC/system-2.27,dummy/] OpenMPI vsystem, GCC vsystem Add system easyconfigs for system GCC/OpenMPI/iccifort/impi with binutils 2.29 Sep 25, 2017
@ocaisa ocaisa changed the title Add system easyconfigs for system GCC/OpenMPI/iccifort/impi with binutils 2.29 Add easyconfigs for system GCC/OpenMPI/iccifort/impi with binutils 2.29 Sep 25, 2017
@ocaisa
Copy link
Member Author

ocaisa commented Sep 25, 2017

@boegel Made the correction to OpenMPI and also did an overall update to use binutils/2.29

I also disabled any arch optimisations for the deps of binutils since there is the possibility that the system bintuils is outdated.

@boegel boegel modified the milestones: 3.5.0, 3.4.1 Sep 27, 2017
Alan O'Cais added 2 commits September 27, 2017 17:37
the system compiler and we need to build the dependencies of binutils
ocaisa added 5 commits September 27, 2017 18:20
as a build dep for easyconfigs with a `GCCcore` toolchain
the system compiler and we need to build the dependencies of binutils
the system compiler and we need to build the dependencies of binutils
@boegel
Copy link
Member

boegel commented Sep 30, 2017

Test report by @boegel
SUCCESS
Build succeeded for 10 out of 10 (1 easyconfigs in this PR)
pichu03 - Linux debian 9.1, ARM Cortex-A53, Python 2.7.13
See https://gist.github.com/11a1febea72f019d0b9b9570f6c75fcc for a full test report.

@boegel
Copy link
Member

boegel commented Sep 30, 2017

Test report by @boegel
SUCCESS
Build succeeded for 15 out of 15 (15 easyconfigs in this PR)
node2004.delcatty.os - Linux centos linux 7.3.1611, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/206f24728498e8bef98d5034815fb087 for a full test report.

@boegel
Copy link
Member

boegel commented Oct 11, 2017

Test report by @boegel
SUCCESS
Build succeeded for 15 out of 15 (15 easyconfigs in this PR)
node2127.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/23d391d087792420bd9483b104d5b884 for a full test report.

add dummy icc/ifort/mpiicc commands to allow tests for {icc,ifort,impi}-system.eb easyconfigs to pass
@boegel
Copy link
Member

boegel commented Oct 11, 2017

test report was submitted on top of most recent easybuilders/easybuild-easyblocks#1106 (with ocaisa/easybuild-easyblocks#12 included)

@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@boegel
Copy link
Member

boegel commented Oct 11, 2017

Test report by @boegel
SUCCESS
Build succeeded for 15 out of 15 (15 easyconfigs in this PR)
node2127.delcatty.os - Linux centos linux 7.4.1708, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/aa5aa3e0112a55bf857e1921c8c9eef2 for a full test report.

@boegel
Copy link
Member

boegel commented Oct 11, 2017

retested on top of easybuilders/easybuild-easyblocks#1261

@easybuilders easybuilders deleted a comment from boegelbot Oct 11, 2017
@boegel
Copy link
Member

boegel commented Oct 11, 2017

Going in, thanks @ocaisa!

@boegel boegel merged commit 7188d33 into easybuilders:develop Oct 11, 2017
@ocaisa ocaisa deleted the 20170215092701_new_pr_OpenMPIsystem branch October 11, 2017 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants