Navigation Menu

Skip to content
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

Add 'env_confdir' to constants #308

Closed
wants to merge 1 commit into from

Conversation

martbab
Copy link
Contributor

@martbab martbab commented Dec 5, 2016

Env confdir is always populated so it should be listed among variables
set during a call to Env._bootstrap().

https://fedorahosted.org/freeipa/ticket/6389

Env confdir is always populated so it should be listed among variables
set during a call to `Env._bootstrap()`.

https://fedorahosted.org/freeipa/ticket/6389
@tiran
Copy link
Member

tiran commented Dec 5, 2016

env_confdir was added in PR #302. This PR is required to make test passing.

@tiran tiran added the ack Pull Request approved, can be merged label Dec 5, 2016
@martbab martbab added the pushed Pull Request has already been pushed label Dec 5, 2016
@martbab
Copy link
Contributor Author

martbab commented Dec 5, 2016

@martbab martbab closed this Dec 5, 2016
@martbab martbab deleted the env-confdir-in-constants branch December 16, 2016 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants