On Mon, 2008-03-31 at 20:04 +0000, Steve Langasek wrote:
> While it may seem obvious that we would want diffstat to use the
> $COLUMNS value from the environment, this change is a behavior change
> for anyone running diffstat in a terminal window that's not 80 columns
> wide, which may have unexpected side effects for tools that invoke
> diffstat.  So I think it's better to defer this change until after the
> hardy release.
> 
> In the meantime, perhaps you could consider getting this change
> accepted into the Debian package, so that it will flow into intrepid
> automatically once the archive unfreezes?  Also, you may want to
> consider whether the decision to honor the $COLUMNS value should be
> conditional on whether stdout points at a terminal - this is something
> that packages like, e.g., dpkg do, to good effect.

I have submitted it upstream for consideration, and I can look into
getting it into Debian.  Not sure what that would entail precisely, but
I can look at it and see what I can do with it.

Also you brought up a great point there about considering what stdout
is.  I will add that to my patch shortly.  Thanks!

-- 
Michael B. Trausch                                   [EMAIL PROTECTED]
home: 404-592-5746, 1                                 www.trausch.us
cell: 678-522-7934                       im: [EMAIL PROTECTED], jabber
Ubuntu Unofficial Backports Project:    http://backports.trausch.us/


** Attachment added: "unnamed"
   http://launchpadlibrarian.net/13003980/unnamed

-- 
diffstat does not honor COLUMNS environment variable
https://bugs.launchpad.net/bugs/209537
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to