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

temporary fix for CI #937

Merged
merged 2 commits into from
Dec 25, 2023
Merged

temporary fix for CI #937

merged 2 commits into from
Dec 25, 2023

Conversation

Albkat
Copy link
Contributor

@Albkat Albkat commented Dec 22, 2023

Address the issue of the new ifort version not compiling programs by temporarily reverting CI to an older version of ifort as a workaround, until ifx is fully integrated.

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
awvwgk
awvwgk previously approved these changes Dec 22, 2023
@awvwgk
Copy link
Member

awvwgk commented Dec 22, 2023

Maybe also add a hint for the upper bound of ifort in the README

Signed-off-by: Albert Katbashev <akatbashev@gmail.com>
@Albkat
Copy link
Contributor Author

Albkat commented Dec 25, 2023

Maybe also add a hint for the upper bound of ifort in the README

Thanks, the hint is added.
Since I don't have access to any CUDA machines, I still lack information on the PGI build. However, this should be irrelevant for the current pull request.

@awvwgk awvwgk merged commit 6933e33 into grimme-lab:main Dec 25, 2023
11 checks passed
gorges97 pushed a commit to gorges97/xtb that referenced this pull request Mar 27, 2024
* temporary fix for CI

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>

* add compiler information

Signed-off-by: Albert Katbashev <akatbashev@gmail.com>

---------

Signed-off-by: albert <92109627+Albkat@users.noreply.github.com>
Signed-off-by: Albert Katbashev <akatbashev@gmail.com>
Signed-off-by: Johannes Gorges <58849467+gorges97@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants