Skip to content

Conversation

@murphi
Copy link

@murphi murphi commented Sep 2, 2023

Enable resolving environment variables in the include statement of configuration files. The path of the included file may contain environment variables.

Enable resolving environment variables in the include statement of
configuration files. The path of the included file may contain
environment variables.

Example:
include ${PATH}/included-file
@murphi
Copy link
Author

murphi commented Sep 2, 2023

A similar approach has been submitted in #200.

@wilx
Copy link
Contributor

wilx commented Sep 5, 2023

It is impossible to see what you changed because you changed the indentation of the lines.

@wilx
Copy link
Contributor

wilx commented Sep 5, 2023

Oh, you moved things around. I can see now.

@wilx wilx self-assigned this Sep 5, 2023
@wilx wilx merged commit 2c15e3d into log4cplus:master Sep 5, 2023
@murphi murphi deleted the resolve_environment_variables_in_include branch September 9, 2023 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants