<div dir="ltr">Unless I misunderstand my usage of go list (which is an absolute possibility) then those instances aren't included.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 21, 2015 at 3:31 PM, John Meinel <span dir="ltr"><<a href="mailto:john@arbash-meinel.com" target="_blank">john@arbash-meinel.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">Looking at the patch, that doesn't seem to filter "testing" packages, does it? Those intentionally have those dependencies, but should only be imported in *_test.go code.<div><br></div><div>John</div><div>=:-></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Tue, Apr 21, 2015 at 5:17 PM, Matthew Williams <span dir="ltr"><<a href="mailto:matthew.williams@canonical.com" target="_blank">matthew.williams@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 dir="ltr">Hi Folks,<div><br></div><div>There seem to be a number of places in core where we end up importing gocheck in non test code. We should have a plan for reducing this down to zero, and at some stage not merge code in that does this. That's not a reasonable thing to do at the moment so I've just proposed a new rule in the Makefile that we can use to track it:</div><div><a href="http://reviews.vapour.ws/r/1460/" target="_blank">http://reviews.vapour.ws/r/1460/</a><br></div><div><br></div><div>Any implementation specific comments should go on the review. But I wanted to email the list incase anyone wanted to discuss the idea in general - and possible resolutions.</div><div><br></div><div>Thanks all</div><div><br></div><div>Matty</div></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>
</blockquote></div><br></div>