Remove the Global Logger

[timraymond]

We currently have a global logger in retinacmd.Logger. There's never a need for a global logger, and it inhibits the proper use of "child loggers", where certain contextual information is injected into scope-limited logger instances (helping to ensure consistent logging without overhead). We should remove the global logger and inject it in every instance.