So there is a request up to do this, which we can move forward on, but I'm not personally convinced. If only because it isn't actually a simple drop-in replacement. Specifically, Godeps defaults to copying all of the dependencies into Godeps/_workspace, and then wants you to actually change your commands to "godeps go test ./..." etc. If I read it correctly, essentially Godeps wants to control your GOPATH rather than using a shared GOPATH. I'm not a big fan of having an extra copy of my dependencies in a hidden directory (since then I'm not actually sure what code is being run at any given time.) We *can* switch, but it means we should be prepared to switch over how we run code, deal with dependencies, etc.
John =:-> On Wed, Jun 4, 2014 at 5:28 AM, Andrew Wilkins <[email protected] > wrote: > Agreed, we should improve that. Sounds like we're going to change over to > using godep instead of godeps, so we should update this at the same time. > > Cheers, > Andrew > > > On Tue, Jun 3, 2014 at 6:33 PM, Jesse Meek <[email protected]> > wrote: > >> After running: >> >> go get -v github.com/juju/core/... >> >> >> I got: >> >> ../../github.com/juju/core/testing/imports.go:17: undefined: " >> github.com/juju/testing".FindImports >> >> which was resolved by updating my dependencies with godeps. Shouldn't we >> add "godeps -u dependencies.tsv" as a step in the readme? >> >> -- >> 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 > >
-- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
