Bug#859387: NeedRestart::UI assumes STDOUT is a terminal

2017-11-01 Thread Thomas Liske
Re, Guillaume writes: > Would the fix make it into the upcoming 2.12 release ? > > I just want to know which version to watch for, no hurry. yes, it will be part of 2.12 as any other fixes and changes being merged before releasing it. There are still a few issues left for fixing before I'm goin

Bug#859387: NeedRestart::UI assumes STDOUT is a terminal

2017-11-01 Thread Guillaume
Le 01/11/2017 à 19:08, Thomas Liske a écrit : > > I've adopted the PR and GetTerminalSize() should be no more called on > a non-TTY STDOUT. Thanks for the report and the PR. > > > Regards, > Thomas > > Hi Thomas, Thank for the quick feedback on the PR. Would the fix make it into the upcomi

Bug#859387: NeedRestart::UI assumes STDOUT is a terminal

2017-11-01 Thread Thomas Liske
tags 859387 upstream fixed-upstream forwarded 859387 https://github.com/liske/needrestart/issues/86 thanks Hi, I've adopted the PR and GetTerminalSize() should be no more called on a non-TTY STDOUT. Thanks for the report and the PR. Regards, Thomas Guillaume writes: > Hi, > > This also a

Bug#859387: NeedRestart::UI assumes STDOUT is a terminal

2017-11-01 Thread Guillaume
Hi, This also affects needrestart 2.11-3 on Debian testing I have logged a bug upstream for this at https://github.com/liske/needrestart/issues/86 Louis' workaround isn't sufficient on my case, I am currently testing a fix locally and may submit a pull request upstream later. Guillaume -- Jabb

Bug#859387: NeedRestart::UI assumes STDOUT is a terminal

2017-04-02 Thread Paul Wise
Package: needrestart Version: 2.11-2 Severity: normal File: /usr/share/perl5/NeedRestart/UI.pm Usertags: terminal I switched to unattended-upgrades and now needrestart prints lots of the following lines, it appears that it prints one for each message it prints. I would recommend checking if the $