<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Feb 21, 2017 at 10:02 PM roger peppe <<a href="mailto:roger.peppe@canonical.com">roger.peppe@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In August, the loggo package was changed to make it log ANSI-terminal<br class="gmail_msg">
colour escape sequences by default.<br class="gmail_msg">
<br class="gmail_msg">
I'm not sure that was the right decision, for a couple of reasons:<br class="gmail_msg">
<br class="gmail_msg">
- daemons are often given a pseudo tty to run in, so the log files<br class="gmail_msg">
produced by our running services are now obscured by escape sequences,<br class="gmail_msg">
making them hard to process automatically (and larger).<br class="gmail_msg">
<br class="gmail_msg">
- it means that the loggo dependencies are much larger.  Where<br class="gmail_msg">
previously the loggo package had no non-stdlib dependencies, it now<br class="gmail_msg">
depends on 5 significantly sized repositories holding >100k lines of<br class="gmail_msg">
source, including at least one system-dependent (Windows-only) repo.<br class="gmail_msg">
<br class="gmail_msg">
I'd like to propose that we remove the colour-by-default logic from<br class="gmail_msg">
loggo and move it to a separate package where it can be used if<br class="gmail_msg">
required by a given client. Perhaps <a href="http://github.com/utils/loggocolor" rel="noreferrer" class="gmail_msg" target="_blank">github.com/utils/loggocolor</a> might<br class="gmail_msg">
be a reasonable place.<br class="gmail_msg">
<br class="gmail_msg">
Then we can change the juju command line client to use that writer by<br class="gmail_msg">
default meaning there should be no change in externally visible<br class="gmail_msg">
behaviour to juju client users. The jujud server executable should<br class="gmail_msg">
probably not be using coloured logging output IMO.<br class="gmail_msg">
<br class="gmail_msg">
Thoughts?<br class="gmail_msg"></blockquote><div><br></div><div>Seems reasonable to me. I've added it to today's tech board agenda.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  cheers,<br class="gmail_msg">
    rog.<br class="gmail_msg">
<br class="gmail_msg">
--<br class="gmail_msg">
Juju-dev mailing list<br class="gmail_msg">
<a href="mailto:Juju-dev@lists.ubuntu.com" class="gmail_msg" target="_blank">Juju-dev@lists.ubuntu.com</a><br class="gmail_msg">
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br class="gmail_msg">
</blockquote></div></div>