Followup-For: Bug #720136
Control: tag -1 patch

Hi,

attached patch fixes the update-alternatives usage in fluxbox.
There is no need to check for the existence of u-a (especially in the
wrong location) since this is part of dpkg.
Do not touch the current alternative in preinst - remove+reinstall would
destroy local customization.

I intent to NMU fluxbox to not ship jessie with this bug.


Andreas
diff -Nru fluxbox-1.3.5/debian/changelog fluxbox-1.3.5/debian/changelog
--- fluxbox-1.3.5/debian/changelog	2013-05-15 03:03:42.000000000 +0200
+++ fluxbox-1.3.5/debian/changelog	2014-01-22 05:27:37.000000000 +0100
@@ -1,3 +1,13 @@
+fluxbox (1.3.5-1.1) UNRELEASED; urgency=medium
+
+  * Non-maintainer upload.
+  * preinst, prerm: Do not test for the existence of update-alternatives
+    (this is part of dpkg).  (Closes: #720136)
+  * Do not remove+reinstall the current alternative on upgrades as that could
+    override local customization.
+
+ -- Andreas Beckmann <[email protected]>  Wed, 22 Jan 2014 05:18:24 +0100
+
 fluxbox (1.3.5-1) unstable; urgency=low
 
   * Release from Experimental to Unstable. No changes.
diff -Nru fluxbox-1.3.5/debian/fluxbox.preinst fluxbox-1.3.5/debian/fluxbox.preinst
--- fluxbox-1.3.5/debian/fluxbox.preinst	2013-02-24 04:18:30.000000000 +0100
+++ fluxbox-1.3.5/debian/fluxbox.preinst	2014-01-22 05:24:04.000000000 +0100
@@ -14,10 +14,7 @@
   fi
 
   # old version contain 'fluxbox' current - 'startfluxbox'
-  if test -x /usr/sbin/update-alternatives; then
     update-alternatives --remove x-window-manager /usr/bin/fluxbox
-    update-alternatives --remove x-session-manager /usr/bin/startfluxbox
-  fi
 
 fi
 
diff -Nru fluxbox-1.3.5/debian/fluxbox.prerm fluxbox-1.3.5/debian/fluxbox.prerm
--- fluxbox-1.3.5/debian/fluxbox.prerm	2013-02-24 04:18:30.000000000 +0100
+++ fluxbox-1.3.5/debian/fluxbox.prerm	2014-01-22 05:20:23.000000000 +0100
@@ -4,9 +4,7 @@
 case "$1" in
   remove)
 
-    if test -x /usr/sbin/update-alternatives; then
       update-alternatives --remove x-window-manager /usr/bin/startfluxbox
-    fi
     ;;
 
   *)

Reply via email to