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

Eigen: explicitly disable Fortran usage #704

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Conversation

NeroBurner
Copy link

We don't want different packages be created wheter a Fortran compiler is available or not. As Eigen 3.4.0 has no flags to disable the Fortran related code hook into the
CheckLanguge CMake feature.

For the variable CMAKE_<LANG>_COMPILER the documentation states the following:

If this variable is already set, either explicitly or cached by a previous
call, the check is skipped.

We use this to effectively force Eigen to think that there was no Fortan compiler.

Fixes the ninja and nmake vs-16-2019 based toolchains.

We don't want different packages be created wheter a Fortran compiler is
available or not. As Eigen 3.4.0 has no flags to disable the Fortran
related code hook into the
[CheckLanguge](https://cmake.org/cmake/help/latest/module/CheckLanguage.html)
CMake feature.

For the variable `CMAKE_<LANG>_COMPILER` the documentation states the
following:

> If this variable is already set, either explicitly or cached by a previous
> call, the check is skipped.

We use this to effectively force Eigen to think that there was no Fortan
compiler.

Fixes the ninja and nmake vs-16-2019 based toolchains.
@NeroBurner NeroBurner added bug Something isn't working package:update Add a new version to an existing package labels Oct 25, 2023
@rbsheth rbsheth merged commit e96d84f into master Oct 26, 2023
36 of 38 checks passed
@rbsheth rbsheth deleted the eigen_force_no_fortan branch October 26, 2023 16:14
@NeroBurner
Copy link
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working package:update Add a new version to an existing package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants