Package: release.debian.org
Severity: normal
User: release.debian....@packages.debian.org
Usertags: unblock

Please update the unblock for package needrestart-session to 0.3-4.1,
it fixes being able to find processes that need to be restarted and
fixes a crash when double-clicking on apps to highlight their windows.

The first one makes this package actually work and the second one
fixes a feature that is advertised directly in the user interface
so is likely to cause complaints from users when it causes crashes.

unblock needrestart-session/0.3-4.1

The maintainer requested that I do this upload/unblock on his behalf.

-- 
bye,
pabs

https://wiki.debian.org/PaulWise
diff -Nru needrestart-session-0.3/debian/changelog needrestart-session-0.3/debian/changelog
--- needrestart-session-0.3/debian/changelog	2017-03-10 23:18:08.000000000 +0800
+++ needrestart-session-0.3/debian/changelog	2017-03-14 15:37:11.000000000 +0800
@@ -1,3 +1,11 @@
+needrestart-session (0.3-4.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Fix path to needrestart, now shows same results (Closes: #787291)
+  * Don't fork before system(), fixes crash on double-click (Closes: #857703)
+
+ -- Paul Wise <p...@debian.org>  Tue, 14 Mar 2017 15:37:11 +0800
+
 needrestart-session (0.3-4) unstable; urgency=high
 
   * Add dependency on adduser and add/delete new system user needrestart-dbus.
diff -Nru needrestart-session-0.3/debian/patches/fix-crash.patch needrestart-session-0.3/debian/patches/fix-crash.patch
--- needrestart-session-0.3/debian/patches/fix-crash.patch	1970-01-01 08:00:00.000000000 +0800
+++ needrestart-session-0.3/debian/patches/fix-crash.patch	2017-03-14 15:35:06.000000000 +0800
@@ -0,0 +1,16 @@
+Description: fix crash when double-clicking on items in the list
+Author: Paul Wise <p...@debian.org>
+Bug-Debian: https://bugs.debian.org/857703
+Forwarded: no
+Last-Update: 2017-03-14
+--- a/needrestart-x11
++++ b/needrestart-x11
+@@ -272,8 +272,6 @@
+ sub OnAppsDClick {
+     my ($frame, $event) = @_;
+ 
+-    my $pid = fork();
+-
+     system(qw(wmctrl -i -a), $plist[$event->GetIndex]->{winid}) if(exists($plist[$event->GetIndex]->{winid}));
+ }
+ 
diff -Nru needrestart-session-0.3/debian/patches/fix-path.patch needrestart-session-0.3/debian/patches/fix-path.patch
--- needrestart-session-0.3/debian/patches/fix-path.patch	1970-01-01 08:00:00.000000000 +0800
+++ needrestart-session-0.3/debian/patches/fix-path.patch	2017-03-14 15:35:39.000000000 +0800
@@ -0,0 +1,16 @@
+Description: fix failure to find processes to restart
+Author: Paul Wise <p...@debian.org>
+Bug-Debian: https://bugs.debian.org/787291
+Forwarded: no
+Last-Update: 2017-03-14
+--- a/needrestart-x11
++++ b/needrestart-x11
+@@ -132,7 +132,7 @@
+ 
+     # get user processes required to be restarted
+     my %fnames;
+-    my $fh = nr_fork_pipe(0, qw(needrestart -b));
++    my $fh = nr_fork_pipe(0, qw(/usr/sbin/needrestart -b));
+     $frProgress->SetTitle(q(Scanning processes...)) if(defined($frProgress));
+     while(<$fh>) {
+ 	chomp;
diff -Nru needrestart-session-0.3/debian/patches/series needrestart-session-0.3/debian/patches/series
--- needrestart-session-0.3/debian/patches/series	1970-01-01 08:00:00.000000000 +0800
+++ needrestart-session-0.3/debian/patches/series	2017-03-14 15:32:14.000000000 +0800
@@ -0,0 +1,2 @@
+fix-crash.patch
+fix-path.patch

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to