Fallback for config will work correctly when having to determine the HOME directory. #38

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@RichardQuadling

getenv() is NOT fed by changes to $_ENV, so using getenv() to load the config file will fail when the fall back is used.

Correctly handle $_ENV['HOME'].
getenv() is NOT fed by changes to $_ENV, so using getenv() to load the config file will fail when the fall back is used.
@jeremeamia

This comment has been minimized.

Show comment Hide comment
@jeremeamia

jeremeamia Sep 19, 2012

Contributor

Thanks for the pull request. We'll try to get this integrated into the SDK soon.

Contributor

jeremeamia commented Sep 19, 2012

Thanks for the pull request. We'll try to get this integrated into the SDK soon.

@jeremeamia

This comment has been minimized.

Show comment Hide comment
@jeremeamia

jeremeamia Sep 20, 2012

Contributor

I've made the changes in my local copy and have added you to the CONTRIBUTORS doc. We do not accept pull requests directly for logistical reasons, but your changes will appear in the next version of the SDK. Thanks again!

Contributor

jeremeamia commented Sep 20, 2012

I've made the changes in my local copy and have added you to the CONTRIBUTORS doc. We do not accept pull requests directly for logistical reasons, but your changes will appear in the next version of the SDK. Thanks again!

@jeremeamia jeremeamia closed this Sep 20, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment