On Thu, Nov 13, 2014 at 4:05 PM, David Cheney <[email protected]> wrote: > Hello, > > Please stop adding dependencies to the state package. > > The dependency tree for state must go > > mgo <- state <- things that depend on state > > At the moment state depends on things like backups, multiwatchers, > presence, toolstorage. > > This is wrong. > > Eric, i'm not picking on you here, it's just that b sorts first, and > this is the one I saw today. There are many other examples, including > the multiwatcher which also break this model > > If you add functionality to state, then the code has to go ***INSIDE** > state, or be written to take a *state.State. > > Please help me out by not adding more dependencies to state.
Thanks for pointing this out, Dave. I'll fix this for backups. -eric -- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
