Source: ncdu
Version: 1.13-1
Severity: wishlist
Tags: patch

Hi Eugene

Thanks for packaging the new ncdu!

Please consider enabling color by default as per the attached patch.
I believe doing so in Debian Testing can assist upstream in perfecting this new feature.

I have already found one bug [1] after doing this in a local build.

Regards
Graham

[1] https://dev.yorhel.nl/ncdu/bug/111
Description: Enable color by default
Forwarded: not-needed
Author: Graham Inggs <gin...@debian.org>
Last-Update: 2018-03-28

--- a/src/util.c
+++ b/src/util.c
@@ -33,7 +33,7 @@
 #include <locale.h>
 #endif
 
-int uic_theme;
+int uic_theme = 1;
 int winrows, wincols;
 int subwinr, subwinc;
 int si;
Description: Fix option to disable color
Forwarded: https://dev.yorhel.nl/ncdu/bug/111
Author: Graham Inggs <gin...@debian.org>
Last-Update: 2018-03-28

--- a/src/main.c
+++ b/src/main.c
@@ -188,7 +188,7 @@
       break;
     case 'c':
       if(strcmp(val, "off") == 0)  { uic_theme = 0; }
-      if(strcmp(val, "dark") == 0) { uic_theme = 1; }
+      else if(strcmp(val, "dark") == 0) { uic_theme = 1; }
       else {
         fprintf(stderr, "Unknown --color option: %s\n", val);
         exit(1);

Reply via email to