Code review comment for lp:~jameinel/juju-core/1.18-local-trusty-and-precise-1306537

Revision history for this message
Andrew Wilkins (axwalk) wrote :

One comment, otherwise LGTM.

https://codereview.appspot.com/90640043/diff/1/cmd/juju/bootstrap.go
File cmd/juju/bootstrap.go (right):

https://codereview.appspot.com/90640043/diff/1/cmd/juju/bootstrap.go#newcode137
cmd/juju/bootstrap.go:137: // If we are going to be forcing the tools,
we might as
Should this be under "if c.UploadTools" below, rather than specific to
the local provider?

https://codereview.appspot.com/90640043/

« Back to merge proposal