-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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.