<div dir="ltr">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.<div>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.</div>
<div>If I read it correctly, essentially Godeps wants to control your GOPATH rather than using a shared GOPATH.</div><div>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.)</div>
<div>We *can* switch, but it means we should be prepared to switch over how we run code, deal with dependencies, etc.</div><div><br>John</div><div>=:-></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, Jun 4, 2014 at 5:28 AM, Andrew Wilkins <span dir="ltr"><<a href="mailto:andrew.wilkins@canonical.com" target="_blank">andrew.wilkins@canonical.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">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.<div><br></div><div>Cheers,</div><div>Andrew</div></div>

<div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Tue, Jun 3, 2014 at 6:33 PM, Jesse Meek <span dir="ltr"><<a href="mailto:jesse.meek@canonical.com" target="_blank">jesse.meek@canonical.com</a>></span> wrote:<br>

</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
  

    
  
  <div bgcolor="#FFFFFF" text="#000000">
    After running:<br>
    <br>
    
    <pre style="font-family:Consolas,'Liberation Mono',Courier,monospace;font-size:15px;margin-top:0px;margin-bottom:0px;white-space:pre-wrap;color:rgb(51,51,51);font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;word-spacing:0px">
go get -v <a href="http://github.com/juju/core/." target="_blank">github.com/juju/core/.</a>..</pre>
    <br>
    I got:<br>
    <br>
    ../../<a href="http://github.com/juju/core/testing/imports.go:17" target="_blank">github.com/juju/core/testing/imports.go:17</a>: undefined:
    "<a href="http://github.com/juju/testing" target="_blank">github.com/juju/testing</a>".FindImports<br>
    <br>
    which was resolved by updating my dependencies with godeps.
    Shouldn't we add "godeps -u dependencies.tsv"  as a step in the
    readme?<br>
  </div>

<br></div></div><span class="HOEnZb"><font color="#888888">--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com" target="_blank">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
<br></font></span></blockquote></div><br></div>
<br>--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
<br></blockquote></div><br></div>