Updates all gnatsd logging to use apcera/logging

[cee-dub]

No description provided.

[cee-dub]

Just in case, @liquidgecka or @krobertson want to take a look? I worked on this with @klobucar.

[cee-dub]

Meant to point this at @derekcollison for main review.

[derekcollison]

See comments from other commits. For now will probably not pull in and undo
other commits that introduce dependencies that could limit OSS of this
component. Once all the other items are OSS, then we can re-introduce them
without effecting OSS release schedules for this component.

On Tue, Jun 11, 2013 at 3:35 PM, Cameron Walters (cee-dub) <
notifications@github.com> wrote:

Meant to point this at @derekcollison https://github.com/derekcollisonfor main review.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-19297045
.

[cee-dub]

This is just a branch and PR, as Jon said. I think it's safe to leave as a branch until you're comfortable with the OSS plans of all dependencies.

---Sent from Boxer | http://getboxer.com

On Tue, Jun 11, 2013 at 07:48 PM, Derek Collison wrote:See comments from other commits. For now will probably not pull in and undo other commits that introduce dependencies that could limit OSS of this component. Once all the other items are OSS, then we can re-introduce them without effecting OSS release schedules for this component. On Tue, Jun 11, 2013 at 3:35 PM, Cameron Walters (cee-dub) < notifications@github.com> wrote: > Meant to point this at @derekcollison https://github.com/derekcollisonfor main review. > > — > Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-19297045 > . > —Reply to this email directly or view it on GitHub.

[cee-dub]

I'd also appreciate if you take a closer look at the true dependencies added. As far as I can see, it just adds apcera/logging and the logging facility's unittest helper. It does not depend on testtool or other private repos.

I think we're close on open sourcing logging, and at that point this should be acceptable (barring any other potential issues).

[derekcollison]

I could have missed it, so that is good that it is just one..

On Tue, Jun 11, 2013 at 11:29 PM, Cameron Walters (cee-dub) <
notifications@github.com> wrote:

I'd also appreciate if you take a closer look at the true dependencies
added. As far as I can see, it just adds apcera/logging and the logging
facility's unittest helper. It does not depend on testtool or other private
repos.

I think we're close on open sourcing logging, and at that point this
should be acceptable (barring any other potential issues).

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/4#issuecomment-19308788
.

[liquidgecka]

Also would love comments: https://github.com/apcera/logging/pull/12 =)