-
Notifications
You must be signed in to change notification settings - Fork 191
Add [install]
section to fpm.toml
#903
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
Conversation
[install]
section to fpm.toml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @moriglia, LGTM. Definitely a good idea, since this is a library. Let's wait for another review and comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @moriglia . It makes sense to me. It might be good to mention this new behaviour in the README
.
cc: @jalvesz |
LGTM @moriglia thanks for this contribution! |
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Thank you all for your positive feedback!
Sure, I think I should open a separate PR though, since the |
The |
@jalvesz is right, thank you for noticing it. These changes should be moved to the |
While reading the |
This commit allows for installation of the
stdlib
, instructingfpm
to install (if the prefix is not modified):libstdlib.a
file -->~/.local/lib/
.[s]mod
files -->~/.local/include/
Example:
fpm install --compiler=gfortran-11 --profile release
Resolves #782