Code review comment for lp:~jameinel/juju-core/init-1147771

Revision history for this message
John A Meinel (jameinel) wrote :

*** Submitted:

environs/config: ensure ~/.juju exists

When doing WriteEnviron we would create ~/.juju/environments.yaml
but we weren't creating the containing dir. (Which meant actually
running 'juju init -w' wouldn't work for new users.)

This adds an unconditional MkdirAll() before writing environments.yaml,
but that should be ok, as it is a no-op if the dir already exists.

R=dfc, dimitern, rog
CC=
https://codereview.appspot.com/7759044

https://codereview.appspot.com/7759044/

« Back to merge proposal