Skip to content

handle problems copying symlink that points to CUDA folder that is not created for non CUDA builds of SuiteSparse #2790

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.

Sign up for GitHub

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

Jump to bottom
Merged
branfosj merged 2 commits into from
Sep 22, 2022
Merged

handle problems copying symlink that points to CUDA folder that is not created for non CUDA builds of SuiteSparse #2790

branfosj merged 2 commits into from
Sep 22, 2022

Conversation

@satishskamath
Copy link
Contributor

@satishskamath satishskamath commented Sep 21, 2022

(created using eb --new-pr)

@satishskamath satishskamath mentioned this pull request Sep 21, 2022
Closed
3 tasks
@branfosj branfosj changed the title symlink points to CUDA folder not created for non GPU nodes. shutil throws an error in this handle problems copying symlink that points to CUDA folder that is not created for non CUDA builds of SuiteSparse Sep 21, 2022
@branfosj
Copy link
Member

branfosj commented Sep 21, 2022

Test report by @branfosj

Overview of tested easyconfigs (in order)

  • SUCCESS SuiteSparse-5.12.0-foss-2022a-METIS-5.1.0.eb
  • SUCCESS SuiteSparse-5.13.0-foss-2022a-METIS-5.1.0.eb
  • SUCCESS METIS-5.1.0-GCCcore-10.2.0.eb
  • SUCCESS MPFR-4.1.0-GCCcore-10.2.0.eb
  • SUCCESS SuiteSparse-5.8.1-foss-2020b-METIS-5.1.0.eb

Build succeeded for 5 out of 5 (3 easyconfigs in total)
bear-pg0105u36b.bear.cluster - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/dc3e85ec79b0a1f53748207f819e6dcf for a full test report.

@branfosj
Copy link
Member

branfosj commented Sep 22, 2022

SuiteSparse-5.12.0-foss-2022a-METIS-5.1.0.eb is based off the 2022.05 version in easybuilders/easybuild-easyconfigs#15574 - i.e. where we were seeing an issue.

@branfosj branfosj added this to the next release (4.6.2?) milestone Sep 22, 2022
branfosj
branfosj approved these changes Sep 22, 2022
View reviewed changes
Copy link
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj
Copy link
Member

branfosj commented Sep 22, 2022

Going in, thanks @satishskamath!

@branfosj branfosj merged commit cef9261 into easybuilders:develop Sep 22, 2022
@satishskamath satishskamath deleted the 20220921175343_new_pr_suitesparse branch October 11, 2022 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@branfosj branfosj branfosj approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.6.2

Development

Successfully merging this pull request may close these issues.

2 participants

@satishskamath @branfosj