Your message dated Sat, 03 Mar 2018 21:47:09 +0000
with message-id <[email protected]>
and subject line Bug#876459: fixed in needrestart 2.11-3+deb9u1
has caused the Debian Bug report #876459,
regarding needrestart: Non-interactive mode not being detected properly
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.)


-- 
876459: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=876459
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: needrestart
Version: 2.11-3
Severity: important
Tags: upstream patch

Dear Maintainer,
the problem described in the archived bug #803249 still exists in 2.11-3 
(Debian 9.1). When needrestart is run without a tty AND with 
DEBIAN_FRONTEND=noninteractive, it still tries to work interactively rather 
than fall back to "list only" mode. In some cases it causes unwanted restarts. 
The easiest way to reproduce it is to run needrestart from cron, redirecting 
the output to a file.

I attach a patch that may work, but I'm not sure, as I know Perl only a little 
and don't know other implementation details. In my case it is working as 
expected.

The problem is rather serious. I use Ansible for automation and once I run it 
to install something on an OpenVZ host. After installation by apt-get, 
needrestart started to restart VZ containers, including the one from which 
Ansible was run. I ended up with all the containers stopped!

Regards,
Piotr


-- System Information:
Debian Release: 9.1
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32.bsd54g54 (SMP w/24 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages needrestart depends on:
ii  dpkg                       1.18.24
ii  gettext-base               0.19.8.1-2
ii  libintl-perl               1.26-2
ii  libmodule-find-perl        0.13-1
ii  libmodule-scandeps-perl    1.23-1
ii  libproc-processtable-perl  0.53-2
ii  libsort-naturally-perl     1.03-1
ii  libterm-readkey-perl       2.37-1
ii  perl                       5.24.1-3+deb9u1
ii  xz-utils                   5.2.2-1.2+b1

Versions of packages needrestart recommends:
ii  sysvinit-core  2.88dsf-59.9

Versions of packages needrestart suggests:
pn  needrestart-session | libnotify-bin  <none>

-- no debconf information
--- needrestart-2.11.orig/needrestart
+++ needrestart-2.11/needrestart
@@ -181,7 +181,7 @@ $nrconf{ui} = qq(NeedRestart::UI::stdio)
 die "Hook directory '$nrconf{hook_d}' is invalid!\n" unless(-d $nrconf{hook_d} 
|| $opt_b);
 $opt_r = $nrconf{restart} unless(defined($opt_r));
 die "ERROR: Unknown restart option '$opt_r'!\n" unless($opt_r =~ /^(l|i|a)$/);
-$is_tty++ if($opt_r eq 'i' && exists($ENV{DEBIAN_FRONTEND}) && 
$ENV{DEBIAN_FRONTEND} eq 'noninteractive');
+$is_tty = 0 if($opt_r eq 'i' && exists($ENV{DEBIAN_FRONTEND}) && 
$ENV{DEBIAN_FRONTEND} eq 'noninteractive');
 $opt_r = 'l' if(!$is_tty && $opt_r eq 'i');
 
 $opt_m = $nrconf{ui_mode} unless(defined($opt_m));

--- End Message ---
--- Begin Message ---
Source: needrestart
Source-Version: 2.11-3+deb9u1

We believe that the bug you reported is fixed in the latest version of
needrestart, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Dominik George <[email protected]> (supplier of updated needrestart package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Wed, 28 Feb 2018 22:48:43 +0100
Source: needrestart
Binary: needrestart
Architecture: source
Version: 2.11-3+deb9u1
Distribution: stretch
Urgency: medium
Maintainer: Patrick Matthäi <[email protected]>
Changed-By: Dominik George <[email protected]>
Description:
 needrestart - check which daemons need to be restarted after library upgrades
Closes: 876459
Changes:
 needrestart (2.11-3+deb9u1) stretch; urgency=medium
 .
   * Non-maintainer upload.
   * Fix switching to list mode if debconf is run non-interactively.
     (Closes: #876459)
Checksums-Sha1:
 86d3463b7fb7d58286ff86712688c0e664958480 1793 needrestart_2.11-3+deb9u1.dsc
 99c83897981c1a2dbdab35397c4ca823fc5a7f05 8072 
needrestart_2.11-3+deb9u1.debian.tar.xz
 cc3dbff340a3f5fab7203a8375dfa2e9106e131e 5941 
needrestart_2.11-3+deb9u1_source.buildinfo
Checksums-Sha256:
 c3032b4bec507bc33618e6cd0ee65a8abf3505460e9bf6f347b6736c7c2b9559 1793 
needrestart_2.11-3+deb9u1.dsc
 16471a1dfc0d45aa6990e31aa0d83a9d4b58ad71beac3a0d9629db92f59aed47 8072 
needrestart_2.11-3+deb9u1.debian.tar.xz
 092ab0eaccb8c269e0e4637bba895fe0a996c18eaeee4e81d673b07803380538 5941 
needrestart_2.11-3+deb9u1_source.buildinfo
Files:
 17fbfdef1d0b8e4d94fff139816287ac 1793 admin optional 
needrestart_2.11-3+deb9u1.dsc
 572fa52c4ac6fd6eb90b8114807128e7 8072 admin optional 
needrestart_2.11-3+deb9u1.debian.tar.xz
 750c4861af8519c681b2f22519f7c2ea 5941 admin optional 
needrestart_2.11-3+deb9u1_source.buildinfo

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEm/uu6GwKpf+/IgeCmvRrMCV3GzEFAlqZ4zwACgkQmvRrMCV3
GzFefw//eFlzCVPJbuLEQJ8cMtQk+YTdOY3cEJMALN/n6n1/KjvJs4KC767eXypd
2QIB+o7XOqUAOvXuEO3OprVHbg2p4n7y/YXAzEHIlDg7dEDI00OvhHo3WAKpFBxB
nRgVvYH8/2v2dXjzjRJ9hZLPAXiOOGejqb4BlAzFg6iLcRKYZeOd8d8KZr1d4Mg9
tvzMPGLDlFaIkRA6Dh0qolLvewXGxJTqlxFdGVoC7c54DbxNVJGqXA0ekADhCkW7
EUk857Dng671B6V35v9ED+r40zWxjfvPNGzZMSvW/Hq/t/Rr5nkG5sXvgtOoQvG/
+JelcGXmg/F87oTGEGHLa9Qk3qK7oEy5tJS6wpBo3NT4Fm3RcVYOlx4LhcmIbG3g
xGd0cDKgHKiobL1d7IlzzDijejpmGvDuRzaZ2yVncqZnLndWNlnEAsCdwqV7t3p5
Ryh99AxJdjnSSyNudOvOXaqqW+sIz9+OsSgOwr/YP47IPXaIebgbWyS4dUwWTrAS
O/R21qJUgJ9d4dmOqvHAUaXxsTu01hcN9arewoqyouh5rIeqb+ROOzno2tlDA64Z
SxhN4EbZI4X7q63OPpGmUOVc10MMT9EJrKZgv4u+wF02DY+dsjRxlXZaxLxONFVz
5+asMOh6Gwr/JmMCoqKKAxJxMPDMOEKiWWbqtEU0yw4WBwqCbrU=
=U69l
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to