Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Oct 6, 2024

(created using eb --new-pr)

fix for failure when installing SuiteSparse-5.13.0-foss-2022a-METIS-5.1.0.eb:

CMake Error at cmake_install.cmake:57 (file):
  file INSTALL cannot copy file
  "/tmp/vsc40023/easybuild_build/SuiteSparse/5.13.0/foss-2022a-METIS-5.1.0/SuiteSparse-5.13.0/GraphBLAS/build/libgraphblas.so.7.2.0"
  to "/usr/local/lib64/libgraphblas.so.7.2.0": Permission denied.

This issue occurs because CMAKE_INSTALL_PREFIX is set by CMAKE_OPTIONS, but it's not getting pass to the make install command correctly...

I'm a bit confused as to why this didn't pop up in the test report for #3004, but maybe the subsequent changes in #3031 have something to do with it...

@boegel boegel added the bug fix label Oct 6, 2024
@boegel boegel added this to the release after 4.9.4 milestone Oct 6, 2024
@boegel boegel changed the title fix setting of CMAKE_OPTIONS for SuiteSparse versions between 5.1.2 and 6.0.0 fix setting of CMAKE_OPTIONS for SuiteSparse versions between 5.1.2 and 6.0.0 Oct 6, 2024
@boegel
Copy link
Member Author

boegel commented Oct 6, 2024

@boegelbot please test @ generoso
EB_ARGS="SuiteSparse-5.7.1-foss-2020a-METIS-5.1.0.eb SuiteSparse-5.8.1-foss-2020b-METIS-5.1.0.eb SuiteSparse-5.10.1-foss-2021a-METIS-5.1.0.eb SuiteSparse-5.10.1-foss-2021b-METIS-5.1.0.eb SuiteSparse-5.10.1-intel-2021a-METIS-5.1.0.eb SuiteSparse-5.10.1-intel-2021b-METIS-5.1.0.eb SuiteSparse-5.13.0-foss-2022a-METIS-5.1.0.eb SuiteSparse-5.13.0-foss-2022b-METIS-5.1.0.eb --installpath /tmp/$USER/pr3471"
CORE_CNT=16

@boegelbot
Copy link

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

PR test command 'EB_PR=3471 EB_ARGS="SuiteSparse-5.7.1-foss-2020a-METIS-5.1.0.eb SuiteSparse-5.8.1-foss-2020b-METIS-5.1.0.eb SuiteSparse-5.10.1-foss-2021a-METIS-5.1.0.eb SuiteSparse-5.10.1-foss-2021b-METIS-5.1.0.eb SuiteSparse-5.10.1-intel-2021a-METIS-5.1.0.eb SuiteSparse-5.10.1-intel-2021b-METIS-5.1.0.eb SuiteSparse-5.13.0-foss-2022a-METIS-5.1.0.eb SuiteSparse-5.13.0-foss-2022b-METIS-5.1.0.eb --installpath /tmp/$USER/pr3471" EB_CONTAINER= EB_REPO=easybuild-easyblocks /opt/software/slurm/bin/sbatch --job-name test_PR_3471 --ntasks="16" ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 14408

Test results coming soon (I hope)...

Details

- notification for comment with ID 2395530592 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
Copy link

Test report by @boegelbot

Overview of tested easyconfigs (in order)

  • SUCCESS SuiteSparse-5.7.1-foss-2020a-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.8.1-foss-2020b-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.10.1-foss-2021a-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.10.1-foss-2021b-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.10.1-intel-2021a-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.10.1-intel-2021b-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.13.0-foss-2022a-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.13.0-foss-2022b-METIS-5.1.0.eb

Build succeeded for 8 out of 8 (8 easyconfigs in total)
cnx1 - 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/2890a99dfa378983b6775dda4cb409d2 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 6, 2024

Merging, thanks @boegel !

@lexming lexming merged commit a1add70 into easybuilders:develop Nov 6, 2024
@boegel boegel deleted the 20241006200121_new_pr_suitesparse branch November 6, 2024 08:11
@boegel boegel modified the milestones: release after 4.9.4, 5.0.0 Mar 18, 2025
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.

3 participants