Code review comment for lp:~vila/bzr/353370-notty-no-term-width

Revision history for this message
Vincent Ladeuil (vila) wrote :

I just push a new version that should address the points raised so far,
including the IRC discussion with Martin.

In summary, terminal_width() now returns None when no terminal can be found
to get the width from. I've also introduced a BZR_COLUMNS env variable that
takes precedence and can be used to force any value when *needed*
(pagers, pager users and babune I'm looking at you).

« Back to merge proposal