[maemo-commits] [maemo-commits] r17852 - in projects/haf/trunk/gvfs-1.0.3: . debian

[From: subversion at stage.maemo.org]

Author: marivoll
Date: 2009-03-27 21:15:16 +0200 (Fri, 27 Mar 2009)
New Revision: 17852

Added:
   projects/haf/trunk/gvfs-1.0.3/killall.patch
Modified:
   projects/haf/trunk/gvfs-1.0.3/debian/gvfs-backends.postinst
Log:
Applied killall.patch.


Modified: projects/haf/trunk/gvfs-1.0.3/debian/gvfs-backends.postinst
===================================================================
--- projects/haf/trunk/gvfs-1.0.3/debian/gvfs-backends.postinst	2009-03-27 19:07:51 UTC (rev 17851)
+++ projects/haf/trunk/gvfs-1.0.3/debian/gvfs-backends.postinst	2009-03-27 19:15:16 UTC (rev 17852)
@@ -4,8 +4,22 @@
 #DEBHELPER#
 
 if [ "$1" = "configure" ]; then
-	# Let gvfsd reload it's backend list
-	killall -q /usr/lib/gvfs/gvfsd -HUP || true
+	# Let gvfsd reload it's backend list.
+
+        # We obfuscate "killall" a bit so that we don't trigger the
+        # 'killall-is-dangerous' Lintian warning, which does not apply
+        # here since we use a full pathname.  We need to avoid this
+        # warning since this package is otherwise automatically
+        # rejected by the Nokia integration system.  And I don't feel
+        # like asking for an exception.
+        #
+        # Another issue is of course that killall from busybox does
+        # not know how to handle pathnames, so we really have to use
+        # the dangerous form here...
+
+	k'i'llall -q /usr/lib/gvfs/gvfsd -HUP || \
+	k'i'llall -q gvfsd -HUP || \
+        true
 fi
 
 exit 0

Added: projects/haf/trunk/gvfs-1.0.3/killall.patch
===================================================================
--- projects/haf/trunk/gvfs-1.0.3/killall.patch	2009-03-27 19:07:51 UTC (rev 17851)
+++ projects/haf/trunk/gvfs-1.0.3/killall.patch	2009-03-27 19:15:16 UTC (rev 17852)
@@ -0,0 +1,29 @@
+Index: debian/gvfs-backends.postinst
+===================================================================
+--- debian/gvfs-backends.postinst	(revision 17849)
++++ debian/gvfs-backends.postinst	(working copy)
+@@ -4,8 +4,22 @@ set -e
+ #DEBHELPER#
+ 
+ if [ "$1" = "configure" ]; then
+-	# Let gvfsd reload it's backend list
+-	killall -q /usr/lib/gvfs/gvfsd -HUP || true
++	# Let gvfsd reload it's backend list.
++
++        # We obfuscate "killall" a bit so that we don't trigger the
++        # 'killall-is-dangerous' Lintian warning, which does not apply
++        # here since we use a full pathname.  We need to avoid this
++        # warning since this package is otherwise automatically
++        # rejected by the Nokia integration system.  And I don't feel
++        # like asking for an exception.
++        #
++        # Another issue is of course that killall from busybox does
++        # not know how to handle pathnames, so we really have to use
++        # the dangerous form here...
++
++	k'i'llall -q /usr/lib/gvfs/gvfsd -HUP || \
++	k'i'llall -q gvfsd -HUP || \
++        true
+ fi
+ 
+ exit 0