<div style="font-family:arial,sans-serif;font-size:13px">Thanks!  I&#39;ve tested this and it works on my dark color scheme (it&#39;s better actually :)).  I also tested on white and it looks fine.  If Richard is ok with this solution, I&#39;ll make the changes and submit another patch.<br>
</div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div style="font-family:arial,sans-serif;font-size:13px">Seth</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 13, 2012 at 4:36 PM, Jason Wessel <span dir="ltr">&lt;<a href="mailto:jason.wessel@windriver.com" target="_blank">jason.wessel@windriver.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11/13/2012 09:22 AM, Richard Purdie wrote:<br>
&gt; On Tue, 2012-11-13 at 08:45 -0600, Seth Bollinger wrote:<br>
&gt;&gt; I&#39;m not sure how best to resolve this.  Would you like me to add some<br>
&gt;&gt; code to try and detect background color?  This seems like it might be<br>
&gt;&gt; more complex than it&#39;s worth...<br>
&gt;&gt;<br>
&gt;&gt; Or would it be better to have a selectable light background color<br>
&gt;&gt; set?  If you think light background is more common (I dislike staring<br>
&gt;&gt; at large blocks of white :)) it could default to the light color set.<br>
&gt; I have not looked into this and am not an expert on terminals and colour<br>
&gt; control of them. I don&#39;t know how we can solve this but the current<br>
&gt; situation where it prints white on white on my terminal is clearly<br>
&gt; unacceptable. This has to be a problem others have run into and solved<br>
&gt; before though...<br>
<br>
<br>
If you are planning to colorize things, my recommendation would be to take the &quot;middle ground&quot; and use the transparency instead of using white which should work in general on any modern terminal.<br>
<br>
Change the patch a bit like this:<br>
+    color_enabled = False<br>
+    BLACK, RED, GREEN, YELLOW, BLUE, MAGENTA, CYAN, WHITE = range(30,38)<br>
<br>
    BASECOLOR, BLACK, RED, GREEN, YELLOW, BLUE, MAGENTA, CYAN, WHITE = range(29,38)<br>
<br>
+<br>
+    COLORS = {<br>
+        DEBUG3  : CYAN,<br>
+        DEBUG2  : CYAN,<br>
+        DEBUG   : CYAN,<br>
+        VERBOSE : BASECOLOR,<br>
+        NOTE    : BASECOLOR,<br>
+        PLAIN   : BASECOLOR,<br>
+        WARNING : YELLOW,<br>
+        ERROR   : RED,<br>
+        CRITICAL: RED,<br>
+    }<br>
<br>
<br>
Most of the time terminals are either &quot;dark&quot; or &quot;light&quot;, and default color is always set to -1 in the curses interface.  For example, the attached patch will allow the patch to work on my &quot;lawngreen&quot; default xterms as well as more typical defaults for gnome-terminal or rxvt.<br>

<br>
Cheers,<br>
Jason.<br>
</blockquote></div><br></div>