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

{devel}[GCC/9.3.0,iccifort/2020.1.217] Boost.Serial v1.72.0 #11145

Conversation

mboisson
Copy link
Contributor

(created using eb --new-pr)

@mboisson
Copy link
Contributor Author

mboisson commented Aug 19, 2020

@SimonPinches, this is a change for Boost"-no_mpi" introduced previously. After the EB call today, we are introducing Boost.Serial, instead of having a version suffix.

#11112

@zao
Copy link
Contributor

zao commented Aug 21, 2020

Test report by @zao
FAILED
Build succeeded for 0 out of 2 (2 easyconfigs in this PR)
freja - Linux ubuntu 18.04, x86_64, Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz, Python 2.7.17
See https://gist.github.com/a4a0fa4bb1770df09b61d5889789a92f for a full test report.

@zao
Copy link
Contributor

zao commented Aug 21, 2020

Sanity check seems hosed as per my test report, on my system files are decorated like:

/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread.a
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread-mt-x64.a
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread-mt-x64.so
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread-mt-x64.so.1
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread-mt-x64.so.1.72
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread-mt-x64.so.1.72.0
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread.so
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread.so.1
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread.so.1.72
/eb/software/Boost.Serial/1.72.0-GCC-9.3.0/lib/libboost_thread.so.1.72.0

@mboisson
Copy link
Contributor Author

Mmm, that seems to be an EasyBlock problem ? I have no custom sanity_check paths in the EC.

@lexming lexming changed the base branch from develop to dual_serial_mpi September 2, 2020 09:47
@lexming
Copy link
Contributor

lexming commented Sep 2, 2020

Changed target to easybuilders:dual_serial_mpi

@lexming
Copy link
Contributor

lexming commented Sep 2, 2020

Test report by @lexming
FAILED
Build succeeded for 0 out of 2 (2 easyconfigs in this PR)
node368.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/021b1e0b162be6e3a61aa2cd1142f481 for a full test report.

@lexming
Copy link
Contributor

lexming commented Sep 2, 2020

Test report by @lexming
FAILED
Build succeeded for 0 out of 2 (2 easyconfigs in this PR)
node101.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/6b12c4f3b6f9ad3ee3debe9c3b8ea609 for a full test report.

@SimonPinches
Copy link
Contributor

@mboisson thanks for the heads-up regarding the change of suffix name. I have to say I'm sorry to see the loss of a GCCcore version since this means I'll now need to build at at least two versions of all the libraries that depend upon it :-( This may even lead to conflicts in some of our mixed compilation workflows but I'll try to work around this.

@lexming
Copy link
Contributor

lexming commented Sep 8, 2020

@SimonPinches going down to GCCcore level for serial builds is indeed a good point to discuss. I'll add it for the conf call.

@mboisson
Copy link
Contributor Author

mboisson commented Sep 8, 2020

@lexming, the reason even Compute Canada did not do that is that we will always want to have a Boost with GCC and one with Intel. Bringing it down to GCCcore means that users run the risk of using GCC-compiled libraries with Intel, or that we need to forgo optimizations brought forward by the Intel compiler (which can be useful for algorithms in Boost).

@lexming
Copy link
Contributor

lexming commented Sep 9, 2020

Fixes for the failed tests can be found in easybuilders/easybuild-easyblocks#2167
easybuilders/easybuild-easyblocks#2231

@lexming
Copy link
Contributor

lexming commented Sep 9, 2020

Test report by @lexming
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node381.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/c1b1130f736fe713e49c91f8a71d5b01 for a full test report.

@lexming
Copy link
Contributor

lexming commented Sep 10, 2020

Test report by @lexming
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node116.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/e7063afd680d43861abfc406ddf9349c for a full test report.

@migueldiascosta
Copy link
Member

Test report by @migueldiascosta
using easyblocks from easybuilders/easybuild-easyblocks#2167
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
sms - Linux centos linux 7.6.1810, x86_64, AMD EPYC 7601 32-Core Processor (zen), Python 2.7.5
See https://gist.github.com/c59fb3f979395147e1e31c229508f21b for a full test report.

@lexming
Copy link
Contributor

lexming commented Sep 16, 2020

Test report by @lexming
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node377.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/c651b636cb43ab282caf01919cdee8b9 for a full test report.

@boegel boegel modified the milestones: 4.3.1 (next release), 4.x Oct 26, 2020
@lexming
Copy link
Contributor

lexming commented Nov 12, 2020

Test report by @lexming
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node302.hydra.os - Linux centos linux 7.7.1908, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/7340253e680975a8341931604cc34480 for a full test report.

Copy link
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

LGTM

@lexming
Copy link
Contributor

lexming commented Nov 18, 2020

Going in, thanks @mboisson !

@lexming lexming merged commit 872cbf7 into easybuilders:dual_serial_mpi Nov 18, 2020
@boegel boegel changed the title {devel}[GCC/9.3.0] Boost.Serial v1.72.0 {devel}[GCC/9.3.0,iccifort/2020.1.217] Boost.Serial v1.72.0 Nov 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants