-
Notifications
You must be signed in to change notification settings - Fork 279
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
feat(config): Allow loading extra config from file #340
base: master
Are you sure you want to change the base?
Conversation
8654f39
to
42e29ba
Compare
@@ -163,6 +165,12 @@ + (void) prepareUserDefaults | |||
for (count = 0; count < sizeof(confFiles)/sizeof(confFiles[0]); count++) | |||
_injectConfigurationFromFile (configFromFiles, confFiles[count], logger); | |||
|
|||
/* Fill/Override configuration with configuration stored in the extra | |||
* config file */ | |||
envFile = [[[NSProcessInfo processInfo] environment] objectForKey:@"SOGO_EXTRA_CONFIG_FILE"]; |
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.
It would be helpful if this was documented
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.
I totally agree. Do you have any pointer where an appropriate location would be?
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.
I also agree the documentation should be updated. I suggest modifying this part : https://github.com/Alinto/sogo/blob/master/Documentation/SOGoInstallationGuide.asciidoc#gnustep-environment-overview
Hi @dasJ thanks for contributing ! Can you update documentation ? |
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.
Please add missing doc
e4c98fd
to
2258d5c
Compare
No description provided.