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

Make linter point to the full compiler path #500

Closed
gnikit opened this issue May 31, 2022 · 0 comments · Fixed by #503
Closed

Make linter point to the full compiler path #500

gnikit opened this issue May 31, 2022 · 0 comments · Fixed by #503
Assignees
Milestone

Comments

@gnikit
Copy link
Member

gnikit commented May 31, 2022

Origin

in the README you are right it should not have ifort appended in the end of that path. I will fix it. In general, I think that I might just remove the tricks with the compiler paths and leave it up to the user to not pass unsafe binaries to the linter.

Originally posted by @gnikit in #492 (comment)

Explanation

Discard having to use the rootpath for the linter, instead use the path to the full binary. This will however make it possible for people to use the wrong linter binary and linter parsing expression.

@gnikit gnikit self-assigned this May 31, 2022
@gnikit gnikit added this to the v3.0 milestone May 31, 2022
gnikit added a commit that referenced this issue May 31, 2022
@gnikit gnikit mentioned this issue May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant