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.

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

Unpin Intel compiler #436

Merged
merged 8 commits into from
Dec 15, 2023
Merged

Unpin Intel compiler #436

merged 8 commits into from
Dec 15, 2023

Conversation

billschereriii
Copy link
Contributor

@billschereriii billschereriii commented Dec 14, 2023 •
edited
Loading

Apply a proper fix to enable use of the Intel OneAPI compiler.

@billschereriii billschereriii mentioned this pull request Dec 14, 2023
Closed
@billschereriii
Copy link
Contributor Author

The underlying issue here is that the Intel Fortran compiler, ifx, is on release 2024.0, but the C/C++ compilers are on release 2023.3. Environment configuration for the compilers is done via /opt/intel/oneapi/setvars.sh. The problem is, the version of that script that sets up the right paths for the Fortran compiler is overwritten by the version that sets up the right paths for the C/C++ compilers when we install them. So we have to invoke setvars.sh TWICE: once after installing the Fortran compiler, and then once again after installing the C/C++ compilers. This gets all the right paths into play. Invoking it a second time generates an error unless you add the --force command-line paramater.

@billschereriii billschereriii added the area: CI/CD Issues related to continuous integration and deployment label Dec 15, 2023
@billschereriii billschereriii self-assigned this Dec 15, 2023
@billschereriii billschereriii marked this pull request as ready for review December 15, 2023 22:47
al-rigazzi
al-rigazzi approved these changes Dec 15, 2023
View reviewed changes
Copy link
Collaborator

@al-rigazzi al-rigazzi left a comment

Choose a reason for hiding this comment

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

LGTM, very strange behavior indeed!

ashao
ashao approved these changes Dec 15, 2023
View reviewed changes
Copy link
Member

@ashao ashao left a comment

Choose a reason for hiding this comment

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

Looks great thanks for tracking this down!

@billschereriii billschereriii merged commit 0151e00 into CrayLabs:develop Dec 15, 2023
46 checks passed
@billschereriii billschereriii deleted the inteltest branch December 15, 2023 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@al-rigazzi al-rigazzi al-rigazzi approved these changes

@ashao ashao ashao approved these changes

Assignees

@billschereriii billschereriii

Labels
area: CI/CD Issues related to continuous integration and deployment
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@billschereriii @ashao @al-rigazzi