Skip to content

Conversation

@awvwgk
Copy link
Member

@awvwgk awvwgk commented Jul 19, 2021

Intel Fortran (ifort) is choking on the TOML proxy module in fpm, due to overloading the get_value interface in the proxy with an fpm specific data type. For some reason this alters the signature of the toml_table data type and causes a compilation error.

urbanjost
urbanjost approved these changes Jul 19, 2021
@urbanjost
Copy link
Contributor

Note this fixed #520, and using an earlier kludged version of my own that worked with ifort, ifort let me verify two bugs in fpm and is very helpful in helping ensure fpm is as standard-conforming as possible going into the future. Can anyone else take a look?

Copy link
Member

@LKedward LKedward left a comment

Choose a reason for hiding this comment

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

Thanks both! Being able to use ifort will be incredibly useful for developing and debugging fpm. LGTM

@LKedward LKedward linked an issue Jul 28, 2021 that may be closed by this pull request
@awvwgk
Copy link
Member Author

awvwgk commented Jul 28, 2021

With two approvals I'll go ahead and merge later today.

Next thing to discuss is whether we want to start testing ifort in the CI as well.

@awvwgk awvwgk merged commit 8c4fdae into fortran-lang:master Jul 30, 2021
@awvwgk awvwgk deleted the ifort-fix branch July 30, 2021 17:07
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.

building fpm with ifort

3 participants