This has been done. github.com/juju/errgo no longer has a dependency on github.com/juju/loggo
On Wed, May 21, 2014 at 7:39 AM, David Cheney <[email protected]> wrote: > SGTM. https://github.com/juju/errgo/pull/6 > > On Tue, May 20, 2014 at 10:06 PM, John Meinel <[email protected]> wrote: >> I'm fine having our split out packages depend on each other where it is >> actually useful (errors clearly depends on errgo), but it does sound like >> loggo isn't a very strong dependency and can just be removed. >> >> John >> =:-> >> >> >> >> On Tue, May 20, 2014 at 3:27 PM, Nate Finch <[email protected]> >> wrote: >>> >>> Actually, I just looked at the code again, and the code that uses loggo >>> will be compiled out unless you modify the code to change a constant. Let's >>> just remove the code and the package include, and if people want to modify >>> the code to print out debugging info, they can do that however they want. >>> >>> >>> On Tue, May 20, 2014 at 7:24 AM, Nate Finch <[email protected]> >>> wrote: >>>> >>>> I just noticed that errgo uses loggo for one single debug statement. I >>>> think this is a mistake if we want errgo to be reusable by the community. >>>> We should strive to make our independent packages actually independent.... >>>> removing unnecessary dependencies is a big part of that. We shouldn't >>>> force >>>> people to include loggo in their product if all they want is errgo.... >>>> especially given that we're barely using loggo in the package. >>>> >>>> Can we remove the debug code, or use the stdlib's logging or something? >>>> I don't think we're getting a lot of value out of using loggo in this >>>> package. >>> >>> >>> >>> -- >>> 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
