Merge lp:~mitya57/ubuntu/trusty/python-coverage/add-preinst-script into lp:ubuntu/trusty/python-coverage

[Dmitry Shachnev]

This fixes Saucy→Trusty upgrades (the problem was described at http://bugs.debian.org/734850, but actually it's Ubuntu-specific problem, and the fix is not needed in Debian).

[Daniel Holbach (dholbach)]

As we have two supported python3 versions, you might need this excerpt in the preinst too:

   for python in $(py3versions --supported) ; do

......

review: Needs Fixing

[Daniel Holbach (dholbach)]

Sorry, ignore the last comment please.

[Daniel Holbach (dholbach)]

Thanks. Uploaded.

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2014-01-13 14:51:52 +0000
+++ debian/changelog	2014-01-20 15:19:50 +0000
@@ -1,3 +1,12 @@
+python-coverage (3.7+dfsg.1-4ubuntu1) trusty; urgency=medium
+
+  * Add python3-coverage.preinst (based on existing python-coverage.preinst)
+    to correctly handle upgrades from previous releases. Previously
+    /usr/lib/python3/dist-packages/coverage/htmlfiles was a directory,
+    now it is a symbolic link (see Debian #734850).
+
+ -- Dmitry Shachnev <[e-mail]>  Mon, 20 Jan 2014 19:08:47 +0400
+
 python-coverage (3.7+dfsg.1-4) unstable; urgency=medium
 
   * The “Thorpekai Yousafzai” release.
 
=== added file 'debian/python3-coverage.preinst'
--- debian/python3-coverage.preinst	1970-01-01 00:00:00 +0000
+++ debian/python3-coverage.preinst	2014-01-20 15:19:50 +0000
@@ -0,0 +1,38 @@
+#! /bin/sh
+# Pre-install script for ‘python3-coverage’.
+#
+# Manpage: ‘dh_installdeb(1)’
+
+set -e
+
+# Summary of ways this script can be called:
+#   * <new-preinst> install
+#   * <new-preinst> install <old-version>
+#   * <new-preinst> upgrade <old-version>
+#   * <old-preinst> abort-upgrade <new-version>
+# For details, see the Debian Policy §6.5 in the ‘debian-policy’ package
+# or on the web at <URL:http://www.debian.org/doc/debian-policy/>.
+
+action="$1"
+
+case "$action" in
+    upgrade)
+        htmlfiles_dir="/usr/lib/python3/dist-packages/coverage/htmlfiles"
+        if [ -d "$htmlfiles_dir" ] && [ ! -L "$htmlfiles_dir" ] ; then
+            # The ‘htmlfiles’ location should be platform-independent.
+            # The new package will replace the directory with a symlink.
+            rm -rf "$htmlfiles_dir"
+        fi
+        ;;
+
+    install|abort-upgrade)
+        ;;
+
+    *)
+        printf "preinst called with unknown action ‘%s’\n" "$action" >&2
+        exit 1
+        ;;
+
+esac
+
+#DEBHELPER#