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

feat: add fypp linting support for Intel and NAG #615

Open
1 task done
gnikit opened this issue Aug 14, 2022 · 3 comments
Open
1 task done

feat: add fypp linting support for Intel and NAG #615

gnikit opened this issue Aug 14, 2022 · 3 comments

Comments

@gnikit
Copy link
Member

gnikit commented Aug 14, 2022

Is there an existing request for this?

  • I have searched the existing issues

Feature Request

Currently, fypp is only capable of handling stdin -> gfortran without creating any temporary files. ifort and nagfor cannot receive input from stdin hence we will have to generate the files and store them, then pass the expanded files through the linter.

This issue relies on #614 being implemented.

@awvwgk
Copy link
Member

awvwgk commented Aug 14, 2022

You could try ifort -fpp-name=$(command -v fypp) to change the preprocessor used with the Intel compilers.

@gnikit
Copy link
Member Author

gnikit commented Aug 14, 2022

You could try ifort -fpp-name=$(command -v fypp) to change the preprocessor used with the Intel compilers.

Cheers @awvwgk, will give it a go

@ivan-pi
Copy link
Member

ivan-pi commented Aug 31, 2022

@awvwgk, have you tried that before? It seems like fypp will need response files before it can work: aradi/fypp#24

@gnikit gnikit modified the milestone: Release: v4.0.0 Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

3 participants