Hi Adam, I'm really busy right now. But if there is a patch out there that I need to apply "upstream", let me know.
Roger. On Thu, May 12, 2016 at 01:40:04PM -0600, Adam Conrad wrote: > Package: mtr > Version: 0.86-1 > Severity: normal > Tags: patch > User: ubuntu-de...@lists.ubuntu.com > Usertags: origin-ubuntu yakkety ubuntu-patch > > > > In Ubuntu, the attached patch was applied to achieve the following: > > * Pull commits from upstream to fix terminal colours (LP: #1581186) > > As I note in the LP bug[1], this is hardly a critical issue, but it's > been driving me nuts since 0.86 landed. Stupidly, I went and learned > curses and fixed it locally, which only then gave me the right search > terms (mtr + use_default_colors) to get Google to point out that it > was already fixed in upstream git. So, I threw away my work, grabbed > the upstream commits, and here we are. > > Would be lovely for you to pick this up in the Debian package, so I > can delete my recently-added delta from Ubuntu (and so Debian users > can enjoy slightly less broken colours in mtr in unstable). > > ... Adam > > [1] https://bugs.launchpad.net/ubuntu/+source/mtr/+bug/1581186 > > -- System Information: > Debian Release: stretch/sid > APT prefers yakkety > APT policy: (500, 'yakkety') > Architecture: amd64 (x86_64) > Foreign Architectures: i386 > > Kernel: Linux 4.4.0-21-lowlatency (SMP w/4 CPU cores; PREEMPT) > Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) > Shell: /bin/sh linked to /bin/dash > Init: systemd (via /run/systemd/system) > diff -Nru mtr-0.86/debian/patches/color1.patch > mtr-0.86/debian/patches/color1.patch > --- mtr-0.86/debian/patches/color1.patch 1969-12-31 17:00:00.000000000 > -0700 > +++ mtr-0.86/debian/patches/color1.patch 2016-05-12 12:58:21.000000000 > -0600 > @@ -0,0 +1,34 @@ > +commit 63a1f1493bfbaf7e55eb7e20b3791fc8b14cf92d > +Author: Rogier Wolff <r.e.wo...@bitwizard.nl> > +Date: Mon Dec 29 09:22:46 2014 +0100 > + > + added use-default-colors... > + > +diff --git a/configure.ac b/configure.ac > +index d5d1b0e..7199781 100644 > +--- a/configure.ac > ++++ b/configure.ac > +@@ -34,6 +34,9 @@ AC_CHECK_FUNC(initscr, , > + AC_DEFINE(NO_CURSES, 1, Define if you don't have the curses libraries > available.) > + CURSES_OBJ=)))) > + > ++AC_CHECK_LIB(ncurses, use_default_colors, > ++ AC_DEFINE(HAVE_USE_DEFAULT_COLORS, 1, [Define this if your curses library > has the use_default_colors() command.])) > ++ > + AC_CHECK_FUNCS(attron fcntl) > + > + AC_CHECK_LIB(m, floor, , AC_MSG_ERROR(No math library found)) > +diff --git a/curses.c b/curses.c > +index 3904cb1..02b7937 100644 > +--- a/curses.c > ++++ b/curses.c > +@@ -701,6 +701,9 @@ void mtr_curses_open(void) > + raw(); > + noecho(); > + start_color(); > ++#ifdef HAVE_USE_DEFAULT_COLORS > ++ use_default_colors(); > ++#endif > + int i; > + for (i = 0; i < 8; i++) > + init_pair(i+1, i, 0); > diff -Nru mtr-0.86/debian/patches/color2.patch > mtr-0.86/debian/patches/color2.patch > --- mtr-0.86/debian/patches/color2.patch 1969-12-31 17:00:00.000000000 > -0700 > +++ mtr-0.86/debian/patches/color2.patch 2016-05-12 12:58:21.000000000 > -0600 > @@ -0,0 +1,30 @@ > +commit 7571201cf7a3394e0dcd2b037aba1836089cc084 > +Author: Narthorn <narth...@gmail.com> > +Date: Mon Oct 12 13:24:57 2015 +0200 > + > + curses: Fix background transparency in terminal > + > + Patch comes from, and closes traviscross/mtr#72. > + > +diff --git a/curses.c b/curses.c > +index 02b7937..f95f5d1 100644 > +--- a/curses.c > ++++ b/curses.c > +@@ -700,13 +700,15 @@ void mtr_curses_open(void) > + initscr(); > + raw(); > + noecho(); > ++ int bg_col = 0; > + start_color(); > + #ifdef HAVE_USE_DEFAULT_COLORS > +- use_default_colors(); > ++ if (use_default_colors() == OK) > ++ bg_col = -1; > + #endif > + int i; > + for (i = 0; i < 8; i++) > +- init_pair(i+1, i, 0); > ++ init_pair(i+1, i, bg_col); > + > + mtr_curses_init(); > + mtr_curses_redraw(); > diff -Nru mtr-0.86/debian/patches/series mtr-0.86/debian/patches/series > --- mtr-0.86/debian/patches/series 2015-12-07 12:49:27.000000000 -0700 > +++ mtr-0.86/debian/patches/series 2016-05-12 12:58:41.000000000 -0600 > @@ -0,0 +1,2 @@ > +color1.patch > +color2.patch -- ** r.e.wo...@bitwizard.nl ** http://www.BitWizard.nl/ ** +31-15-2600998 ** ** Delftechpark 26 2628 XH Delft, The Netherlands. KVK: 27239233 ** *-- BitWizard writes Linux device drivers for any device you may have! --* The plan was simple, like my brother-in-law Phil. But unlike Phil, this plan just might work.