Your message dated Sun, 13 Jan 2013 12:33:56 +0100
with message-id 
<cab4xwxzc2dk+ccablr6mcezhbcbsktnow+7ezfyzbzctqqz...@mail.gmail.com>
and subject line Re: Bug#697696: diffstat: Doesn't fetch terminal width 
correctly
has caused the Debian Bug report #697696,
regarding diffstat: Doesn't fetch terminal width correctly
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.)


-- 
697696: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697696
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: diffstat
Version: 1.55-3
Severity: normal

Diffstat tries to get the terminal width to auto-scale the histogram, but
the method it uses simply doesn't work, at least with bash.

The method it uses is getenv("COLUMNS").  Unfortunately, at least with bash,
the COLUMNS variable is normally a "magic" variable, it doesn't really exist
in the environment, and so diffstat always thinks the terminal is 80 chars
wide, which often results in forcing the histogram to only be its minimum 10
chars wide, no matter how big the terminal window is.

Workaround: export COLUMNS

-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.6-trunk-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages diffstat depends on:
ii  libc6  2.13-37

diffstat recommends no packages.

diffstat suggests no packages.

-- no debconf information

--- End Message ---
--- Begin Message ---
On Wed, Jan 9, 2013 at 11:50 PM, Sandro Tosi <[email protected]> wrote:
> By default, diffstat doesn't try to identify the terminal width at
> all, but uses a default 80 chars columns width value. It was a recent
> patch[1] that introduces the behaviour of checking COLUMNS (else you'd
> have to use -w NUM). So the current situation might be better than the
> static columns width in same cases, so can you elaborate a bit more
> what you want from this bug report (I would close it actually)? :)
>
> [1] 
> http://patch-tracker.debian.org/patch/series/view/diffstat/1.55-3/01-check-columns-env-var.patch

Delivery to the following recipient failed permanently:

     [email protected]

Technical details of permanent failure:
Google tried to deliver your message, but it was rejected by the
recipient domain. We recommend contacting the other email provider for
further information about the cause of this error. The error that the
other server returned was: 451 451-209.85.212.171 is not yet
authorized to deliver mail from 451 <me> to <[email protected]>.
Please try later. (state 13).

I don't see a reason to keep this bug opened when I can't contact the
submitter, closing.

Regards,
--
Sandro Tosi (aka morph, morpheus, matrixhasu)
My website: http://matrixhasu.altervista.org/
Me at Debian: http://wiki.debian.org/SandroTosi

--- End Message ---

Reply via email to