-
Notifications
You must be signed in to change notification settings - Fork 176
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
Add [install]
section to fpm.toml
#906
Conversation
Co-authored-by: Jeremie Vandenplas <jeremie.vandenplas@gmail.com>
Another thing I noticed is that Should I also change the project name to |
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.
LGTM, thank you @moriglia.
Regarding the fpm library name, I believe it should remain stdlib
, because that is the name that is typically used as a fpm
dependency (including as a "metapackage").
Thank you! @perazz |
Ok, thanks, got it! I'll fix and amend my latest commit as soon as I'm back to my laptop.
…On December 15, 2024 15:20:26 GMT+01:00, jalvesz ***@***.***> wrote:
@jalvesz commented on this pull request.
> @@ -219,7 +219,29 @@ python config/fypp_deployment.py --help
git checkout stdlib-fpm
fpm build --profile release
```
-#### Runing the examples
+
+
+#### Installing with fpm
+
+Either option you chose for building the `stdlib`, you can install it with:
+```sh
+fpm install --profile release
+```
+The command above will install the following files:
+- `libstdlib.a` into `~/.local/lib/`
Same pattern, what changes is that longer root, so it would be:
`C:\Users\<username>\AppData\Roaming\local\lib` and `C:\Users\<username>\AppData\Roaming\local\include`
--
Reply to this email directly or view it on GitHub:
#906 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
|
I think this PR can be merged. Let's wait a couple of days and if there are no remarks I'll merge it. |
Moved PR from
stdlib-fpm
tomaster
branch.Initial discussion here: #903