easybuilders / easybuild-easyconfigs

A collection of easyconfig files that describe which software to build using which build options with EasyBuild.
https://easybuild.io
GNU General Public License v2.0
356 stars 685 forks source link

Add internal CUDA header patch for PSM2 #20804

Closed bartoldeman closed 1 week ago

bartoldeman commented 2 weeks ago

(created using eb --new-pr)

bartoldeman commented 2 weeks ago

This eliminates the CUDA builddependency by adding a minimal internal cuda.h header, exactly like we already did for Open MPI.

See https://github.com/easybuilders/easybuild-easyconfigs/pull/20794 and https://github.com/easybuilders/easybuild-framework/issues/4560 for more info.

bartoldeman commented 2 weeks ago

@boegelbot please test @ jsc-zen3

bartoldeman commented 2 weeks ago

@boegelbot please test @ generoso

boegelbot commented 2 weeks ago

@bartoldeman: Request for testing this PR well received on login1

PR test command 'EB_PR=20804 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --job-name test_PR_20804 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

Test results coming soon (I hope)...

*- notification for comment with ID 2163114855 processed* *Message to humans: this is just bookkeeping information for me, it is of no use to you (unless you think I have a bug, which I don't).*
boegelbot commented 2 weeks ago

Test report by @boegelbot SUCCESS Build succeeded for 7 out of 7 (7 easyconfigs in total) cns1 - Linux Rocky Linux 8.9, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8 See https://gist.github.com/boegelbot/37e444a075c8e8ae93db7ac3a0e4b05e for a full test report.

jfgrimm commented 2 weeks ago

Test report by @jfgrimm SUCCESS Build succeeded for 13 out of 13 (7 easyconfigs in total) node026.viking2.yor.alces.network - Linux Rocky Linux 8.9, x86_64, AMD EPYC 7643 48-Core Processor, Python 3.6.8 See https://gist.github.com/jfgrimm/945a6b8d5f4f7d0c95c07533f91317fc for a full test report.

jfgrimm commented 1 week ago

Going in, thanks @bartoldeman!