LGTM
https://codereview.appspot.com/11321043/diff/1/environs/local/config.go File environs/local/config.go (right):
https://codereview.appspot.com/11321043/diff/1/environs/local/config.go#newcode16 environs/local/config.go:16: var rootCheckFunction = func() bool { Perhaps checkRoot() is better?
https://codereview.appspot.com/11321043/diff/1/environs/local/config.go#newcode86 environs/local/config.go:86: // change ownership of the directories. I don't think the above comment belongs here
https://codereview.appspot.com/11321043/
« Back to merge proposal
LGTM
https:/ /codereview. appspot. com/11321043/ diff/1/ environs/ local/config. go local/config. go (right):
File environs/
https:/ /codereview. appspot. com/11321043/ diff/1/ environs/ local/config. go#newcode16 local/config. go:16: var rootCheckFunction = func() bool {
environs/
Perhaps checkRoot() is better?
https:/ /codereview. appspot. com/11321043/ diff/1/ environs/ local/config. go#newcode86 local/config. go:86: // change ownership of the directories.
environs/
I don't think the above comment belongs here
https:/ /codereview. appspot. com/11321043/