I don't know if it is because of bad imports but I certainly see github.com/juju/juju/environs/testing github.com/juju/juju/juju/testing etc in the output of the go list output.
John =:-> On Tue, Apr 21, 2015 at 5:34 PM, Matthew Williams < [email protected]> wrote: > Unless I misunderstand my usage of go list (which is an absolute > possibility) then those instances aren't included. > > On Tue, Apr 21, 2015 at 3:31 PM, John Meinel <[email protected]> > wrote: > >> 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. >> >> John >> =:-> >> >> >> On Tue, Apr 21, 2015 at 5:17 PM, Matthew Williams < >> [email protected]> wrote: >> >>> Hi Folks, >>> >>> 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: >>> http://reviews.vapour.ws/r/1460/ >>> >>> 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. >>> >>> Thanks all >>> >>> Matty >>> >>> -- >>> 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
