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

specify ifort location #906

Merged
merged 1 commit into from
Nov 22, 2023
Merged

specify ifort location #906

merged 1 commit into from
Nov 22, 2023

Conversation

Albkat
Copy link
Contributor

@Albkat Albkat commented Nov 20, 2023

Apparently, in the new oneAPI version, the setvars.sh script does not configure ifort.

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

awvwgk commented Nov 20, 2023

Can we check whether we are happy with the ifx compiler instead of fort, since the latter is now slowly getting deprecated by Intel?

@Albkat Albkat added the build Build system related issues (OS, compilers, meson) label Nov 21, 2023
@Albkat
Copy link
Contributor Author

Albkat commented Nov 22, 2023

As a simple trick with the name switching does not work, I have opened an issue #908.
I would suggest pushing this PR anyway, because the ifx is also not configured by the setvars.sh, and one has to explicitly source the corresponding vars.sh.

@MtoLStoN
Copy link
Member

We should definitely look into transitioning to the ifx compiler before it is deprecated in 2024, but I think, we can use this as an intermediate solution.

@Albkat Albkat merged commit 2d57755 into grimme-lab:main Nov 22, 2023
11 checks passed
@Albkat Albkat deleted the meson_intel branch November 22, 2023 16:09
gorges97 pushed a commit to gorges97/xtb that referenced this pull request Mar 27, 2024
Signed-off-by: albert <92109627+Albkat@users.noreply.github.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
build Build system related issues (OS, compilers, meson)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants