Your message dated Fri, 7 Feb 2014 16:48:26 -0800
with message-id <[email protected]>
and subject line Re: git: auto colors broken with lv
has caused the Debian Bug report #730527,
regarding git: auto colors broken with lv
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
730527: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=730527
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: git
Version: 1:1.8.4.3-1
Severity: normal

Dear Maintainer,

Since upstream has made color.ui=auto the default, my git experience has
suffered.  I use lv as my preferred pager and somehow that doesn't jive
with what the various git commands feed it.  Pagination itself works but
instead of colors I get to see control codes like below (partial output
of git diff):

  ^[[1mdiff --git a/.mailfilter b/.mailfilter^[[m
  ^[[1mindex aa4f0b2..17e113e 100644^[[m
  ^[[1m--- a/.mailfilter^[[m
  ^[[1m+++ b/.mailfilter^[[m
  ^[[36m@@ -1,11 +1,58 @@^[[m

Using GIT_PAGER=more or GIT_PAGER=less works fine and show paginated,
colored output.  Nice!

I am loathe to switch to less though, as I have to deal with files in a
number of different encodings at the same time.  The lv pager does a
good job at automatically detecting those on the fly, less does not.

I am not sure what would be the best approach to fix this.  The pager
command is there for a purpose and if git would silently change to a
different one, I would certainly be surprised (and depending on the
diff, confused as to why changes don't look as expected).

Come to think of this, you might even argue it is something that ought
to be fixed in lv (but I don't know if that's even possible).

I guess for now I'll just turn off color.ui ...

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.11-2-amd64 (SMP w/8 CPU cores)
Locale: LANG=ja_JP.UTF-8, LC_CTYPE=ja_JP.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages git depends on:
ii  git-man          1:1.8.4.3-1
ii  libc6            2.17-93
ii  libcurl3-gnutls  7.33.0-1
ii  liberror-perl    0.17-1.1
ii  libexpat1        2.1.0-4
ii  libpcre3         1:8.31-2
ii  perl-modules     5.18.1-4
ii  zlib1g           1:1.2.8.dfsg-1

Versions of packages git recommends:
ii  less                         458-2
ii  openssh-client [ssh-client]  1:6.4p1-1
ii  patch                        2.7.1-4
ii  rsync                        3.1.0-2

Versions of packages git suggests:
ii  gettext-base                          0.18.3.1-1
pn  git-arch                              <none>
pn  git-bzr                               <none>
ii  git-cvs                               1:1.8.4.3-1
pn  git-daemon-run | git-daemon-sysvinit  <none>
pn  git-doc                               <none>
ii  git-el                                1:1.8.4.3-1
ii  git-email                             1:1.8.4.3-1
ii  git-gui                               1:1.8.4.3-1
ii  git-svn                               1:1.8.4.3-1
ii  gitk                                  1:1.8.4.3-1
pn  gitweb                                <none>

-- no debconf information

-- 
Olaf Meeuwissen, LPIC-2           FLOSS Engineer -- AVASYS CORPORATION
FSF Associate Member #1962               Help support software freedom
                 http://www.fsf.org/jf?referrer=1962

--- End Message ---
--- Begin Message ---
Version: 1:1.9~rc0-1

Olaf Meeuwissen wrote:

>            I use lv as my preferred pager and somehow that doesn't jive
> with what the various git commands feed it.  Pagination itself works but
> instead of colors I get to see control codes like below (partial output
> of git diff):
>
>   ^[[1mdiff --git a/.mailfilter b/.mailfilter^[[m
>   ^[[1mindex aa4f0b2..17e113e 100644^[[m
>   ^[[1m--- a/.mailfilter^[[m
>   ^[[1m+++ b/.mailfilter^[[m
>   ^[[36m@@ -1,11 +1,58 @@^[[m

Should be fixed by v1.9-rc0~15^2 (pager: set LV=-c alongside
LESS=FRSX, 2014-01-06).  Thanks again for reporting it.

Ciao,
Jonathan

--- End Message ---

Reply via email to