-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
RootDir unused / Writing to zefs config #384
Comments
Looks like and now its just using Lines 21 to 24 in 3c75618
The original RootDir behavior could be revived, but it might be worth considering adding a subdir that is the current zef version (via |
I'm not objected to a versioning of configuration files. I do think we should make sure that custom configuration options are not silently disabled on a zef upgrade though. If I understand the RootDir approach correctly, it would not allow setting a single configuration value individually (by creating a configuration file in |
Ah right. At the very least there needs to be a way to append and replace. I liked the NuGet way other than being XML. |
Context
The documentation explains a mechanism of how the config file is chosen. That mechanism seems not to be implemented. (A search for
RootDir
in the zef sources yields no relevant results).I stumbled upon this while looking at a fez bug where fez tries to write to the zef config and fails because of missing permissions. In general I would consider this a valid use case. So I would favour a solution that will always allow changing the configuration. The first thing that comes to mind is a configuration hierarchy, where a config key in
$*HOME/.zef/config.json
would overwrite one in zefs default config which is part of it's installation. Then one could always create (should it not already exist) a config file in$*HOME/.zef/config.json
and add the keys one wishes to override. The other keys would still be read from the default config.Expected Behavior
Documentation and implementation match.
Actual Behavior
Documentation and implementation don't match up.
The text was updated successfully, but these errors were encountered: