-
Notifications
You must be signed in to change notification settings - Fork 100
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
Enter custom path to the config file via command line #945
base: main
Are you sure you want to change the base?
Conversation
There are a few things I'd like to point out about this PR before I think it's ready to merge:
|
https://fpm.fortran-lang.org/en/spec/manifest.html#global-config-file
That's a bit tricky to answer. I agree that it is more of a command option (which it also is) than it is part of the dependency tree. But just as some other members of the dependency tree which seem a bit misplaced, such as |
Actually unrelated to |
# Conflicts: # src/fpm.f90 # src/fpm/dependency.f90 # src/fpm_settings.f90
The points have been addressed. Let's move on with the reviews. Having this PR in will make it easier to set up some example packages that include, e.g., a local registry. |
db4f1f0
to
de1fcdd
Compare
CI failure is unrelated to this PR, It is due to #1025 . |
Instead of having the global config file stored in a predetermined system location, users can specify a custom location for the file by using a command option. That allows for having all configurations in a single folder and makes it easier, e.g., to use the global config file in the CI.
Use:
fpm build --config-file config.toml # read file from project root
The documentation will be updated as soon as this PR is approved.