Code review comment for lp:~cmars/juju-core/no-default-series

Revision history for this message
Dave Cheney (dave-cheney) wrote :

On 2014/03/17 22:55:29, cmars wrote:
> Tim,
> Please review my proposal for eliminating default series from the juju
client
> (or pass it along to someone who's up for it). In a nutshell:

> - Allow charm.URL to have an implied "empty" series.
> - charm.Repository can Resolve them.
> - The charm store "resolves" series by using a hard-coded value (for
now).
> - A local repository uses the environment config default series.
> - It's a client command's responsibility to make sure a charm URL is
resolved
> before calling the state server with it.

> Thanks,
> Casey

I think this needs to be broken out into two changes. The store changes
need to land well before the release that contains the client changes.

https://codereview.appspot.com/76860044/

« Back to merge proposal