On Wed, Dec 18, 2013 at 10:44 AM, Curtis Hovey-Canonical <
[email protected]> wrote:

> On Tue, Dec 17, 2013 at 6:14 PM, David Cheney
> <[email protected]> wrote:
> > The branch is not checked out properly.
>
> I don't see this path
>     $GOPATH/src/juju-core/ssh-options
> in
>
> http://bazaar.launchpad.net/~axwalk/juju-core/lp1258240-bootstrap-refresh-dnsname-take2/files
> or
>     trunk
>
> The make-release-tarball script
>
> http://bazaar.launchpad.net/~juju-qa/juju-core/ci-cd-scripts2/view/head:/make-release-tarball.bash
> uses "go get -v -d launchpad.net/juju-core/..."
> then uses bzr to force the version (and branch) to be the one under
> test. Finally it runs godeps to set everything to be right version.
> godeps is not happy.
>
> My use of a user branch is unorthodox, but I think it should work. I see in
>     provider/common/bootstrap.go
> an import of
>     "juju-core/ssh-options/utils/ssh"
>

This looks wrong, the import path should be

import "launchpad.net/juju-core/utils/ssh"


> but looking at his branch, I would think this path is actually trying to
> import
>
> http://bazaar.launchpad.net/~axwalk/juju-core/lp1258240-bootstrap-refresh-dnsname-take2/view/head:/utils/ssh/ssh.go
>
> --
> Curtis Hovey
> Canonical Cloud Development and Operations
> http://launchpad.net/~sinzui
>
> --
> Juju-dev mailing list
> [email protected]
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>
-- 
Juju-dev mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev

Reply via email to