Skip to content

Add Intel oneAPI env variables.#148

Merged
BillyONeal merged 2 commits intomicrosoft:mainfrom
Neumann-A:intel_passthrough
Aug 6, 2021
Merged

Add Intel oneAPI env variables.#148
BillyONeal merged 2 commits intomicrosoft:mainfrom
Neumann-A:intel_passthrough

Conversation

@Neumann-A
Copy link
Contributor

Required to use the Intel Fortran compiler instead of gfortran and probably also required to better find the mkl libraries?
Should I also add I_MPI_ONEAPI_ROOT (although that can be calculated by having ONEAPI_ROOT)?

Required to use the Intel Fortran compiler instead of gfortran
L"ANDROID_NDK_HOME",
// Environmental variables generated automatically by Intel oneAPI after installation
L"ONEAPI_ROOT",
L"IFORT_COMPILER19",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
L"IFORT_COMPILER19",
L"IFORT_COMPILER19",
L"IFORT_COMPILER20",
L"IFORT_COMPILER21",

maybe move all those variables into the triplet via VCPKG_ENV_PASSTHROUGH_UNTRACKED ?
And I don't mean only the ones I added I mean all of them.

@BillyONeal
Copy link
Member

Should I also add I_MPI_ONEAPI_ROOT (although that can be calculated by having ONEAPI_ROOT)?

I think that's more dependent on OneAPI rather than us

@Neumann-A
Copy link
Contributor Author

I think that's more dependent on OneAPI rather than us

I was thinking of more for what it could be useful. After checking FindMPI it is not using I_MPI_ONEAPI_ROOT but I_MPI_ROOT. Changing the MPI implementation used within vcpkg probably requires a port overlay and a custom triplet anyway so I would say that it is not required to forward I_MPI_ONEAPI_ROOT

@BillyONeal BillyONeal merged commit f9131cb into microsoft:main Aug 6, 2021
@BillyONeal
Copy link
Member

Thanks for your contribution!

@Neumann-A Neumann-A deleted the intel_passthrough branch August 6, 2021 20:50
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.

3 participants